Skip to content

python313Packages.nixpkgs-plugin-update: init from maintainer script #453943

Merged
wolfgangwalther merged 2 commits intoNixOS:masterfrom
emilazy:push-utrsmvvxozvk
Oct 22, 2025
Merged

python313Packages.nixpkgs-plugin-update: init from maintainer script #453943
wolfgangwalther merged 2 commits intoNixOS:masterfrom
emilazy:push-utrsmvvxozvk

Conversation

@emilazy
Copy link
Member

@emilazy emilazy commented Oct 20, 2025

We have machinery for Python packaging, so let’s make use of it. This also helps unblock moving by-name overrides out of all-packages.nix, as it was previously being used to work around nixpkgs-vet forbidding the direct reference to the path.

cc @teto @PerchunPak @khaneliman – I added you as maintainers as the people who committed to this file; let me know if you’re happy to be listed here or if you want removing.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

We have machinery for Python packaging, so let’s make use of
it. This also helps unblock moving `by-name` overrides out of
`all-packages.nix`, as it was previously being used to work around
`nixpkgs-vet` forbidding the direct reference to the path.
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 6.topic: python Python is a high-level, general-purpose programming language. 6.topic: vim Advanced text editor labels Oct 20, 2025
@khaneliman khaneliman mentioned this pull request Oct 20, 2025
13 tasks
Copy link
Contributor

@wolfgangwalther wolfgangwalther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I went through the diff and didn't spot anything obvious.

It seems this has been tested in #453954 already.

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Oct 21, 2025
Copy link
Member

@teto teto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change makes sense, I had started sthg similar in #344320. Fine by me

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Oct 21, 2025
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 3+ This PR was reviewed and approved by three or more persons. and removed 12.approvals: 2 This PR was reviewed and approved by two persons. labels Oct 22, 2025
@wolfgangwalther wolfgangwalther added this pull request to the merge queue Oct 22, 2025
Merged via the queue into NixOS:master with commit 9a3e70e Oct 22, 2025
35 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: python Python is a high-level, general-purpose programming language. 6.topic: vim Advanced text editor 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 12.approvals: 3+ This PR was reviewed and approved by three or more persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants