-
Notifications
You must be signed in to change notification settings - Fork 3
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
env dependency ignores vars which dont use backticks #6
Labels
bug
Something isn't working
Comments
Couldnt reproduce with: [testenv:indirect]
setenv =
FOO=`python -c 'import sys; sys.stdout.write("foo")'`
BAR={env:FOO}
BAZ=`python -c 'import sys; sys.stdout.write("{env:BAR}")'`
commands =
python -c 'assert "{env:BAZ}" == "foo"' |
It does occur on py27, so likely it is passing on py37 due to dict ordering. |
jayvdb
added a commit
that referenced
this issue
Jul 17, 2019
jayvdb
added a commit
that referenced
this issue
Jul 17, 2019
A real solution has dragons everywhere. The regex is the problem, as it doesnt allow identification of If I dont find a good fix for that soon, I'll go with my existing solution which ignores that, or add a second make-shift dodgy regex which can handle those some of the time. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Current algorithm doesnt see that
c
must be deferred until aftera
The text was updated successfully, but these errors were encountered: