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

Specify virtualenv as extra dependency for build in tox.ini #417

Merged
merged 1 commit into from Mar 10, 2021

Conversation

abravalheri
Copy link
Collaborator

@abravalheri abravalheri commented Mar 9, 2021

Purpose

build seems to rely by default in the stdlib's implementation of venv and unfortunately for a specific combination of Windows + conda env + virtualenv (as used by tox), that might be problematic as identified in pyscaffold/pyscaffoldext-markdown#10.

Approach

According to the discussion in pypa/build#253, specifying virtualenv as an extra dependency of build seems to be a good workaround. Moreover I would expect some kind speed improvements (virtualenv is reported to be faster than venv).

`build` seems to rely by default in the stdlib's implementation of `venv`
and unfortunately for a specific combination of Windows + conda env + virtualenv (as used by tox),
that might be problematic as identified in pyscaffold/pyscaffoldext-markdown#10.

According to the discussion in pypa/build#253,
specifying `virtualenv` as an extra dependency of `build` seems to be a
good workaround. Moreover I would expect some kind speed improvements
(virtualenv is reported to be faster than venv).
@abravalheri abravalheri marked this pull request as ready for review March 9, 2021 17:00
@abravalheri abravalheri marked this pull request as draft March 9, 2021 17:01
@abravalheri
Copy link
Collaborator Author

abravalheri commented Mar 9, 2021

This approach is now documented in build's docs

@abravalheri abravalheri marked this pull request as ready for review March 9, 2021 17:19
@abravalheri abravalheri merged commit 3f3e114 into master Mar 10, 2021
@abravalheri abravalheri deleted the fix-markdown-10 branch March 10, 2021 08:27
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.

None yet

1 participant