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 the canonicalize_name function for finding .dist-info #3794

Merged
merged 1 commit into from Oct 30, 2016

Conversation

dstufft
Copy link
Member

@dstufft dstufft commented Jun 12, 2016

Fixes #3793

@xavfernandez
Copy link
Member

Maybe we should directly add a canonical_name property to InstallRequirement ?

@yarko
Copy link

yarko commented Jun 14, 2016

this fixes installation; but now the problem is uninstall:

foo.bar-version installs (and *.dist-info), but during uninstall,
req/req_install.py:InstallRequirement.check_if_exists() tries to find:

self.req, where self.req.name == foo_bar,

and pkg_resources.get_distribution() will find foo.bar, but not foo_bar (nor foo-bar);

Probably need to look at _vendor/pkg_resources/init.py: WorkingSet.resolve();
Wasn't immediately clear to me what what / where would fix this...

@yarko
Copy link

yarko commented Jun 14, 2016

Update:

pip list will produce "foo.bar" as an installed package;
pip uninstall foo.bar seems to work ok, so maybe this is not a problem
even though its a bit confusing, because at code level you're doing things like:

from foo_bar import something

@pradyunsg
Copy link
Member

(to notify people) +1 to @xavfernandez's idea.

@xavfernandez
Copy link
Member

@dstufft Maybe add a changelog for the #3793 issue ?

@xavfernandez xavfernandez added this to the 8.2 milestone Jul 5, 2016
@xavfernandez xavfernandez added the type: bug A confirmed bug or unintended behavior label Jul 5, 2016
@xavfernandez xavfernandez reopened this Oct 14, 2016
@dstufft dstufft merged commit 2743edc into pypa:master Oct 30, 2016
@dstufft dstufft deleted the fix-normalized-wheel branch October 30, 2016 15:42
@lock lock bot added the auto-locked Outdated issues that have been locked by automation label Jun 3, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Jun 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto-locked Outdated issues that have been locked by automation type: bug A confirmed bug or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Installing package with period in name (normalized to a hyphen) fails on 8.1.2
4 participants