Skip to content

gss: properly autoreconf#444996

Merged
alyssais merged 1 commit intoNixOS:stagingfrom
alyssais:gss-autoreconf
Sep 21, 2025
Merged

gss: properly autoreconf#444996
alyssais merged 1 commit intoNixOS:stagingfrom
alyssais:gss-autoreconf

Conversation

@alyssais
Copy link
Member

Should let us drop the Darwin hack, and not have to apply it on other platfomrs, like musl, too.

I used Nixpkgs' gnulib source because it works, but if it stops working in future we could use fetchgit to get the gnulib submodule.

Closes: #436189

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.

Should let us drop the Darwin hack, and not have to apply it on other
platfomrs, like musl, too.

I used Nixpkgs' gnulib source because it works, but if it stops
working in future we could use fetchgit to get the gnulib submodule.

Fixes: 90805a4 ("gss: use autoreconfHook")
Closes: NixOS#436189
@alyssais alyssais mentioned this pull request Sep 21, 2025
13 tasks
@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 Sep 21, 2025
Copy link
Contributor

@yuyuyureka yuyuyureka left a comment

Choose a reason for hiding this comment

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

built on aarch64-darwin

LGTM

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 2 This PR was reviewed and approved by two persons. label Sep 21, 2025
@alyssais alyssais merged commit de42e6e into NixOS:staging Sep 21, 2025
30 of 34 checks passed
@alyssais alyssais deleted the gss-autoreconf branch September 21, 2025 17:01
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: 2 This PR was reviewed and approved by two persons.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants