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

Set array of envvars at once #3

Open
jayvdb opened this issue May 21, 2019 · 1 comment
Open

Set array of envvars at once #3

jayvdb opened this issue May 21, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@jayvdb
Copy link
Owner

jayvdb commented May 21, 2019

Rather than using backticks multiple times to set several envvars, there are times when it is more logical and efficient to dynamically emit an array of envvars to be added to setenv.

This is possibly a different tox plugin, at least because it should be done with a different testenv key instead of setenv (indeed, even the backticks could have been implemented using a different key setenv_dynamic).

@jayvdb
Copy link
Owner Author

jayvdb commented May 22, 2019

If any item in setenv refers to items created by setenv_dynamic, or inverse, this plugin is the correct place, as the algorithm to determine the correct order of evaluation, and invalidation of old envvars, is similar to backticks.

First chuck of analysis needed is whether/when tox config parser will barf if the setenv list uses a {env:foo} which has not been defined. This will determine when setenv_dynamic must be processed if the setenv list refers to envvar created by setenv_dynamic.

@jayvdb jayvdb added the enhancement New feature or request label Jul 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant