Skip to content

python3: let lib.makeScopeWithSplicing' do its job#482863

Closed
doronbehar wants to merge 1 commit intoNixOS:masterfrom
doronbehar:pkg/setuptools-rust@properSolution
Closed

python3: let lib.makeScopeWithSplicing' do its job#482863
doronbehar wants to merge 1 commit intoNixOS:masterfrom
doronbehar:pkg/setuptools-rust@properSolution

Conversation

@doronbehar
Copy link
Contributor

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.

Fixes NixOS#476822 and NixOS#480467 . If merged, NixOS#480005 should be reverted (on
branch `staging`).

Co-Authored-By: Colin <colin@uninsane.org>
@nixpkgs-ci nixpkgs-ci bot requested review from mweinelt and natsukium January 23, 2026 00:27
@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. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 6.topic: python Python is a high-level, general-purpose programming language. labels Jan 23, 2026
@uninsane uninsane changed the title python3: let lib.mkScopeWithSplicing do its job python3: let lib.makeScopeWithSplicing' do its job Jan 23, 2026
@doronbehar doronbehar requested a review from alois31 January 24, 2026 18:10
Copy link
Contributor

@alois31 alois31 left a comment

Choose a reason for hiding this comment

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

I can confirm it fixes the issue. However I am not familiar with the Python splicing to judge what the effects of removing the inputs' overriding entirely are, so someone else should look at that.

@doronbehar
Copy link
Contributor Author

Since this is not urgent, and it is easier to see the justification of this in conjunction with #482866 , which should be merged a bit urgently (to the on-going staging-next cycle), I'm closing this PR and will cherry-pick the commit here to #482866 .

@doronbehar doronbehar closed this Feb 9, 2026
@doronbehar doronbehar deleted the pkg/setuptools-rust@properSolution branch February 10, 2026 08:09
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. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants