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

tox doesn't work with pyrun #227

Closed
pytoxbot opened this issue Sep 17, 2016 · 3 comments
Closed

tox doesn't work with pyrun #227

pytoxbot opened this issue Sep 17, 2016 · 3 comments

Comments

@pytoxbot
Copy link

@pytoxbot pytoxbot commented Sep 17, 2016

When running tox, it sets up a virtualenv using the command line:

-mvirtualenv --setuptools --python py27

This works with CPython, but is not really in line with the documented use of the -m option:

-m mod : run library module as a script (terminates option list)

pyrun uses a different command line parser, since it has to emulate the CPython in Python and does not know how to handle options which use arguments without separating space.

Could this be changed to use "-m virtualenv" instead, to be in line with the Python documentation ?

@pytoxbot
Copy link
Author

@pytoxbot pytoxbot commented Sep 17, 2016

Original comment by @malemburg

Thanks, Holger. It works now :-)

Aside: I ran into this while trying to check a user error report for installing egenix-mx-base using tox. This gives intermittent installation errors, which I've not seen anywhere else. I'm still investigating this, since I'm not sure whether it's caused by the tox virtualenv setup or our web installer on PyPI.

@pytoxbot
Copy link
Author

@pytoxbot pytoxbot commented Sep 17, 2016

Original comment by @hpk42

you can try it with "pip install --pre -i https://devpi.net/hpk/dev tox" to get the dev version which incorporates the change.

@pytoxbot
Copy link
Author

@pytoxbot pytoxbot commented Sep 17, 2016

Original comment by @hpk42

fix issue227: use "-m virtualenv" instead of "-m virtualenv" to make
it work with pyrun.

→ <<cset 338b48333f7c>>

@pytoxbot pytoxbot closed this Sep 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.