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

Consider whitelisting PROCESSOR_ARCHITECTURE environment variable #740

Closed
asottile opened this issue Jan 12, 2018 · 0 comments
Closed

Consider whitelisting PROCESSOR_ARCHITECTURE environment variable #740

asottile opened this issue Jan 12, 2018 · 0 comments
Labels
bug:minor does not affect many people or has no big impact level:easy rought estimate that this might be quite easy to implement os:windows

Comments

@asottile
Copy link
Contributor

On windows, this is necessary for platform.machine() to function

$ tox --version
2.9.1 imported from c:\users\anthony\appdata\local\temp\venv\lib\site-packages\tox\__init__.pyc
[tox]
envlist = py27
skipsdist = true

[testenv]
commands =
    python -c 'import platform; print(platform.machine())'
$ tox -e py27
py27 create: C:\Users\Anthony\AppData\Local\Temp\.tox\py27
py27 installed:
py27 runtests: PYTHONHASHSEED='154'
py27 runtests: commands[0] | python -c import platform; print(platform.machine())

______________________________________________________ summary _______________________________________________________
  py27: commands succeeded
  congratulations :)

$ TOX_TESTENV_PASSENV=PROCESSOR_ARCHITECTURE tox -e py27
py27 installed:
py27 runtests: PYTHONHASHSEED='864'
py27 runtests: commands[0] | python -c import platform; print(platform.machine())
AMD64
______________________________________________________ summary _______________________________________________________
  py27: commands succeeded
  congratulations :)

Hitting this in pre-commit/pre-commit#685 where I'm working around this by adding PROCESSOR_ARCHITECTURE to passenv

@asottile asottile added bug:minor does not affect many people or has no big impact os:windows level:easy rought estimate that this might be quite easy to implement labels Jan 12, 2018
@tox-dev tox-dev locked and limited conversation to collaborators Jan 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug:minor does not affect many people or has no big impact level:easy rought estimate that this might be quite easy to implement os:windows
Projects
None yet
Development

No branches or pull requests

1 participant