open-gpu-kernel-modules icon indicating copy to clipboard operation
open-gpu-kernel-modules copied to clipboard

Align DKMS package name with RPM spec

Open cartermckinnon opened this issue 2 years ago • 8 comments

The RPM spec for the kernel module DKMS package uses nvidia-open as the package name, but the dkms.conf in this repository uses nvidia.

If you:

yum install https://developer.download.nvidia.com/compute/cuda/repos/rhel7/x86_64/kmod-nvidia-open-dkms-545.23.06-1.el7.x86_64.rpm

Then you end up with a package in your DKMS tree called nvidia-open because of this: https://github.com/NVIDIA/yum-packaging-dkms-nvidia/blob/b28ed69631673c076b797caeec5fd83c452a6f61/dkms-open-nvidia.spec#L2C9-L2C18

If you mktarball and ldtarball that package, it becomes nvidia.

cartermckinnon avatar Oct 31 '23 18:10 cartermckinnon

Thanks for pointing this out. I'm unsure if the nvidia-open in kmod-nvidia-open-dkms-545.23.06-1.el7.x86_64.rpm is correct. I'll check with the maintainer of that package.

aritger avatar Oct 31 '23 18:10 aritger

FWIW, It's convenient in my case to use nvidia-open, to distinguish it from the closed-source kmod in my DKMS tree.

cartermckinnon avatar Oct 31 '23 18:10 cartermckinnon

@aritger do you have any update on this? I'd like to get rid of the sed I have to work around this 😄

cartermckinnon avatar Jan 13 '24 00:01 cartermckinnon

Sorry, I didn't get much traction on this before. I've filed NVIDIA internal bug 4360392 for this, though.

aritger avatar Jan 13 '24 01:01 aritger

Sounds good, thanks for looking into it!

cartermckinnon avatar Jan 13 '24 01:01 cartermckinnon

Hey @aritger, is there any update for this?

Issacwww avatar Jun 04 '24 23:06 Issacwww

Sorry, we haven't made any progress on this, yet.

aritger avatar Jun 05 '24 05:06 aritger

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jun 06 '24 06:06 CLAassistant

I'm just going to close this to clear out my dashboard; but I believe this is still an issue.

cartermckinnon avatar Nov 05 '25 20:11 cartermckinnon