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.

Sign up for GitHub

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

Jump to bottom

Make xml and jaxb disabled by default and opt-in when depending on camel #188

Closed
gnodet opened this issue Sep 6, 2019 · 4 comments
Closed

Make xml and jaxb disabled by default and opt-in when depending on camel #188

gnodet opened this issue Sep 6, 2019 · 4 comments
Assignees
@gnodet
Milestone
1.0.0

Comments

@gnodet
Copy link
Contributor

gnodet commented Sep 6, 2019

No description provided.

@lburgazzoli
Copy link
Contributor

@gnodet in quarkus master there is a new annotation @Overridable that could help here to provide a specialized implementation with xml support.

Here a very simple example: https://github.com/lburgazzoli/quarkus-conditional-buildstep/blob/master/deployment/src/main/java/com/github/lburgazzoli/quarkus/playground/DeploymentProcessor.java

@lburgazzoli
Copy link
Contributor

Also note that with https://github.com/quarkusio/quarkus/pull/4405/files#diff-3, JaxbEnabledBuildItem is not more there in quarkus master

@gnodet
Copy link
Contributor Author

gnodet commented Oct 9, 2019

Working on it, i just need a minor adaption in camel apache/camel#3234

gnodet added a commit to gnodet/camel-quarkus that referenced this issue Oct 9, 2019
@gnodet
Make xml and jaxb disabled by default and opt-in when depending on ca…
1008658 
…mel apache#188
@gnodet gnodet mentioned this issue Oct 9, 2019
Merged
@lburgazzoli lburgazzoli added this to the quarkus 1.0.0 milestone Oct 15, 2019
gnodet added a commit to gnodet/camel-quarkus that referenced this issue Oct 16, 2019
@gnodet
Make xml and jaxb disabled by default and opt-in when depending on ca…
152e10d 
…mel-quarkus-xml-common, also upgrade to quarkus 0.24.0, apache#188
lburgazzoli pushed a commit that referenced this issue Oct 17, 2019
@gnodet @lburgazzoli
Make xml and jaxb disabled by default and opt-in when depending on ca…
081cea2 
…mel-quarkus-xml-common, also upgrade to quarkus 0.24.0, #188
@lburgazzoli
Copy link
Contributor

Done by #249

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gnodet gnodet

Labels
None yet
Projects
None yet
Milestone
1.0.0
Development

No branches or pull requests
2 participants
@gnodet @lburgazzoli