You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/11/22 08:52:39 UTC
[GitHub] [camel] Croway opened a new pull request #6459: [CAMEL-17216] Allow to include properties file
Croway opened a new pull request #6459:
URL: https://github.com/apache/camel/pull/6459
Hi, properties file can be added via --properties option, for example
```
jbang -Dcamel.jbang.version=3.14.0 CamelJBang@apache/camel run github:apache:camel-k:examples/languages/routes.yaml --properties /path/to/application.properties
```
--
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 merged pull request #6459: [CAMEL-17216] Allow to include properties file
Posted by GitBox <gi...@apache.org>.
davsclaus merged pull request #6459:
URL: https://github.com/apache/camel/pull/6459
--
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 #6459: [CAMEL-17216] Allow to include properties file
Posted by GitBox <gi...@apache.org>.
davsclaus commented on pull request #6459:
URL: https://github.com/apache/camel/pull/6459#issuecomment-975371328
What if you have a properties file in the current folder, or a sub folder, where you do not have a leading slash, eg
```
jbang -Dcamel.jbang.version=3.14.0 CamelJBang@apache/camel run github:apache:camel-k:examples/languages/routes.yaml --properties my-prop.properties
```
or
```
jbang -Dcamel.jbang.version=3.14.0 CamelJBang@apache/camel run github:apache:camel-k:examples/languages/routes.yaml --properties ../config/application.properties
```
The default resolver is classpath, but this wont work with jbang (well it would be odd to have a JAR that embeds .properties that should be loaded). So we could maybe always just use file: as scheme prefix
--
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] Croway commented on pull request #6459: [CAMEL-17216] Allow to include properties file
Posted by GitBox <gi...@apache.org>.
Croway commented on pull request #6459:
URL: https://github.com/apache/camel/pull/6459#issuecomment-975416447
Good point, the latest commit pre append the absolute path in case of missing /. I tested it with
```
jbang -Dcamel.jbang.version=3.14.0-SNAPSHOT CamelJBang@apache/camel run github:Croway:camel-k:examples/languages/routes.yaml --properties /home/federico/test.properties
jbang -Dcamel.jbang.version=3.14.0-SNAPSHOT CamelJBang@apache/camel run github:Croway:camel-k:examples/languages/routes.yaml --properties file:///home/federico/test.properties
jbang -Dcamel.jbang.version=3.14.0-SNAPSHOT CamelJBang@apache/camel run github:Croway:camel-k:examples/languages/routes.yaml --properties ../test.properties
jbang -Dcamel.jbang.version=3.14.0-SNAPSHOT CamelJBang@apache/camel run github:Croway:camel-k:examples/languages/routes.yaml --properties test.properties
```
and they are working
--
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