Skip to content

Fix DNF group advice#1111

Merged
MikeMcQuaid merged 1 commit into
mainfrom
fix/dnf-fedora-rhel-groups
Jun 21, 2026
Merged

Fix DNF group advice#1111
MikeMcQuaid merged 1 commit into
mainfrom
fix/dnf-fedora-rhel-groups

Conversation

@MikeMcQuaid

Copy link
Copy Markdown
Member
  • Fedora-derived and RHEL-derived systems may expose non-exact ID values in /etc/os-release.
  • Matching ID and ID_LIKE avoids recommending the legacy display group name where DNF expects development-tools.

Copilot AI review requested due to automatic review settings June 21, 2026 15:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Linux dependency installation guidance in install.sh so DNF-based Fedora/RHEL-family systems are detected more reliably via /etc/os-release (ID and ID_LIKE), avoiding the legacy “Development Tools” group name when dnf expects development-tools.

Changes:

  • Adjust DNF distro detection to match Fedora/RHEL-like systems using substring matches on ID and ID_LIKE.
  • Update the surrounding comment to reflect Fedora + RHEL-like behavior and keep the guidance aligned with DNF group IDs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Fedora-derived and RHEL-derived systems may expose non-exact `ID`
  values in `/etc/os-release`.
- Matching `ID` and `ID_LIKE` avoids recommending the legacy display
  group name where DNF expects `development-tools`.
@MikeMcQuaid MikeMcQuaid force-pushed the fix/dnf-fedora-rhel-groups branch from a5b5574 to 184a7bc Compare June 21, 2026 15:35
@MikeMcQuaid MikeMcQuaid enabled auto-merge June 21, 2026 15:37
@MikeMcQuaid MikeMcQuaid merged commit 5e78e69 into main Jun 21, 2026
8 checks passed
@MikeMcQuaid MikeMcQuaid deleted the fix/dnf-fedora-rhel-groups branch June 21, 2026 15:51
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.

3 participants