Created attachment 894131 [details] Strip compression suffixes from kmod Provides Description of problem: With bug 1025513, Kyle added support to generate kmod(foo.ko) provides for kernel modules. That works fine, however we're looking at shipping xz compressed modules. That means the existing script would generate: Provides: kmod(foo.ko.xz) instead of Provides: kmod(foo.ko) (the modules.builtin provides aren't impacted). We'll be compressing the modules with xz, but the userspace utilities can support xz or gzip. The attached patch fixes the kmod.prov script to strip off the .gz/.xz suffix and generate the Provides we'd still like. I've done a build in COPR against a redhat-rpm-config that has this patch included. The provides appear to be generated correctly. You can find the results here: http://copr-be.cloud.fedoraproject.org/results/jwboyer/kernel-kmod-autoprov/fedora-rawhide-x86_64/kernel-3.15.0-0.rc4.git4.8.fc21.2/ If this looks OK to everyone, I can push it to git and kick off a build.
I did a quick check and the dependencies seemed to get resolved properly: Resolving Dependencies --> Running transaction check ---> Package joystick-support.noarch 0:1.0.0-10.2.fc21 will be installed --> Processing Dependency: kmod(joydev.ko) for package: joystick-support-1.0.0-10.2.fc21.noarch --> Running transaction check ---> Package kernel-PAE-modules-extra.i686 0:3.15.0-0.rc4.git4.8.fc21.2 will be installed --> Finished Dependency Resolution
(In reply to Josh Boyer from comment #0) > If this looks OK to everyone, I can push it to git and kick off a build. As for the script change, that's yours to change :) For the spec part, please note that redhat-rpm-config modifications should always just bump the version and leave release alone (as commented at the top of the spec). Its unlike anything else yes, but then redhat-rpm-config is.
(In reply to Panu Matilainen from comment #2) > (In reply to Josh Boyer from comment #0) > > If this looks OK to everyone, I can push it to git and kick off a build. > > As for the script change, that's yours to change :) OK. > For the spec part, please note that redhat-rpm-config modifications should > always just bump the version and leave release alone (as commented at the > top of the spec). Its unlike anything else yes, but then redhat-rpm-config > is. Ah, sorry. Missed that. I'll fix it before I commit/push. Thanks Panu.
Pushed and built.