Python-Setup: run auto_install_packages.py with -B flag#1126
Conversation
This reverts commit 3ff1fd9.
This avoids creating a __pycache__ folder in the _actions folder, which may cause file ownership problems on self-hosted runners when run in a docker container.
|
Is there any performance improvements to creating a pycache folder? We could use the |
|
Ah...I see you already went over that in the internal issue. That option only exists in 3.8 or later. If there is likely to be noticeable speedup when using the pycache, then we can first detect which python version we are using and only use the option for 3.8 or later. I suspect that this is not the case, though and what you have is fine. |
|
out of curiosity, why did you choose to revert the |
I meant to mention that approach in the PR and ask whether it would be preferred or not. The drawback of setting the environment variable is that it affects all |
dabaeef to
cae9a1f
Compare
4cdea43 to
53bc5e6
Compare
This avoids creating a pycache folder in the _actions folder, which
may cause file ownership problems on self-hosted runners
when run in a docker container.
Merge / deployment checklist