chala2001 commented on PR #26023:
URL: https://github.com/apache/camel/pull/26023#issuecomment-5505220430
You're right, I checked and the docs don't support this.
`camel.jbang.camel-version` is documented as a configuration option, but
only for
`camel config set` / the `camel-cli.properties` files —
`CamelJBangConstants` says as much
("options that can be defined in application.properties"). Nothing documents
it as a `-D`
system property. The `-Dcamel.jbang.*` examples in the docs are
`camel.jbang.version`, which
is a different option, and I think that's what the issue conflated.
The one thing that made it look intentional is that
`Export.overrideFromSystemProperties()`
already reads the Quarkus platform and `camel.jbang.camelSpringBootVersion`
from system
properties, so `camel-version` is the odd one out in that method. Whether
that's worth making
consistent, or whether those shouldn't be there either, is your call.
Happy to close this and CAMEL-24400 as invalid if the `-D` route isn't meant
to be supported.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
