search

hayd / pep8radius

PEP8 clean only the parts of the files touched since the last commit, a previous commit or (the merge-base of) a branch.
MIT License
183 stars 9 forks source link

Cannot uninstall pep8radius from pip --user install #77

Closed lofidevops closed 10 years ago

lofidevops commented 10 years ago

Steps to reproduce:

What should happen:

pep8radius is installed then uninstalled.

What happens instead:

pep8radius is installed. When attempting uninstallation:

Can't uninstall 'pep8radius'. No files were found to uninstall.

Notes:

Workaround:

Manually delete files listed in .local/lib/python2.7/site-packages/pep8radius-0.9.1-py2.7.egg-info/installed-files.txt

lofidevops commented 10 years ago

FYI I am installing this way because pep8radius is not in the Ubuntu repos (yet) so I can't do a system install. Additionally sudo pip install is discouraged (and has the same error on uninstall/upgrade).

hayd commented 10 years ago

Thanks for reporting this.

I'll have a play with my ubuntu machine this evening and see if I can reproduce (I can't on my mac but that's using conda which no doubt also has an effect) and +1 for encouraging user installs.

I wonder if this is something incorrect/missing in setup.py, would you mind testing this install/uninstall with autopep8? (Since my setup.py was originally a copy and paste from there!) I had a quick google and looked at youtube-dl's setup.py and MANIFEST.in, and nothing jumped out :(

Perhaps it's this pip bug for non-wheeled packages on debian?

https://github.com/pypa/pip/issues/1962 https://github.com/pypa/pip/issues/1895

The workaround is for pep8radius to publish a wheel, which is probably a good idea anyway.

hayd commented 10 years ago

@kwill I backported a wheel for v0.9.1 to pypi, please have try again and see if that fixes this, thanks! https://pypi.python.org/pypi/pep8radius https://github.com/hayd/pep8radius/commit/c3b76efcac5e26df35a7e687a6a9597bddf3fb23

lofidevops commented 10 years ago

I will let you know as soon as I have a chance to test (unless someone else gets there first :)

lofidevops commented 10 years ago

It works! (Dependencies remain, but I presume that's a pip thing.) Many thanks.

hayd commented 10 years ago

Great! Thanks again for filing the bug report.

ps. FWIW this means it's a debian bug, something about pip & virtualenv, see bug report in the above issue https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=751827.