Skip to content

gcc: move version iteration out of all-packages.nix#265934

Merged
3 commits merged intomasterfrom
unknown repository
Nov 7, 2023
Merged

gcc: move version iteration out of all-packages.nix#265934
3 commits merged intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Nov 6, 2023

Description of changes

This PR causes zero rebuilds and does not affect eval.

It simply moves a bunch of gcc-specific logic out of all-packages.nix and into pkgs/compilers/gcc.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • 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/)
  • 23.11 Release Notes (or backporting 23.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.

This resolves a TODO previously in all-packages.nix.

Adam Joseph added 2 commits November 6, 2023 14:00
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Nov 6, 2023
@ghost ghost marked this pull request as ready for review November 6, 2023 23:56
Copy link
Copy Markdown
Contributor

@Mindavi Mindavi left a comment

Choose a reason for hiding this comment

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

I like this conceptually. Less things in all-packages is good

@delroth delroth added the 12.approvals: 1 This PR was reviewed and approved by one person. label Nov 7, 2023
@ghost
Copy link
Copy Markdown
Author

ghost commented Nov 7, 2023

Normally I would not merge changes to gcc at this point in the release cycle, but this one has no effect on eval.

It will also make it easier to backport any changes that happen after branch-off if necessary.

@ghost ghost merged commit 855695d into NixOS:master Nov 7, 2023
@ghost ghost deleted the pr/gcc/cleanup branch November 7, 2023 09:57
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any 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.

2 participants