Skip to content

icu: enable updateAutotoolsGnuConfigScriptsHook#317218

Merged
7c6f434c merged 1 commit intoNixOS:stagingfrom
rhelmot:freebsd-icu
Jun 5, 2024
Merged

icu: enable updateAutotoolsGnuConfigScriptsHook#317218
7c6f434c merged 1 commit intoNixOS:stagingfrom
rhelmot:freebsd-icu

Conversation

@rhelmot
Copy link
Contributor

@rhelmot rhelmot commented Jun 4, 2024

Description of changes

Continuation of #313496

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
    • x86_64-freebsd
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@ofborg ofborg bot added the 10.rebuild-darwin-stdenv This PR causes stdenv to rebuild on Darwin and must target a staging branch. label Jun 4, 2024
@ofborg ofborg bot requested a review from 7c6f434c June 4, 2024 16:17
@ofborg ofborg bot added 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-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. labels Jun 4, 2024
@rhelmot
Copy link
Contributor Author

rhelmot commented Jun 4, 2024

Oh dear, this is a mass rebuild. I'm going to try to rebase it as per CONTRIBUTING, let's cross our fingers...

@rhelmot rhelmot changed the base branch from master to staging June 4, 2024 18:33
@7c6f434c
Copy link
Member

7c6f434c commented Jun 4, 2024

Does this mean that it is not enough that the upstream regenerates the configure script?

@ofborg ofborg bot requested a review from 7c6f434c June 4, 2024 19:59
@rhelmot
Copy link
Contributor Author

rhelmot commented Jun 4, 2024

If upstream regenerates the configure script with a gnu-config that postdates https://git.savannah.gnu.org/cgit/config.git/commit/?id=e4786449e1c26716e3f9ea182caf472e4dbc96e0 then it will be fine. Until then we get errors from /usr/bin/uname not found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments