Skip to content

Conversation

cgfrost
Copy link
Contributor

@cgfrost cgfrost commented Mar 24, 2015

This commit adds support for overriding buildpack configuration with
environment variables. The way Java config is processed has been
modified to allow this new support to work when configuring the
required version of Java. Includes new tests and documentation, see
the main README.md for more details.

[#86997400]

@cfdreddbot
Copy link

Hey cgfrost!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you've already signed the CLA.

@youngm
Copy link
Contributor

youngm commented Mar 24, 2015

Looks good to me though perhaps the [version: 1.7.0_+] syntax could use a little more explanation. How would I specify more than one config entry for a file?

@jgawor
Copy link
Contributor

jgawor commented Mar 25, 2015

How about using JBP_CONFIG instead of JAVA_BUILDPACK_CONFIG as the environment variable prefix? It's a bit shorter and makes it consistent with the JBP_LOG_LEVEL environment variable.

@cgfrost
Copy link
Contributor Author

cgfrost commented Mar 25, 2015

@jgawor Good point, I'll make that change. I'll update the example to show a bigger change as well.

README.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Broken link

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which link? They are all working for me...

This commit adds support for overriding buildpack configuration with
environment variables. The way Java config is processed has been
modified to allow this new support to work when configuring the
required version of Java. Includes new tests and documentation, see
the main README.md for more details.

[#86997400]
@cgfrost
Copy link
Contributor Author

cgfrost commented Mar 25, 2015

Updated

@nebhale nebhale merged commit 3268448 into cloudfoundry:master Mar 25, 2015
@cgfrost cgfrost deleted the 86997400-user-provided-config branch March 25, 2015 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants