Bug 1056192 - kernel-PAE-modules-extra no longer provides kernel-modules-extra
Summary: kernel-PAE-modules-extra no longer provides kernel-modules-extra
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: xl2tpd
Version: rawhide
Hardware: i686
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Paul Wouters
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1058331
TreeView+ depends on / blocked
 
Reported: 2014-01-21 16:35 UTC by Bruno Wolff III
Modified: 2014-05-15 12:53 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-05-15 12:53:16 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch to switch to using Requires on individual kernel modules (1.49 KB, patch)
2014-05-14 18:27 UTC, Josh Boyer
no flags Details | Diff

Description Bruno Wolff III 2014-01-21 16:35:13 UTC
Description of problem:
kernel-PAE-modules-extra no longer provides kernel-modules-extra, so that if you are using a PAE kernel, dlm will pull in non-PAE kernel packages.

Depending on the intent of requiring kernel-modules-extra, requiring installonlypkg(kernel-module) may be a suitable replacement.

Version-Release number of selected component (if applicable):
xl2tpd-0:1.3.1-14.fc20.i686

Comment 1 Didier G 2014-01-26 10:09:26 UTC
Upgrading to fc20 using fedup on a computer running PAE Kernel, xl2tpd requires no-PAE kernel to install.

Comment 2 Bruno Wolff III 2014-01-26 19:37:03 UTC
I got some more feedback on this from the Fedora kernel guys and we probably need to live with this for a while. (I'll add more details later.) Mostly it is just a bit of disk space being wasted, which shouldn't be a significant problem for most people.

Comment 3 Bruno Wolff III 2014-01-27 15:05:29 UTC
installonlypkg(kernel-module) is not suitable as a requires replacement. We should wait for the *.ko autoprovides feature to make changes.

(If we use a requires on installonlypkg(kernel-module) it could potentially bring back the problem that prompted the original problem. Also that provides is special and it is a bad idea to overload its use.)

Comment 4 Andre Robatino 2014-02-10 16:16:40 UTC
Just saw this with the latest F20 kernel update. Three packages including xl2tpd (which the other two require) were removed on a 32-bit PAE machine. Trying to install them pulls in the non-PAE kernel and kernel-modules-extra.

Comment 5 Paul Wouters 2014-05-13 01:58:24 UTC
Bruno: any idea on what the real fix should be for this?

xl2tpd requires kernel-modules-extras for the l2tp_ppp.ko module.

Comment 6 Bruno Wolff III 2014-05-13 05:09:09 UTC
There isn't a perfect fix because you can have multiple kernels installed and for the package system any of the sets can provide the module, but you really want the running kernel set to provide it. However requiring the kernel module should work pretty well in practice. If you look at the tracking bug 1058331, you will see some more info. Josh has set up a test copr and seems pretty close to enabling auto provides for kernel modules.

Comment 7 Bruno Wolff III 2014-05-14 15:20:13 UTC
Auto provides for kernel modules has been enabled in kernel-3.15.0-0.rc5.git2.8.fc21, which should show up in tomorrow's rawhide barring unexpected issues.
You can now use contructs like kmod(joydev.ko) to make sure the joystick driver is installed, instead of requiring kernel-modules-extra. This also makes packages proofed against modules moving between kernel-modules and kernel-modules-extra.

Comment 8 Josh Boyer 2014-05-14 18:27:58 UTC
Created attachment 895578 [details]
Patch to switch to using Requires on individual kernel modules

The attached patch switches xl2tpd from using a Requires on kernel-modules-extra to using a requires on the individual kernel modules now that bug 1058331 is in rawhide.

Please review and let me know if it's OK to commit and build.

Comment 9 Paul Wouters 2014-05-14 19:39:33 UTC
looks okay. go ahead or I'll do it as soon as this appears in rawhide and the fedora branches.

Comment 10 Josh Boyer 2014-05-15 12:53:16 UTC
Pushed and building.  Thanks for the review Paul.


Note You need to log in before you can comment on or make changes to this bug.