Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[cometvisu UI] Depends on Full JRE #4

Closed
svilenvul opened this issue Mar 30, 2018 · 2 comments
Closed

[cometvisu UI] Depends on Full JRE #4

svilenvul opened this issue Mar 30, 2018 · 2 comments
Labels
cometvisu ui Cometvisu UI

Comments

@svilenvul
Copy link

Looking at the report generated by openhab/static-code-analysis#283, I see that several packages in cometvisu UI depend on Full JRE:

  • org.openhab.ui.cometvisu.internal.config depends on javax.xml.bind (Full JRE)
  • org.openhab.ui.cometvisu.internal.config depends on javax.xml.bind.annotation.adapters (Full JRE)
  • org.openhab.ui.cometvisu.internal.config.beans depends on javax.xml.bind (Full JRE)
  • org.openhab.ui.cometvisu.internal.config.beans depends on javax.xml.bind.annotation (Full JRE)
  • org.openhab.ui.cometvisu.internal.config.beans depends on javax.xml.bind.annotation.adapters (Full JRE)

This is a violation of C.1 in the openHAB coding guidelines https://docs.openhab.org/developers/development/guidelines.html#c-language-levels-and-libraries.

It will be good if we can refactor it, so the bundle remains restricted to Compact Profile 2, or if it is not possible document that the binding is an exception.

@kaikreuzer kaikreuzer transferred this issue from openhab/openhab-addons Feb 20, 2019
@kubawolanin kubawolanin added the cometvisu ui Cometvisu UI label Mar 8, 2019
@peuter
Copy link
Member

peuter commented Apr 15, 2020

Is this still relevant? The link to the documenation is not woring anymore. It its relevant I would need some guidance on how to replace that.

@wborn
Copy link
Member

wborn commented Jan 1, 2023

This can be closed. JAXB became an optional dependency with Java 11.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cometvisu ui Cometvisu UI
Projects
None yet
Development

No branches or pull requests

5 participants