Skip to content

[staging] pkgsMusl.which: fix build#477577

Merged
wolfgangwalther merged 1 commit intoNixOS:stagingfrom
yuyuyureka:musl-which
Jan 10, 2026
Merged

[staging] pkgsMusl.which: fix build#477577
wolfgangwalther merged 1 commit intoNixOS:stagingfrom
yuyuyureka:musl-which

Conversation

@yuyuyureka
Copy link
Contributor

@yuyuyureka yuyuyureka commented Jan 6, 2026

Fixes: #478542

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.

@nixpkgs-ci nixpkgs-ci bot added the 2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". label Jan 6, 2026
@nixpkgs-ci nixpkgs-ci bot requested a review from mdaniels5757 January 6, 2026 22:45
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-linux-stdenv This PR causes stdenv to rebuild on Linux and must target a staging branch. 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. labels Jan 6, 2026
Copy link
Member

@mdaniels5757 mdaniels5757 left a comment

Choose a reason for hiding this comment

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

Upstream declined to apply this patch for a specific reason, see https://lists.gnu.org/archive/html/which-bugs/2025-04/msg00003.html.

This needs to be gated behind lib.optionals stdenv.hostPlatform.isMusl.

@yuyuyureka yuyuyureka closed this Jan 7, 2026
@yuyuyureka yuyuyureka mentioned this pull request Jan 7, 2026
13 tasks
@mdaniels5757 mdaniels5757 reopened this Jan 9, 2026
@mdaniels5757 mdaniels5757 dismissed their stale review January 9, 2026 21:54

For the reason I gave in #473403 (comment), this is actually likely fine

Copy link
Member

@mdaniels5757 mdaniels5757 left a comment

Choose a reason for hiding this comment

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

I think this is likely fine, but am not quite confident enough to merge it myself. Built {pkgsMusl,}which on aarch64-linux.

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Jan 9, 2026
@mdaniels5757 mdaniels5757 mentioned this pull request Jan 10, 2026
3 tasks
@wolfgangwalther wolfgangwalther self-requested a review January 10, 2026 11:32
@wolfgangwalther
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 477577 --package pkgsMusl.which --package which
Commit: 45b1bbef701502130d1e97ecd681dd0909365bfe


x86_64-linux

✅ 2 packages built:
  • pkgsMusl.which
  • which

aarch64-linux

✅ 2 packages built:
  • pkgsMusl.which
  • which

still double-checking the build for aarch64-darwin.

@wolfgangwalther
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 477577 --package which
Commit: 45b1bbef701502130d1e97ecd681dd0909365bfe


aarch64-darwin

✅ 1 package built:
  • which

@wolfgangwalther wolfgangwalther added this pull request to the merge queue Jan 10, 2026
Merged via the queue into NixOS:staging with commit 77f5ebd Jan 10, 2026
65 of 67 checks passed
@yuyuyureka yuyuyureka deleted the musl-which branch January 11, 2026 16:11
@yuyuyureka
Copy link
Contributor Author

After more research, I found the comment originates from an old version of getopt.h from gnulib. The comment was removed in gnulib with commit coreutils/gnulib@1c64ecc#diff-acd0cc21a35657b5071913f8865087c0c0598e701419d32036c7dcfbf22e29bdL148 from 2004...

@uninsane uninsane added the 6.topic: musl Running or building packages with musl libc label Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". 6.topic: musl Running or building packages with musl libc 10.rebuild-darwin: 501+ This PR causes many rebuilds on Darwin and should normally target the staging branches. 10.rebuild-darwin: 5001+ This PR causes many rebuilds on Darwin and must target the staging branches. 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 5001+ This PR causes many rebuilds on Linux and must target the staging branches. 10.rebuild-linux-stdenv This PR causes stdenv to rebuild on Linux and must target a staging branch. 12.approvals: 1 This PR was reviewed and approved by one person. 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.

4 participants