Skip to content

[Backport release-25.05] buildPython*: unconditionally add .override #455389

Merged
MattSturgeon merged 1 commit intorelease-25.05from
backport-455362-to-release-25.05
Oct 24, 2025
Merged

[Backport release-25.05] buildPython*: unconditionally add .override #455389
MattSturgeon merged 1 commit intorelease-25.05from
backport-455362-to-release-25.05

Conversation

@nixpkgs-ci
Copy link
Contributor

@nixpkgs-ci nixpkgs-ci bot commented Oct 24, 2025

Bot-based backport to release-25.05, triggered by a label in #455362.

  • Before merging, ensure that this backport is acceptable for the release.
    • Even as a non-committer, if you find that it is not acceptable, leave a comment.

To be able to splice, we can't fail to eval before returning an
attribute set. By checking for f ? override, we need to force f which
isn't always possible, whereas mirrorFunctionArgs serves as an
indirection which wraps the inner thunks in an attribute set that can
always be returned. Fortunately, there should be no case when f
evaluates successfully and does not have the override attribute, so we
can just remove the condition.

This fixes evaluation of buildPython* based packages in certain splicing
situations by being lazier. An example of this is
pkgsCross.ghcjs.buildPackages.nixpkgs-openjdk-updater on staging right
now (ff5098e).

(cherry picked from commit 0443b98)
@MattSturgeon MattSturgeon added this pull request to the merge queue Oct 24, 2025
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 6.topic: python Python is a high-level, general-purpose programming language. 4.workflow: backport This targets a stable branch labels Oct 24, 2025
@nix-owners nix-owners bot requested review from mweinelt and natsukium October 24, 2025 22:50
Merged via the queue into release-25.05 with commit 810ffa7 Oct 24, 2025
29 of 30 checks passed
@MattSturgeon MattSturgeon deleted the backport-455362-to-release-25.05 branch October 24, 2025 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.workflow: backport This targets a stable branch 6.topic: python Python is a high-level, general-purpose programming language. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants