Skip to content

Fixes#4

Merged
maliberty merged 8 commits intoThe-OpenROAD-Project:masterfrom
gonsolo:fixes
Jan 7, 2026
Merged

Fixes#4
maliberty merged 8 commits intoThe-OpenROAD-Project:masterfrom
gonsolo:fixes

Conversation

@gonsolo
Copy link
Contributor

@gonsolo gonsolo commented Jan 7, 2026

Fix one cmake error, one GCC 15 error, and all warnings.
This causes lemon-graph and openroad in Nix to fail.
@maliberty

@maliberty maliberty merged commit e9e3bc6 into The-OpenROAD-Project:master Jan 7, 2026
@maliberty
Copy link
Member

@gonsolo
Copy link
Contributor Author

gonsolo commented Jan 7, 2026

Note that we check out by tag in https://github.com/The-OpenROAD-Project/OpenROAD/blob/master/etc/DependencyInstaller.sh so you'll need to make a change there as well.

Are you sure? I just see a LEMON_VERSION.
I try to make nix work again: NixOS/nixpkgs#477789

@maliberty
Copy link
Member

LEMON_VERSION is a tag that we check out.

@gonsolo
Copy link
Contributor Author

gonsolo commented Jan 7, 2026

LEMON_VERSION is a tag that we check out.

Could you tag it 1.3.2?

@maliberty
Copy link
Member

I made 1.3.1.1

@gonsolo
Copy link
Contributor Author

gonsolo commented Jan 8, 2026

I made 1.3.1.1

Thanks. The Nix maintainers are hesitant to switch to this fork, so I emailed the original author at lemon.cs.elte.hu.
Hopefully we can resolve this together.

@maliberty
Copy link
Member

I'm not a nix user but I'm glad to help however I can.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants