Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 447116 - app-emulation/qemu: installs udev rules into /lib/udev/rules.d
Summary: app-emulation/qemu: 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: Doug Goldstein (RETIRED)
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks: udevdir
  Show dependency tree
 
Reported: 2012-12-13 14:46 UTC by Alexander Tsoy
Modified: 2012-12-14 12:24 UTC (History)
1 user (show)

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

Attachments
qemu-1.1.2-r2.ebuild.patch (qemu-1.1.2-r2.ebuild.patch,682 bytes, patch)
2012-12-13 14:47 UTC, Alexander Tsoy 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Tsoy 2012-12-13 14:46:40 UTC 
New versions of udev installs files into /usr/lib/udev/ directory, so qemu ebuilds should use udev.eclass to install rules into proper dir.

$ grep udev /usr/portage/app-emulation/qemu/*.ebuild
/usr/portage/app-emulation/qemu/qemu-1.1.1-r1.ebuild:	# remove part to make udev happy
/usr/portage/app-emulation/qemu/qemu-1.1.1-r1.ebuild:			insinto /lib/udev/rules.d/
/usr/portage/app-emulation/qemu/qemu-1.1.2-r1.ebuild:			insinto /lib/udev/rules.d/
/usr/portage/app-emulation/qemu/qemu-1.1.2-r2.ebuild:			insinto /lib/udev/rules.d/
/usr/portage/app-emulation/qemu/qemu-1.1.2-r3.ebuild:			insinto /lib/udev/rules.d/
/usr/portage/app-emulation/qemu/qemu-1.2.0.ebuild:			insinto /lib/udev/rules.d/
/usr/portage/app-emulation/qemu/qemu-1.2.1.ebuild:			insinto /lib/udev/rules.d/
/usr/portage/app-emulation/qemu/qemu-9999.ebuild:			insinto /lib/udev/rules.d/

Reproducible: Always
Comment 1 Alexander Tsoy 2012-12-13 14:47:39 UTC 
Created attachment 332226 [details, diff]
qemu-1.1.2-r2.ebuild.patch
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2012-12-13 20:06:00 UTC 
+  13 Dec 2012; Samuli Suominen <ssuominen@gentoo.org> qemu-1.1.1-r1.ebuild,
+  qemu-1.1.2-r1.ebuild, qemu-1.1.2-r2.ebuild, qemu-1.1.2-r3.ebuild,
+  qemu-1.2.0.ebuild, qemu-1.2.1.ebuild, qemu-9999.ebuild:
+  Use udev_dorules to install udev rules into correct path wrt #447116 by
+  Alexander Tsoy

+  13 Dec 2012; Samuli Suominen <ssuominen@gentoo.org>
+  qemu-guest-agent-1.1.1.ebuild, qemu-guest-agent-1.2.0.ebuild:
+  Use udev_newrules to install udev rules into correct path wrt #447116 by
+  Alexander Tsoy
Comment 3 iGentoo 2012-12-14 00:14:00 UTC 
There is a typo in the ebuild:
--- qemu-1.2.1.ebuild~
+++ qemu-1.2.1.ebuild
@@ -327,7 +327,7 @@ src_install() {
 		dohtml qemu-doc.html qemu-tech.html || die
 	fi
 
-	use python & dobin scripts/kvm/kvm_stat
+	use python && dobin scripts/kvm/kvm_stat
 
 	# Avoid collision with app-emulation/libcacard
 	use smartcard && mv "${ED}/usr/bin/vscclient" "${ED}/usr/bin/qemu-vscclient"
Comment 4 Alexander Tsoy 2012-12-14 12:24:01 UTC 
(In reply to comment #1)
> Created attachment 332226 [details, diff] [details, diff]
> qemu-1.1.2-r2.ebuild.patch

Yes, udev_dorules of course, not udev_newrules :)

(In reply to comment #3)
> There is a typo in the ebuild:

I've opened a new bug 447226.