Skip to content

pkgsMusl.mailutils: fix build#436206

Merged
yuyuyureka merged 1 commit intoNixOS:stagingfrom
yuyuyureka:mailutils-musl
Sep 29, 2025
Merged

pkgsMusl.mailutils: fix build#436206
yuyuyureka merged 1 commit intoNixOS:stagingfrom
yuyuyureka:mailutils-musl

Conversation

@yuyuyureka
Copy link
Contributor

Depends on #436187, #436189

  • 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 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. 10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. labels Aug 23, 2025
@nix-owners nix-owners bot requested a review from lovek323 August 23, 2025 14:27
@@ -31,7 +31,7 @@ index 930861d8a..3b67844d2 100644
}

set POP3D_ETC_DIR "$MU_DATA_DIR/etc"
+set MU_PASSWD_HASH [exec mkpasswd --method=yescrypt guessme]
+set MU_PASSWD_HASH [exec mkpasswd -m yescrypt guessme]
Copy link
Member

Choose a reason for hiding this comment

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

It supports long options on glibc but not musl?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, --method fails with the pkgsMusl.mkpasswd

@alyssais
Copy link
Member

Needs a rebase on staging to pick up the merged GSS fix.

github-actions[bot]

This comment was marked as resolved.

@github-actions github-actions bot dismissed their stale review September 29, 2025 12:44

All good now, thank you!

@yuyuyureka yuyuyureka changed the base branch from master to staging-next September 29, 2025 12:47
@nixpkgs-ci nixpkgs-ci bot closed this Sep 29, 2025
@nixpkgs-ci nixpkgs-ci bot reopened this Sep 29, 2025
@yuyuyureka yuyuyureka changed the base branch from staging-next to staging September 29, 2025 13:24
@nixpkgs-ci nixpkgs-ci bot closed this Sep 29, 2025
@nixpkgs-ci nixpkgs-ci bot reopened this Sep 29, 2025
Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

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

LGTM

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Sep 29, 2025
@yuyuyureka
Copy link
Contributor Author

built pkgsMusl.mailutils on aarch64-linux

@yuyuyureka yuyuyureka merged commit abc7e7f into NixOS:staging Sep 29, 2025
28 of 32 checks passed
@yuyuyureka yuyuyureka deleted the mailutils-musl branch October 2, 2025 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 11-100 This PR causes between 11 and 100 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants