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

Restore unintentionally-modified pre-2186 cleanup behaviors #2199

Closed

Conversation

arcivanov
Copy link
Contributor

  1. Do not disturb platdir
  2. Do delete a target directory on install if it is present and
    wasn't a part of the distro that could've been uninstalled

fixes #2198

Thanks for contributing, make sure you address all the checklists (for details on how see

development documentation)!

  • ran the linter to address style issues (tox -e fix_lint)
  • wrote descriptive pull request text
  • ensured there are test(s) validating the fix
  • added news fragment in docs/changelog folder
  • updated/extended the documentation

1. Do not disturb platdir
2. Do delete a target directory on install if it is present and
wasn't a part of the distro that could've been uninstalled

fixes pypa#2198
Copy link
Contributor

@gaborbernat gaborbernat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, improve the wording of the changelog/PR to describe the change, and not speak about unintentional changes. They were very much intentional. Needs test too.

Comment on lines +154 to +155
# Only uninstall purelib as it's the only one that VirtualEnv installs into
in_folders = chain.from_iterable([i.iterdir() for i in {self._creator.purelib}])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this.

@arcivanov arcivanov closed this Sep 25, 2021
@arcivanov arcivanov deleted the fix_uninstall_regression_behavior branch September 25, 2021 23:49
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.

Support overriding corrupt virtual environments
2 participants