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

EnvLog does not respect passenv/setenv #1422

Closed
connyay opened this issue Sep 17, 2019 · 2 comments
Closed

EnvLog does not respect passenv/setenv #1422

connyay opened this issue Sep 17, 2019 · 2 comments
Labels
bug:normal affects many people or has quite an impact

Comments

@connyay
Copy link

connyay commented Sep 17, 2019

Hello!

This will likely be a crappy bug report because I don't have an isolated repro. Sorry 😞

Working with tox in a pretty strange environment that basically uses relocatable python installations in specific directories. Noticed on centos boxes an environment could not be created due to the .so not being loaded properly.

Tracked it down to venc.setupenv on this line. Noticed that EnvLog. set_python_info has no concept of the configuration's passenv or setenv so the LD_LIBRARY_PATH does not get set.

Couldn't quickly grok a fix for this. action.popen looks promising? but not sure the implications.

Let me know if I can provide any further info.

@connyay connyay added the bug:normal affects many people or has quite an impact label Sep 17, 2019
@gaborbernat
Copy link
Member

Feel free to open a PR against master and fix it. My available efforts at the moment are aimed at fixing this as part of #1394, but that probably will take a while (ETA September).

@gaborbernat
Copy link
Member

This issue has now been avoided in the rewrite branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:normal affects many people or has quite an impact
Projects
None yet
Development

No branches or pull requests

2 participants