Skip to content

[Build] Fixed Issue in Rhel9 Build #3183

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

AlexandreSinger
Copy link
Contributor

The rhel9 build of VTR is unable to resolve the "onetbb" prefix in the include path for tbb. This is easily resolved by just removing this prefix.

Other tbb includes do not use this prefix. This is only found in this one place.

The rhel9 build of VTR is unable to resolve the "onetbb" prefix in the
include path for tbb. This is easily resolved by just removing this
prefix.
@github-actions github-actions bot added VPR VPR FPGA Placement & Routing Tool lang-cpp C/C++ code labels Jul 3, 2025
@AmirhosseinPoolad
Copy link
Contributor

Oh hey my quest to use Fedora might actually prove useful to someone! Kind of surprising that it didn't work on RHEL9 but also I've only used Fedora 40 and 41 and RHEL 9 is AFAIK based on Fedora 34.

LGTM.

@AlexandreSinger
Copy link
Contributor Author

Oh hey my quest to use Fedora might actually prove useful to someone! Kind of surprising that it didn't work on RHEL9 but also I've only used Fedora 40 and 41 and RHEL 9 is AFAIK based on Fedora 34.

LGTM.

Well, you certainly made some random users of Silicon Compiler happy lol: siliconcompiler/siliconcompiler#3764 (comment)

SC maintains support for Ubuntu 20, 22, and 24 as well as Rhel.8 and 9. We dropped the support for Ubuntu 20 and Rhel 8; but with this we can keep the support for Rhel 9 unless I find any more issues.

I was suprised that this broke since you use Fedora and do not hit this issue and all the Ubuntu builds do not have an issue either. But whatever, easy fix.

@AlexandreSinger AlexandreSinger merged commit 4bb340a into verilog-to-routing:master Jul 3, 2025
33 checks passed
@AlexandreSinger AlexandreSinger deleted the feature-fix-tbb-include branch July 3, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang-cpp C/C++ code VPR VPR FPGA Placement & Routing Tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants