Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 432686 - app-emulation/qemu-kvm should use pkg-config to determine the udev directory to install rules or/and executables
Summary: app-emulation/qemu-kvm should use pkg-config to determine the udev directory ...
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo QEMU Project
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2012-08-25 07:05 UTC by Arne Flagge
Modified: 2012-12-13 16:34 UTC (History)
0 users

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


Attachments
emerge --info (emerge-info,14.91 KB, text/plain)
2012-08-25 07:06 UTC, Arne Flagge
Details
Ebuild patch (qemu-kvm.patch,617 bytes, patch)
2012-08-25 07:06 UTC, Arne Flagge
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Arne Flagge 2012-08-25 07:05:49 UTC
arne ~ # emerge qemu-kvm
app-emulation/qemu-kvm-1.1.1-r3  USE="aio opengl pulseaudio sdl vhost-net -alsa -bluetooth -brltty -caps -curl -debug -doc -fdt -mixemu -ncurses -python (-rbd) -sasl -smartcard -spice -static -systemtap -tci -tls -usbredir -vde -virtfs -xattr -xen -xfs" QEMU_SOFTMMU_TARGETS="i386 x86_64 -alpha -arm -cris -m68k -microblaze -microblazeel -mips -mips64 -mips64el -mipsel -ppc -ppc64 -ppcemb -s390x -sh4 -sh4eb -sparc -sparc64 -xtensa -xtensaeb" QEMU_USER_TARGETS="i386 x86_64 -alpha -arm -armeb -cris -m68k -microblaze -microblazeel -mips -mipsel -ppc -ppc64 -ppc64abi32 -s390x -sh4 -sh4eb -sparc -sparc32plus -sparc64 -unicore32" 0 kB

arne ~ # qlist qemu-kvm | grep udev/rules.d
/lib/udev/rules.d/65-kvm.rules

arne ~ # pkg-config --variable=udevdir udev
/usr/lib/udev


Reproducible: Always
Comment 1 Arne Flagge 2012-08-25 07:06:06 UTC
Created attachment 322164 [details]
emerge --info
Comment 2 Arne Flagge 2012-08-25 07:06:21 UTC
Created attachment 322166 [details, diff]
Ebuild patch
Comment 3 Doug Goldstein (RETIRED) gentoo-dev 2012-09-10 02:26:49 UTC
This is all still being debated on the gentoo-dev ML. I would take it up on there and once the result is decided on we can make the change.
Comment 4 Martin von Gagern 2012-09-10 05:16:14 UTC
(In reply to comment #3)
> This is all still being debated on the gentoo-dev ML.

You mean http://thread.gmane.org/gmane.linux.gentoo.devel/78598 about udev-rules.eclass?

> I would take it up on there and once the result is decided on
> we can make the change.

Doesn't sound like the "CANTFIX" resolution you chose. More like "LATER".
Comment 5 Doug Goldstein (RETIRED) gentoo-dev 2012-09-10 06:04:48 UTC
The problem is "LATER" was not available so I picked the next thing. If you follow the thread you mentioned the decision was to patch udev to always use the hard coded path and not add the eclass. The topic has come up again on the ML without a resolution.
Comment 6 Arne Flagge 2012-09-12 11:01:12 UTC
(In reply to comment #5)
> If you follow the thread you mentioned the decision was to patch udev to always > use the hard coded path and not add the eclass. The topic has come up again on
> the ML without a resolution.
There is no eclass, but see the udev changelog for udev-187-r3:

"I also spoke with the other udev maintainer (Samuli), and he agrees that
we should be using /usr/lib and allowing packages to put things in /lib
for now."

So /usr/lib is the preferred location (for both udev maintainers) for newer udev versions.
Comment 7 Samuli Suominen (RETIRED) gentoo-dev 2012-09-13 21:52:23 UTC
The eclass would be an enchancement but there is nothing stopping fixing this now... 
Every package should use the udevdir= as defined by udev in it's pkg-config file, that's just common sense, it's not something you can debate about unless you get udev upstream to discontinue the dynamic udevdir= system, which is not going to happen...
Comment 8 Samuli Suominen (RETIRED) gentoo-dev 2012-10-27 16:09:58 UTC
no such package as "app-emulation/qemu-kvm" in Portage anymore, if the problem is still present in some other qemu package, please do file a bug

emerge: there are no ebuilds to satisfy "qemu-kvm"