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

Replace quarkus.camel.native.resources.* with quarkus.native.resources.* #988

Closed
ppalaga opened this issue Mar 26, 2020 · 6 comments · Fixed by #2813
Closed

Replace quarkus.camel.native.resources.* with quarkus.native.resources.* #988

ppalaga opened this issue Mar 26, 2020 · 6 comments · Fixed by #2813
Assignees
Milestone

Comments

@ppalaga
Copy link
Contributor

ppalaga commented Mar 26, 2020

Native resources and reflection inclusion and exclusion is a very generic functionality that all Quarkus users could benefit from.

@ppalaga ppalaga self-assigned this Mar 31, 2020
@ppalaga
Copy link
Contributor Author

ppalaga commented Mar 31, 2020

Proposed the idea on quarkus-dev https://groups.google.com/forum/#!topic/quarkus-dev/SLuS7DiDMVA

@lburgazzoli
Copy link
Contributor

@ppalaga the pr quarkusio/quarkus#8484 has been merged so we can probably close this one ?

@ppalaga
Copy link
Contributor Author

ppalaga commented May 29, 2020

The original idea was to deprecate and later remove quarkus.camel.native.resources.include-patterns and quarkus.camel.native.resources.exclude-patterns from Camel Quarkus once the same functionality is available in Quarkus. However, quarkusio/quarkus#8484 adds only support for includes. Excludes are blocked by oracle/graal#2347
So I'd keep this open as long as there is any hope to get excludes from Quarkus.

@ppalaga
Copy link
Contributor Author

ppalaga commented Nov 25, 2020

Blocked by quarkusio/quarkus#13476

@aldettinger
Copy link
Contributor

Discussed with @jamesnetherton and we should update all of the tests and deprecate quarkus.camel.native.resources.*

@ppalaga ppalaga added this to the 2.0.0 milestone May 26, 2021
@ppalaga ppalaga changed the title Consider moving quarkus.camel.native.* to Quarkus Consider moving quarkus.camel.native.resources.* to Quarkus Jun 21, 2021
@ppalaga
Copy link
Contributor Author

ppalaga commented Jun 21, 2021

I am narrowing the scope from quarkus.camel.native.* to quarkus.camel.native.resources.*.

We still have quarkus.camel.native.reflection.* which does not have a drop in replacement in Quarkus and I do not think it is a big issue to keep it as is for now.

@ppalaga ppalaga changed the title Consider moving quarkus.camel.native.resources.* to Quarkus Replace quarkus.camel.native.resources.* with quarkus.native.resources.* Jun 21, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 21, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 21, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants