Skip to content

Clarify dots canonicalisation change is breaking#7607

Closed
PeterJCLaw wants to merge 2 commits intopython-poetry:masterfrom
PeterJCLaw:clarify-canonicalisation-is-breaking
Closed

Clarify dots canonicalisation change is breaking#7607
PeterJCLaw wants to merge 2 commits intopython-poetry:masterfrom
PeterJCLaw:clarify-canonicalisation-is-breaking

Conversation

@PeterJCLaw
Copy link
Copy Markdown

@PeterJCLaw PeterJCLaw commented Mar 5, 2023

The handling of package names within the Python packaging ecosystem is unfortunately not in a great state at the moment. There exist a large number of packages which contain characters which (some of) the specs indicate are invalid, yet there is not a clear migration path nor destination at the moment.

Poetry's approach is to normalise project names towards their "canonical" form, meaning that existing projects on PyPI which use the more relaxed name forms will be silently renamed if they move to (recent versions of) Poetry. As well as changing the name of these packages (which maintainers are unlikely to expect nor desire) this unfortunately ends up breaking introspection via importlib.metadata in Python 3.8 and 3.9 which are not aware of the canonicalisation rules which Poetry is using.

Adding this compatibility note informs maintainers so that they can decide how (and if) they want this name canonicalisation to happen.

See also #6198, python-poetry/poetry-core#560.

The handling of package names within the Python packaging ecosystem
is unfortunately not in a great state at the moment. There exist
a large number of packages which contain characters which (some of)
the specs indicate are invalid, yet there is not a clear migration
path nor destination at the moment.

Poetry's approach is to normalise project names towards their
"canonical" form, meaning that existing projects on PyPI which
use the more relaxed name forms will be silently renamed if they
move to (recent versions of) Poetry. As well as changing the name
of these packages (which maintainers are unlikely to expect nor
desire) this unfortunately ends up breaking introspection via
`importlib.metadata` in Python 3.8 and 3.9 which are not aware of
the canonicalisation rules which Poetry is using.

Adding this compatibility note informs maintainers so that they can
decide how (and if) they want this name canonicalisation to happen.

See also python-poetry#6198
@PeterJCLaw PeterJCLaw force-pushed the clarify-canonicalisation-is-breaking branch from bbc3459 to a047f08 Compare March 5, 2023 14:15
@dimbleby
Copy link
Copy Markdown
Contributor

dimbleby commented Aug 2, 2023

per python-poetry/poetry-core#560 (comment): pypi/warehouse#10030 was closed - fixed apparently - a couple of weeks ago, which should make this unwanted.

@radoering
Copy link
Copy Markdown
Member

I'm closing it since 1.2 was released quite a while ago and I don't think it's worth to still update the changelog. Especially, if we can fix / change back now.

(Accidentally clicked on the update button.)

@radoering radoering closed this Aug 3, 2023
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 3, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants