Skip to content

openroad: fix build with gcc15#478024

Merged
GaetanLepage merged 1 commit intoNixOS:masterfrom
eljamm:fix/openroad-gcc15
Jan 9, 2026
Merged

openroad: fix build with gcc15#478024
GaetanLepage merged 1 commit intoNixOS:masterfrom
eljamm:fix/openroad-gcc15

Conversation

@eljamm
Copy link
Contributor

@eljamm eljamm commented Jan 8, 2026

Tracking #475479

Note that this fix can be removed on updates that come after The-OpenROAD-Project/OpenROAD@bbbcf49.

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.

@nixpkgs-ci nixpkgs-ci bot requested review from hzeller and trepetti January 8, 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. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Jan 8, 2026
@gonsolo
Copy link
Contributor

gonsolo commented Jan 8, 2026

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 478024
Commit: 0a17916fff3cacfc9e05592684f309bbd81b4a46


x86_64-linux

✅ 1 package built:
  • openroad

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Jan 8, 2026
@gonsolo gonsolo mentioned this pull request Jan 9, 2026
@gonsolo gonsolo requested a review from imincik January 9, 2026 14:44
@gonsolo
Copy link
Contributor

gonsolo commented Jan 9, 2026

Maybe @GaetanLepage wants to review?

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/prs-already-reviewed/2617/2728

@GaetanLepage
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 478024
Commit: 0a17916fff3cacfc9e05592684f309bbd81b4a46


x86_64-linux

✅ 1 package built:
  • openroad

Copy link
Contributor

@GaetanLepage GaetanLepage left a comment

Choose a reason for hiding this comment

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

Thanks!

@GaetanLepage GaetanLepage added this pull request to the merge queue Jan 9, 2026
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. 2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Jan 9, 2026
Merged via the queue into NixOS:master with commit 03d91cf Jan 9, 2026
39 of 41 checks passed
@eljamm eljamm deleted the fix/openroad-gcc15 branch January 10, 2026 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: merge-bot eligible This PR can be merged by commenting "@NixOS/nixpkgs-merge-bot merge". 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. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants