From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 5/7] alsa-utils: use base_libdir for udev rules.
Date: Fri, 27 Apr 2012 11:47:17 +0100 [thread overview]
Message-ID: <1335523637.20130.57.camel@ted> (raw)
In-Reply-To: <94e2c16d1d19f800c14e012b0fc24b8117944652.1335498445.git.lianhao.lu@intel.com>
On Fri, 2012-04-27 at 13:04 +0800, Lianhao Lu wrote:
> Use the base_libdir to set the udev rules directory.
>
> Signed-off-by: Lianhao Lu <lianhao.lu@intel.com>
> ---
> meta/recipes-multimedia/alsa/alsa-utils_1.0.25.bb | 6 ++++--
> 1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.0.25.bb b/meta/recipes-multimedia/alsa/alsa-utils_1.0.25.bb
> index 597e8b6..c39cd11 100644
> --- a/meta/recipes-multimedia/alsa/alsa-utils_1.0.25.bb
> +++ b/meta/recipes-multimedia/alsa/alsa-utils_1.0.25.bb
> @@ -6,7 +6,7 @@ LICENSE = "GPLv2+"
> LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552 \
> file://alsactl/utils.c;beginline=1;endline=20;md5=fe9526b055e246b5558809a5ae25c0b9"
> DEPENDS = "alsa-lib ncurses libsamplerate0"
> -PR = "r2"
> +PR = "r3"
>
> SRC_URI = "ftp://ftp.alsa-project.org/pub/utils/alsa-utils-${PV}.tar.bz2 \
> file://ncursesfix.patch \
> @@ -21,7 +21,9 @@ SRC_URI[sha256sum] = "2e676a2f634bbfe279b260e10a96f617cb72ee63c5bbf6c5f96bb61570
> # http://bugs.openembedded.org/show_bug.cgi?id=2348
> # please close bug and remove this comment when properly fixed
> #
> -EXTRA_OECONF = "--disable-xmlto"
> +EXTRA_OECONF = "--disable-xmlto \
> + --with-udev-rules-dir=${base_libdir}/udev/rules.d \
> + "
> EXTRA_OECONF_append_libc-uclibc = " --disable-nls"
>
> inherit autotools gettext
The udev configuration files are probably one case where using /lib
makes a lot of sense since we should only have one set of these. I
therefore think this should be added as an exception to the sanity
test...
Cheers,
Richard
next prev parent reply other threads:[~2012-04-27 10:57 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-27 5:03 [PATCH 0/7] Misc fixings for multilib Lianhao Lu
2012-04-27 5:04 ` [PATCH 1/7] multilib.bbclass: Added multilib specific package QA Lianhao Lu
2012-04-27 5:04 ` [PATCH 2/7] popt: Fixing pkgconfig file installation issue Lianhao Lu
2012-04-27 5:04 ` [PATCH 3/7] avahi: not using hard coded libdir Lianhao Lu
2012-04-27 5:04 ` [PATCH 4/7] pulseaudio: use base_libdir for udev rules Lianhao Lu
2012-04-27 5:04 ` [PATCH 5/7] alsa-utils: " Lianhao Lu
2012-04-27 10:47 ` Richard Purdie [this message]
2012-04-27 11:22 ` Koen Kooi
2012-04-27 5:04 ` [PATCH 6/7] xorg-app: Use ${libdir} for packaging Lianhao Lu
2012-04-27 5:36 ` Martin Jansa
2012-04-27 5:04 ` [PATCH 7/7] xf86-input-vmmouse: use base_libdir for udev rules Lianhao Lu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1335523637.20130.57.camel@ted \
--to=richard.purdie@linuxfoundation.org \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox