]> code.ossystems Code Review - openembedded-core.git/commit
alsa-utils: Fix error when removing unwanted udev rules
authorMike Crowe <mac@mcrowe.com>
Wed, 2 May 2018 13:38:03 +0000 (14:38 +0100)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 4 May 2018 08:55:00 +0000 (09:55 +0100)
commit022b644e6ba2caa0b32ce3323621c07f78166234
tree643c87b1189ef6cca1215279b030655053af0581
parentc16bc7c9b1526ff4b9496af00ada08aa4109c0ef
alsa-utils: Fix error when removing unwanted udev rules

If alsa-utils configure is not passed a --with-udev-rules-dir option then
it defaults to using /lib/udev/rules.d. This meant that the hard-coded use
of ${D}/lib in do_install in 262e69c9c7acf0beb7bb6b96299e3c993c906434
worked correctly to remove the unwanted rules.

Subsequently, 0a4372705a030ca54ed420cdfec33d46ab93499c changed do_install
to use ${nonarch_base_libdir}, claiming to fix this in the usrmerge case.

This means that if udev is not present in PACKAGECONFIG and usrmerge is
present in DISTRO_FEATURES then the alsa-utils build system will install
the rules in ${D}/lib/udev/rules.d but do_install will attempt to remove
${D}/usr/lib, resulting in something like:

 rmdir: failed to remove '.../tmp-glibc/work/i586-oe-linux/alsa-utils/1.1.5-r0/image/usr/lib': No such file or directory

To fix this, let's just tell configure to install the rules in a specific
known location when udev is disabled. This location can then easily be
cleaned up in do_install without doing any harm if udev is enabled.

Tested both with and without usrmerge in DISTRO_FEATURES and with and
without udev in PACKAGECONFIG.

Signed-off-by: Mike Crowe <mac@mcrowe.com>
Cc: Phil Blundell <pb@pbcl.net>
Cc: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/recipes-multimedia/alsa/alsa-utils_1.1.5.bb