-
Notifications
You must be signed in to change notification settings - Fork 428
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
[Feature] Add a new command line argument to set environment variables #1054
Comments
Yep, those both sound like great ideas, @frol. I'd much prefer we even specify |
@jakirkham That was also a part of the intention behind it. Currently, somebody can potentially hit some nasty issues if they use something fancy in their |
In that case you may want to revert these: #917 conda/conda-docs#270 cc @jakirkham |
cc @ocefpaf |
Hi there, thank you for your contribution! This issue has been automatically marked as stale because it has not had recent activity. It will be closed automatically if no further activity occurs. If you would like this issue to remain open please:
NOTE: If this issue was closed prematurely, please leave a comment. Thanks! |
Currently, you can specify a list of passthrough environment variables in
meta.yaml
(build/script_env
). However, there are use-cases when it is not feasible to list all possible env variables. One example of such a useful environment variable isMAKEFLAGS
and it is even made into the conda-build core. Imagine you don't need to make extra PRs just to pass through a new build-system specific variable.Please, consider the following syntax (copied from
docker run
command):This way we won't worry about implicitly and unexpectedly passed settings from a host system. Also, there would be no need to list all sorts of possibly useful only for build systems env variables in recipes.
P.S. It might be also a good idea to implement a config file for conda-build where these environment variables and other useful settings can be saved and reused.
/cc @jschueller @groutr @msarahan @jakirkham
The text was updated successfully, but these errors were encountered: