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

Revert "Keeping virtualenv<20 on macOS, until problems get fixed by new release" #274

Merged
merged 4 commits into from
Feb 28, 2020

Conversation

YannickJadoul
Copy link
Member

The issue (pypa/virtualenv#1561) was fixed by PR pypa/virtualenv#1641, and is now released in virtualenv 20.0.5. Let's undo the hack/workaround.

@Czaki
Copy link
Contributor

Czaki commented Feb 25, 2020

what with this?

@YannickJadoul
Copy link
Member Author

Still waiting for a fix of one issue.

The issue reported in pypa/virtualenv#1561 was fixed, and the problem in macos.py was solved.

The problem in .circleci/prepare.sh was judged to not be linked to virtualenv: pypa/virtualenv#1561. It's now reported to PyPy: https://foss.heptapod.net/pypy/pypy/issues/3175

@Czaki
Copy link
Contributor

Czaki commented Feb 27, 2020

virtualenv 20.0.7 is released. Maybe build restart?

@YannickJadoul
Copy link
Member Author

virtualenv 20.0.7 is released. Maybe build restart?

The release is unrelated. I had already tried out 20.0.6, tried out 20.0.7 just now, and it's no difference.

The issue is unrelated to virtualenv, btw:

The problem in .circleci/prepare.sh was judged to not be linked to virtualenv:

pypa/virtualenv#1654 (comment)

@YannickJadoul
Copy link
Member Author

@joerick Got this working. Good to merge?

cibuildwheel/macos.py Outdated Show resolved Hide resolved
Serves me right for editing code in the github editor!!
@YannickJadoul YannickJadoul merged commit 774af30 into pypa:master Feb 28, 2020
@YannickJadoul YannickJadoul deleted the undo-virtualenv-workaround branch February 28, 2020 17:51
@YannickJadoul
Copy link
Member Author

Thanks, @joerick!

@YannickJadoul YannickJadoul mentioned this pull request Feb 29, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants