You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "CMoH (via GitHub)" <gi...@apache.org> on 2023/02/14 00:47:03 UTC
[GitHub] [camel] CMoH opened a new pull request, #9343: `[CAMEL-19045] OSGi integration for camel-cdi based on aries-cdi
CMoH opened a new pull request, #9343:
URL: https://github.com/apache/camel/pull/9343
# Description
There are 2 commits inhere:
- one fix for the camel-xml-jaxb OSGi bundle, which fails to locate the `errorhandler` package
- changes to camel-cdi that integrate it with aries-cdi (CDI 2.0) within an OSGi container
The code is running well in production, but it does require some clean-up before merging. However, given the Camel 4 roadmap, I am presenting this to help understand the path I should take further (see JIRA issue - in short, clean it up for camel, or move my efforts to camel-karaf).
Regardless, the OSGi fix for camel-xml-jaxb might still interest you.
# Target
- [x] I checked that the commit is targeting the correct branch (note that Camel 3 uses `camel-3.x`, whereas Camel 4 uses the `main` branch)
# Tracking
- [x] If this is a large change, bug fix, or code improvement, I checked there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL) filed for the change (usually before you start working on it).
<!--
# *Note*: trivial changes like, typos, minor documentation fixes and other small items do not require a JIRA issue. In this case your pull request should address just this issue, without pulling in other changes.
-->
# Apache Camel coding standards and style
- [x] I checked that each commit in the pull request has a meaningful subject line and body.
<!--
If you're unsure, you can format the pull request title like `[CAMEL-XXX] Fixes bug in camel-file component`, where you replace `CAMEL-XXX` with the appropriate JIRA issue.
-->
- [ ] I formatted the code using `mvn -Pformat,fastinstall install && mvn -Psourcecheck`
<!--
You can run the aforementioned command in your module so that the build auto-formats your code and the source check verifies that is complies with our coding style. This will also be verified as part of the checks and your PR may be rejected if the checkstyle does not pass.
You can learn more about the contribution guidelines at https://github.com/apache/camel/blob/main/CONTRIBUTING.md
-->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] davsclaus closed pull request #9343: `[CAMEL-19045] OSGi integration for camel-cdi based on aries-cdi
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus closed pull request #9343: `[CAMEL-19045] OSGi integration for camel-cdi based on aries-cdi
URL: https://github.com/apache/camel/pull/9343
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] github-actions[bot] commented on pull request #9343: `[CAMEL-19045] OSGi integration for camel-cdi based on aries-cdi
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #9343:
URL: https://github.com/apache/camel/pull/9343#issuecomment-1428928448
:star2: Thank you for your contribution to the Apache Camel project! :star2:
:warning: Please note that the changes on this PR may be **tested automatically**.
If necessary Apache Camel Committers may access logs and test results in the job summaries!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] davsclaus commented on pull request #9343: `[CAMEL-19045] OSGi integration for camel-cdi based on aries-cdi
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #9343:
URL: https://github.com/apache/camel/pull/9343#issuecomment-1430794099
Apache Camel Karaf 3.x only supports what we provide in the features file - there is no support for camel-cdi in OSGi
https://github.com/apache/camel-karaf/blob/camel-karaf-3.x/platforms/karaf/features/src/main/resources/features.xml
We do not plan to support this, and Camel 3.x is "stable" and in low maintenance mode.
A PR to fix the "error handler" in is welcome
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org