Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 433912 - app-emulation/vmware-modules-264.4 installs udev rules into /lib/udev/rules.d
Summary: app-emulation/vmware-modules-264.4 installs udev rules into /lib/udev/rules.d
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Vadim Kuznetsov (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: udevdir
  Show dependency tree
 
Reported: 2012-09-04 11:33 UTC by Martin von Gagern
Modified: 2012-12-12 11:47 UTC (History)
1 user (show)

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


Attachments
Patch to fix udev rules location (vmware-modules-264.4-r1.ebuild.patch,832 bytes, patch)
2012-09-25 11:55 UTC, Paolo Pedroni
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin von Gagern 2012-09-04 11:33:49 UTC
The emerge log for sys-fs/udev-189 wrote:

"This version of udev moves the files that were installed in
 /lib/udev to /usr/lib/udev.
 We include a backward compatibility patch for gentoo to
 allow the rules in /lib/udev/rules.d to be read. However,
 bugs should be filed against packages that are installing
 files in /lib/udev so they can be fixed."

app-emulation/vmware-modules-264.4 installs files into that old location, so it should be fixed. Remerging the package is not enough to fix this issue.
Comment 1 Paolo Pedroni 2012-09-25 11:55:59 UTC
Created attachment 324934 [details, diff]
Patch to fix udev rules location

The attached patch does the trick.
Comment 2 Paolo Pedroni 2012-09-25 12:08:28 UTC
Comment on attachment 324934 [details, diff]
Patch to fix udev rules location

>--- /usr/portage/app-emulation/vmware-modules/vmware-modules-264.4.ebuild	2012-06-17 15:09:10.000000000 +0200
>+++ /usr/portage/app-emulation/vmware-modules/vmware-modules-264.4-r1.ebuild	2012-09-25 12:26:18.737602513 +0200
>@@ -4,7 +4,7 @@
> 
> EAPI="4"
> 
>-inherit eutils flag-o-matic linux-info linux-mod user versionator
>+inherit eutils flag-o-matic linux-info linux-mod user versionator toolchain-funcs
> 
> PV_MAJOR=$(get_major_version)
> PV_MINOR=$(get_version_component_range 2)
>@@ -77,6 +77,9 @@
> 		KERNEL=="vmmon", GROUP="vmware", MODE=660
> 		KERNEL=="vsock", GROUP="vmware", MODE=660
> 	EOF
>-	insinto /lib/udev/rules.d/
>+
>+	local udevdir="$($(tc-getPKG_CONFIG) --variable=udevdir udev)"
>+	has_version sys-fs/udev && udevdir="$($(tc-getPKG_CONFIG) --variable=udevdir udev)"
>+	insinto "${udevdir}/rules.d"
> 	doins "${udevrules}"
> }
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2012-12-12 11:47:26 UTC
+  12 Dec 2012; Samuli Suominen <ssuominen@gentoo.org>
+  vmware-modules-238.6.ebuild, vmware-modules-264.4.ebuild,
+  vmware-modules-271.1.ebuild:
+  Use udev.eclass and udev_dorules to install rules into correct directory wrt
+  #433912 by Martin von Gagern

Thanks for the patch, but we deprecated that method and started using udev.eclass to minimize duplication of code around ebuilds