Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 433904 - media-gfx/sane-backends-1.0.22-r2 installs udev rules into /lib/udev/rules.d
Summary: media-gfx/sane-backends-1.0.22-r2 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: Patrick Kursawe (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: udevdir
  Show dependency tree
 
Reported: 2012-09-04 11:23 UTC by Martin von Gagern
Modified: 2012-10-07 10:49 UTC (History)
2 users (show)

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


Attachments
Patch to fix udev rules location (sane-backends-1.0.22-r3.ebuild.patch,668 bytes, patch)
2012-09-25 12:18 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:23:54 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."

media-gfx/sane-backends-1.0.22-r2 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 12:18:16 UTC
Created attachment 324938 [details, diff]
Patch to fix udev rules location

The attached patch does the trick (shamelessly stolen from sys-fs/mdadm-3.2.3-r1 ebuild).
Comment 2 Patrick Kursawe (RETIRED) gentoo-dev 2012-10-06 22:21:03 UTC
Done in .23 with slight modifications - I guess executing the same command one time conditionally, one time unconditionally is a mistake?
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2012-10-07 10:49:26 UTC
(In reply to comment #2)
> Done in .23 with slight modifications - I guess executing the same command
> one time conditionally, one time unconditionally is a mistake?

indeed it is, the correct syntax for ebuild that doesn't have sys-fs/udev in 
DEPEND, the code is:

inherit toolchain-funcs

DEPEND="virtual/pkgconfig"

local udevdir=/lib/udev
has_version sys-fs/udev && udevdir="$($(tc-getPKG_CONFIG) --variable=udevdir udev)"

And it could be around "if use kernel_linux" too, if the package has non-linux keywords