Adding explicit install configuration for EL8 and EL9 systems#3408
Merged
vvbandeira merged 4 commits intoAug 28, 2025
Merged
Conversation
Contributor
Author
|
One small thing which I'm not sure about, is if the cleanup functions need the # Enterprise Linux 8/9 cleanup
_install_EL8_EL9_CleanUp() {
dnf clean -y all
rm -rf /var/lib/apt/lists/*
}I included it here since |
rovinski
reviewed
Aug 22, 2025
vvbandeira
requested changes
Aug 22, 2025
vvbandeira
left a comment
Member
There was a problem hiding this comment.
Please address comments above; otherwise, it looks good.
BTW, thanks for both PRs!
This was
linked to
issues
Aug 22, 2025
Member
|
@kcaisley there's a merge conflict |
- Add AlmaLinux, Rocky Linux, and RHEL support to DependencyInstaller.sh - Update OpenROAD submodule with matching AlmaLinux support and sudo permission fixes - Resolves file ownership issues when running setup scripts with sudo Signed-off-by: kcaisley <kcaisley@uni-bonn.de>
- Enterprise Linux distribution detection to handle all variants (CentOS, RHEL, AlmaLinux, Rocky Linux) - Add version-based dispatch to use appropriate package managers (yum for EL7, dnf for EL8/EL9) - Create separate functions for EL7 vs EL8/EL9 with correct KLayout package URLs - For Klayout, use CentOS_7 package for EL7, CentOS_8 package for EL8, RockyLinux_9 package for EL9 - Fix argument expansion issue that broke dependency installer calls Signed-off-by: kcaisley <kcaisley@uni-bonn.de>
…ntation Signed-off-by: kcaisley <kcaisley@uni-bonn.de>
* master: Update metrics again for small diffs rebased private designs after new opt transforms are enabled FlowTutorial.md Typo flow: update sky130hd/uW mpl params Update metrics recent changes Signed-off-by: Vitor Bandeira <vvbandeira@precisioninno.com>
vvbandeira
approved these changes
Aug 28, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request is meant to be done in partnership with The-OpenROAD-Project/OpenROAD#7997.
My goal here is to enable building on the most commonly used Enterprise Linux 8/9 systems, by explicitly naming AlmaLinux as a valid target. The
NAMEfield of/etc/os-releaseacross EL7, EL8, and EL9 system can take any of the following values:So I first match against these, and then run one of the cases:
I fetch the correct
.rpmversion ofklayoutwithin the EL8/9 functions, and usednfoveryumfor clarity.