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

Normalize python-modules attribute names and pnames #245383

Closed
mweinelt opened this issue Jul 25, 2023 · 10 comments
Closed

Normalize python-modules attribute names and pnames #245383

mweinelt opened this issue Jul 25, 2023 · 10 comments
Labels
0.kind: enhancement 3.skill: sprintable 5. scope: tracking Long-lived issue tracking long-term fixes or multiple sub-problems 6.topic: python
Milestone

Comments

@mweinelt
Copy link
Member

mweinelt commented Jul 25, 2023

Python attribute and pnames should be normalized according to https://packaging.python.org/en/latest/specifications/name-normalization/#normalization.

The following list of names, that need work, was provided by @adisbladis in the nixpkgs python room.

Cc'ing people who are currently doing the work in the python ecosystem.
@dotlambda @fabaff @SuperSandro2000 @tjni @natsukium

The workflow would be to

  1. Normalize pname in the derivation
  2. Normalize the directory name
  3. Normalize attribute name and path in pkgs/top-level/python-packages.nix
  4. Update all references to the old attribute name treewide
  5. Create an alias in pkgs/top-level/python-aliases.nix for the old attribute name
  6. Commit python311Packages.newname: rename from oldname

@tjni tjni mentioned this issue Jul 25, 2023
12 tasks
@mweinelt mweinelt added this to To do in Picking up garbage via automation Jul 26, 2023
@mweinelt mweinelt removed this from To do in Picking up garbage Jul 26, 2023
@natsukium natsukium mentioned this issue Sep 2, 2023
12 tasks
AndersonTorres added a commit to atorres1985-contrib/nixpkgs that referenced this issue Sep 2, 2023
@Stunkymonkey
Copy link
Contributor

Stunkymonkey commented Mar 26, 2024

@dotlambda can you please verify again. I think we can close this issue. (staging not yet merged)

@wegank wegank mentioned this issue Apr 11, 2024
13 tasks
@Stunkymonkey
Copy link
Contributor

all the lists in this thread are now done. staging is merged into master.

Therefore I consider this done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: enhancement 3.skill: sprintable 5. scope: tracking Long-lived issue tracking long-term fixes or multiple sub-problems 6.topic: python
Projects
None yet
Development

No branches or pull requests

8 participants