Set PATH and other vars in .profile.d script #7

Open
wants to merge 1 commit into
from

Projects

None yet

1 participant

@weibeld
weibeld commented Mar 24, 2015

I moved the setting of the config vars from the YAML hash in the release script to the .profile.d script. This should fix the problem that the config vars, including the PATH, might not be set if the buildpack is used together with other buildpacks (see issue #3 which would be fixed with this).

Setting config vars in the release script also seems to be deprecated, as it is not mentioned here https://devcenter.heroku.com/articles/buildpack-api#buildpack-api (under bin/release), and here https://devcenter.heroku.com/articles/buildpack-api#default-config-values it is advised to use a .profile.d script for adding default config values.

Daniel Weibel Set PATH and other vars in .profile.d script
This fixes the problem that the config vars (including PATH) might not
be set if the buidpack is used together with other buidpacks.
696c21c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment