Skip to content

Conversation

@eljamm
Copy link
Contributor

@eljamm eljamm commented Jan 9, 2026

The ISO C23, which is the default in GCC 15, changes the meaning of an empty argument list from an unspecified list to no arguments, thus breaking the build.

This patch fixes that by specifying all the arguments.

Note: this fix was possible thanks to a similar patch: https://github.com/NixOS/nixpkgs/blob/ed124dcd6e5e07395c7d7ec77cd54a962f59bedc/pkgs/by-name/sh/sharutils/gcc15-c23-port.patch

Tracking: #475479

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.

The ISO C23, which is the default in GCC 15, changes the meaning of an
empty argument list from an unspecified list to no arguments, thus
breaking the build.

This patch fixes that by specifying all the arguments.

Note: this fix was possible thanks to a similar patch:
https://github.com/NixOS/nixpkgs/blob/ed124dcd6e5e07395c7d7ec77cd54a962f59bedc/pkgs/by-name/sh/sharutils/gcc15-c23-port.patch
@nixpkgs-ci nixpkgs-ci bot requested a review from wegank January 9, 2026 10:30
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. labels Jan 9, 2026
url = "https://github.com/msys2/MINGW-packages/raw/c6691ad1bd9d4c6823a18068ca0683c3e32ea005/mingw-w64-termcap/0001-tparam-replace-write-with-fprintf.patch";
hash = "sha256-R9XaLfa8fzQBt+M+uA1AFTvKYCeOWLUD/7GViazXwto=";
})
./0001-Fix-too-many-arguments-to-function-with-GCC15-C23.patch
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Now that I think about it, perhaps a better solution would be to change the std, similar to what fedora does?

@eljamm
Copy link
Contributor Author

eljamm commented Jan 9, 2026

nixpkgs-review result for #478354

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 478354
Commit: 4e2076388867c90d337bf4f8842662fb996daff5 (subsequent changes)
Merge: 5da72a7e5979b1968629ab4bd8b779e080159e81

Logs: https://github.com/eljamm/nixpkgs-review-gha/actions/runs/20849008610


x86_64-linux

✅ 4 packages built:
  • gnucap
  • gnucap-full
  • termcap
  • termcap.dev

aarch64-linux

✅ 4 packages built:
  • gnucap
  • gnucap-full
  • termcap
  • termcap.dev

x86_64-darwin (sandbox = relaxed)

✅ 2 packages built:
  • termcap
  • termcap.dev

aarch64-darwin (sandbox = relaxed)

✅ 2 packages built:
  • termcap
  • termcap.dev

@eljamm eljamm requested a review from imincik January 9, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant