Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename PACK_ env vars to use CNB_ prefix instead #38

Merged
merged 1 commit into from
Jan 29, 2019
Merged

Conversation

sclevine
Copy link
Member

No description provided.

Signed-off-by: Stephen Levine <stephen.levine@gmail.com>
@hone
Copy link
Member

hone commented Jan 22, 2019

Do we need to put deprecation stuff in the spec or since we're < 1.0 we can just ignore? Looks good otherwise.

@nebhale
Copy link
Contributor

nebhale commented Jan 23, 2019

Change at will for now. Some coordination is required the closer we get to 1.0 but no problem making breaking changes for now.

@nebhale
Copy link
Contributor

nebhale commented Jan 23, 2019

And @hone can you please officially approve in the GitHub review feature?

@sclevine sclevine merged commit eb01a23 into master Jan 29, 2019
nebhale added a commit to buildpacks/libbuildpack that referenced this pull request Jan 29, 2019
This change updates the code to examine CNB_ environment variables instead of
PACK_ environment variables.  Where PACK_ was previously used, it still exists
as a fallback to be removed once the lifecycle has been updated and released.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
nebhale added a commit to cloudfoundry/libcfbuildpack that referenced this pull request Jan 29, 2019
This change updates dependencies to support CNB_ environment variables.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
nebhale added a commit to cloudfoundry/openjdk-cnb that referenced this pull request Jan 29, 2019
This change updates dependencies to support CNB_ environment variables.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
nebhale added a commit to cloudfoundry/jvm-application-cnb that referenced this pull request Jan 29, 2019
This change updates dependencies to support CNB_ environment variables.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
nebhale added a commit to cloudfoundry/build-system-cnb that referenced this pull request Jan 29, 2019
This change updates dependencies to support CNB_ environment variables.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
nebhale added a commit to cloudfoundry/debug-cnb that referenced this pull request Jan 29, 2019
This change updates dependencies to support CNB_ environment variables.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
nebhale added a commit to cloudfoundry/jmx-cnb that referenced this pull request Jan 29, 2019
This change updates dependencies to support CNB_ environment variables.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
nebhale added a commit to cloudfoundry/azure-application-insights-cnb that referenced this pull request Jan 29, 2019
This change updates dependencies to support CNB_ environment variables.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
nebhale added a commit to cloudfoundry/google-stackdriver-cnb that referenced this pull request Jan 29, 2019
This change updates dependencies to support CNB_ environment variables.

[buildpacks/spec#38]

Signed-off-by: Ben Hale <bhale@pivotal.io>
@nebhale nebhale deleted the change-prefix branch August 30, 2019 17:14
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.

None yet

4 participants