Skip to content

Conversation

@christophd
Copy link
Contributor

Description

The PR improves the camel-kamelets version handling in Camel JBang and makes sure to consistently use the same version in all camel-kamelets-* libraries (e.g. camel-kamelets, camel-kamelets-utils, camel-kamelets-catalog, and so on).

Users either set the version via system properties -Dcamel-kamelets.version= or via command line setting --kamelets-version= when using Camel JBang. As a fallback the value gets derived from the version used by the library camel-kamelets on the current JVM classpath (only when none of the user settings should be set).

  • Avoids inconsistent versions on camel-kamelets dependencies (e.g. camel-kamelets-utils library)
  • Make sure to use same version for camel-kamelets and camel-kamelets-utils dependency
  • Add --kamelets-version setting on Camel JBang run command
  • Propagate camel-kamelets version to KameletMain as initial property and use this version in Kamelet dependency downloader
  • Fallback to deriving the camel-kamelets version from classpath only if version has not been set by the user
  • Support camel-kamelets: prefix when referencing dependencies in Kamelets and Pipe/KameletBinding resources and resolve with proper camel-kamelets version

- Avoids inconsistent versions on camel-kamelets dependencies (e.g. camel-kamelets-utils library)
- Make sure to use same version for camel-kamelets and camel-kamelets-utils dependency
- Add --kamelets-version setting on Camel JBang run command
- Propagate camel-kamelets version to KameletMain as initial property and use this version in Kamelet dependency downloader
- Fallback to deriving the camel-kamelets version from classpath only if version has not been set
- Support camel-kamelets: prefix when referencing dependencies in Kamelets and Pipe/KameletBinding resources and resolve with proper camel-kamelets version
@christophd christophd merged commit f14fb0c into apache:main Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants