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

Use plain python venv instead of virtualenv #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

igordertigor
Copy link

The package was using virtualenv as an unlisted dependency. I'm switching here to using python -m venv instead, which appears to be part of the standard library. Without this, try is broken for me.

@timofurrer
Copy link
Owner

Let me give this a shot later today before I merge it :) But looks great!

@Bobronium
Copy link

Bobronium commented May 23, 2022

@timofurrer, any updates on this?

Even if virtualenv is present, pip it for some reason creates venv with outdated pip which produces warnings.

Workaround:

  1. install pipx via pipx-in-pipx
    pip install pipx-in-pipx
  2. Install trypackage and virtualenv in isolated venvs:
    pipx install trypackage
    pipx install virtualenv
  3. Bonus: upgrade bundled libs for virtualenv (gets rid of old pip warnings)
    virtualenv  --upgrade-embed-wheels

trypackage/core.py Outdated Show resolved Hide resolved
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

3 participants