Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 265281 - app-emulation/virtualbox-modules message is incorrect for sys-apps/openrc users
Summary: app-emulation/virtualbox-modules message is incorrect for sys-apps/openrc users
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Patrick Lauer
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-07 11:21 UTC by Michael Sawczuk
Modified: 2010-06-08 16:15 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Proposed fix (virtualbox-modules-3.0.12.ebuild.patch,571 bytes, patch)
2010-01-17 17:51 UTC, Trevor Summers Smith
Details | Diff
Proposed fix (virtualbox-modules-3.1.0.ebuild.patch,569 bytes, patch)
2010-01-17 17:52 UTC, Trevor Summers Smith
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Sawczuk 2009-04-07 11:21:51 UTC
After merging virtualbox-modules package, the message at the end of the build tells the user to add the modules to /etc/modules.autoload.d/kernel-2.6 however this would not work for OpenRC users so perhaps the message could be tweaked to state the location needed for OpenRC users (/etc/conf.d/modules)

Reproducible: Always

Steps to Reproduce:
Comment 1 Alessio Cassibba (X-Drum) 2009-04-29 16:24:51 UTC
fixed on jokey's overlay[1] thanks for pointing this out

Alessio

[1] http://overlays.gentoo.org/dev/jokey
Comment 2 Trevor Summers Smith 2010-01-17 17:51:47 UTC
Created attachment 216745 [details, diff]
Proposed fix

These two patches fix the problem in both ebuilds. I tested this with OpenRC 0.6.0. 

N.b. the patch uses the temporary method discussed in bug #270646 to detect
OpenRC.
Comment 3 Trevor Summers Smith 2010-01-17 17:52:55 UTC
Created attachment 216746 [details, diff]
Proposed fix

See comments for attachment virtualbox-modules-3.0.12.ebuild.patch.
Comment 4 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2010-06-08 16:15:26 UTC
This is already fixed