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

Build fails because of unrecognized argument passed to virtualenv #6725

Closed
Jackenmen opened this issue Feb 28, 2020 · 7 comments
Closed

Build fails because of unrecognized argument passed to virtualenv #6725

Jackenmen opened this issue Feb 28, 2020 · 7 comments
Labels
Needed: more information A reply from issue author is required

Comments

@Jackenmen
Copy link

Details

Expected Result

Build should be successful.

Actual Result

Build failed on creating virtualenv step:

[rtd-command-info] start-time: 2020-02-28T18:50:40.863059Z, end-time: 2020-02-28T18:50:41.986144Z, duration: 1, exit-code: 2
python3.8 -mvirtualenv --no-site-packages /home/docs/checkouts/readthedocs.org/user_builds/red-discordbot/envs/latest
usage: virtualenv [--version] [--with-traceback] [-v | -q] [--app-data APP_DATA] [--clear-app-data] [--discovery {builtin}] [-p py] [--creator {builtin,cpython3-posix,venv}] [--seeder {app-data,pip}] [--no-seed]
                  [--activators comma_sep_list] [--clear] [--system-site-packages] [--symlinks | --copies] [--download | --no-download] [--extra-search-dir d [d ...]] [--pip version] [--setuptools version] [--wheel version] [--no-pip]
                  [--no-setuptools] [--no-wheel] [--symlink-app-data] [--prompt prompt] [-h]
                  dest
virtualenv: error: unrecognized arguments: --no-site-packages
@gvanrossum
Copy link

I see this too for mypy (python/mypy#8459)

@Jackenmen
Copy link
Author

I forgot to post an update but on our project we only really got it at the build I linked and one other that was ran around same time, it seems like it fixed itself later.

@gvanrossum
Copy link

I still think this requires some fix in readthedocs. This code passes either --no-site-packages or --system-site-packages. But (as of now -- I don't know since when) virtualenv doesn't have a --no-site-packages flag, so readthedocs shouldn't be passing it.

As to why it sometimes works and sometimes doesn't, maybe it's a recent change to virtualenv and this code path is only taken for fresh VMs/containers, or something.

@gvanrossum
Copy link

Update: this disappeared (without specific mention in the ChangeLog) in virtualenv 20.0.0.

@gaborbernat
Copy link
Contributor

Hello, you can still find the legacy (v16) documentation here https://virtualenv.pypa.io/en/legacy/reference.html. v20 aimes to be cli compatible with v16 baring some flags that have been deprecated for years now. https://virtualenv.pypa.io/en/legacy/reference.html#cmdoption-no-site-packages is one of this, so we've dropped it during the migration to the new major version. We're sorry if this caused issues for you, you're right we should have mentioned it somewhere in our release changelog though.

@humitos
Copy link
Member

humitos commented Mar 1, 2020

Hi all! We did a release of our docker image that it's used to build the documentation, to fix another issue on Friday. Since we don't pin virtualenv on it, it got updated and we didn't realize that there was a breaking change. I think we rolled back this update, but I will double check this tomorrow with more time (from red-discordbot builds, it seems it's already rolled back because those builds are passing)

Please, let me know if you are still experimenting this issue.

@stsewd stsewd added the Needed: more information A reply from issue author is required label Mar 2, 2020
@ericholscher
Copy link
Member

We've also released a fix for this in #6738, which was deployed yesterday. Closing this now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: more information A reply from issue author is required
Projects
None yet
Development

No branches or pull requests

6 participants