All of lore.kernel.org
 help / color / mirror / Atom feed
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




  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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.