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

Uninstall doesn't delete the kernel #1

Open
aiguofer opened this issue Jul 21, 2019 · 4 comments · May be fixed by #7
Open

Uninstall doesn't delete the kernel #1

aiguofer opened this issue Jul 21, 2019 · 4 comments · May be fixed by #7

Comments

@aiguofer
Copy link
Owner

When I pyenv uninstall a version, the kernel file is not deleted. This is due to pyenv/pyenv-virtualenv#320. Once that's fixed, the hook that deletes the kernel file should work.

aiguofer pushed a commit that referenced this issue Oct 12, 2022
@ghost
Copy link

ghost commented Sep 4, 2023

Was this fixed?

@ghost
Copy link

ghost commented Sep 4, 2023

No it isn't fixed ... still deleting the environment doesn't delete the kernel

@raffaem
Copy link

raffaem commented Sep 7, 2023

It looks like it works with pyenv uninstall VENV, but not with pyenv virtualenv-delete VENV, as there are no hooks in pyenv-virtualenv-delete

@raffaem
Copy link

raffaem commented Sep 7, 2023

Also pyenv virtualenv-delete seems to be the correct command to call for removing a virtualenv, not pyenv uninstall.

While pyenv virtualenv-delete will remove both $(pyenv root)/versions/[VENV_NAME] and $(pyenv root)/versions/[PYTHON_VERSION]/envs/[VENV_NAME], pyenv uninstall will remove only the former, but not the latter.

Consider the following example:

> pyenv virtualenv 3.11.5 newenv
> ls $(pyenv root)/versions/3.11.5/envs/
newenv
> ls $(pyenv root)/versions/ | grep newenv
newenv
> pyenv uninstall newenv
> ls $(pyenv root)/versions/3.11.5/envs/
newenv
> ls $(pyenv root)/versions/ | grep newenv

So it's actually working with the wrong command.

@appleparan appleparan linked a pull request Jan 22, 2024 that will close this issue
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 a pull request may close this issue.

2 participants