You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by yuruki <gi...@git.apache.org> on 2014/07/16 11:31:34 UTC

[GitHub] camel pull request: Add system property and env variable resolving...

GitHub user yuruki opened a pull request:

    https://github.com/apache/camel/pull/221

    Add system property and env variable resolving to property prefix and suffix

    This change adds system property and environment variable resolving to PropertiesComponent's propertyPrefix and propertySuffix. This makes custom PropertiesParser unnecessary in the use case stated here: https://issues.jboss.org/browse/MR-854.
    
    Also renamed org.apache.camel.util.FilePathResolver to SystemAndEnvPropertyResolver as its functionality has nothing to do with files.
    
    Finally, fixed unintuitive behavior of override properties not being used if there are no properties loaded from elsewhere. Now you can use override properties to put properties into PropertiesComponent in plain Java (handy in unit tests or in the SCR world, for example)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/yuruki/camel master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/221.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #221
    
----
commit dea843521d7edda59608c214c3df64cb94c860da
Author: Jyrki Ruuskanen <yu...@kotikone.fi>
Date:   2014-07-16T07:12:46Z

    Renamed org.apache.camel.util.FilePathResolver -> SystemAndEnvPropertyResolver as its function has nothing to do with files.
    Added system property and environment variable resolving to propertyPrefix and propertySuffix in org.apache.camel.component.properties.PropertiesComponent.
    Use override properties even when there are no properties to override.

commit 3937b89f91f024627f8de0be762e716c06441c52
Author: Jyrki Ruuskanen <yu...@kotikone.fi>
Date:   2014-07-16T07:40:02Z

    Added tests for property prefix/suffix resolving.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] camel pull request: Add system property and env variable resolving...

Posted by yuruki <gi...@git.apache.org>.
Github user yuruki closed the pull request at:

    https://github.com/apache/camel/pull/221


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---