Skip to content

dkms: Use a generic missing kernel headers error message#594

Merged
scaronni merged 1 commit into
dkms-project:mainfrom
HadiHbus:fix-rpm-kernel-devel-message
Jul 13, 2026
Merged

dkms: Use a generic missing kernel headers error message#594
scaronni merged 1 commit into
dkms-project:mainfrom
HadiHbus:fix-rpm-kernel-devel-message

Conversation

@HadiHbus

Copy link
Copy Markdown

The missing kernel headers error only suggested the Debian-style linux-headers package.
Add the rpm-based kernel-devel package to make the DKMS guidance useful across packaging ecosystems, and update the expected test output accordingly.

@scaronni scaronni self-requested a review July 13, 2026 09:39
@scaronni scaronni self-assigned this Jul 13, 2026
@scaronni

scaronni commented Jul 13, 2026

Copy link
Copy Markdown
Member

True, but I would remove the mention completely about the speciific debian package and just use a generic sentence:

  • kernel-devel-${KERNEL_VER}-noheaders does not exist in most distributions (RHEL/Fedora), it's usually kernel-devel.
  • SUSE has completely different package names
  • All the other distributions are not covered (Arch, Gentoo, etc.).

Avoid suggesting distro-specific package names such as
linux-headers-<version> or kernel-devel-<version>, since package
naming differs across distributions. Point users to their
distribution's matching kernel headers package, or
--kernelsourcedir.

Signed-off-by: Hadi Hbus <hhbus@nvidia.com>
@HadiHbus HadiHbus force-pushed the fix-rpm-kernel-devel-message branch from fd2374c to cb6297f Compare July 13, 2026 11:26
@HadiHbus HadiHbus changed the title dkms: Mention rpm-based package in missing kernel headers error dkms: Use a generic missing kernel headers error message Jul 13, 2026
@scaronni scaronni merged commit b07d7fa into dkms-project:main Jul 13, 2026
31 checks passed
@HadiHbus HadiHbus deleted the fix-rpm-kernel-devel-message branch July 13, 2026 12:42
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