Patchwork [02/10] default-providers: Automatically set PREFERRED_PROVIDER_udev

login
register
mail settings
Submitter Ross Burton
Date Jan. 19, 2013, 10:47 p.m.
Message ID <7c26b8f7417dfbe245926137acfa61d0f659488c.1358635193.git.ross.burton@intel.com>
Download mbox | patch
Permalink /patch/42941/
State Accepted
Commit 4c8b2ddbecfbddd3e12c04dc39caea9212358276
Headers show

Comments

Ross Burton - Jan. 19, 2013, 10:47 p.m.
From: Radu Moisan <radu.moisan@intel.com>

This is a more generic way to set preferred provider for udev.
We expect to have multiple choices once we integrate other init
managers, and this way we can automatically set it considering
distro settings.

Signed-off-by: Radu Moisan <radu.moisan@intel.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/conf/distro/include/default-providers.inc |    1 +
 1 file changed, 1 insertion(+)
Martin Jansa - Jan. 20, 2013, 11:12 p.m.
On Sat, Jan 19, 2013 at 10:47:05PM +0000, Ross Burton wrote:
> From: Radu Moisan <radu.moisan@intel.com>
> 
> This is a more generic way to set preferred provider for udev.
> We expect to have multiple choices once we integrate other init
> managers, and this way we can automatically set it considering
> distro settings.
> 
> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>  meta/conf/distro/include/default-providers.inc |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/meta/conf/distro/include/default-providers.inc b/meta/conf/distro/include/default-providers.inc
> index 8ed703c..89b6465 100644
> --- a/meta/conf/distro/include/default-providers.inc
> +++ b/meta/conf/distro/include/default-providers.inc
> @@ -37,3 +37,4 @@ PREFERRED_PROVIDER_nativesdk-opkg ?= "nativesdk-opkg"
>  PREFERRED_PROVIDER_console-tools ?= "kbd"
>  PREFERRED_PROVIDER_gzip-native ?= "pigz-native"
>  PREFERRED_PROVIDER_make ?= "make"
> +PREFERRED_PROVIDER_udev ?= "${@base_contains('DISTRO_FEATURES','sysvinit','udev','',d)}"

'systemd' missing in 4th param? or another
"${@base_contains('DISTRO_FEATURES','systemd','systemd','',d)}"
?

> -- 
> 1.7.10.4
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
Martin Jansa - Jan. 20, 2013, 11:15 p.m.
On Mon, Jan 21, 2013 at 12:12:40AM +0100, Martin Jansa wrote:
> On Sat, Jan 19, 2013 at 10:47:05PM +0000, Ross Burton wrote:
> > From: Radu Moisan <radu.moisan@intel.com>
> > 
> > This is a more generic way to set preferred provider for udev.
> > We expect to have multiple choices once we integrate other init
> > managers, and this way we can automatically set it considering
> > distro settings.
> > 
> > Signed-off-by: Radu Moisan <radu.moisan@intel.com>
> > Signed-off-by: Ross Burton <ross.burton@intel.com>
> > ---
> >  meta/conf/distro/include/default-providers.inc |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/meta/conf/distro/include/default-providers.inc b/meta/conf/distro/include/default-providers.inc
> > index 8ed703c..89b6465 100644
> > --- a/meta/conf/distro/include/default-providers.inc
> > +++ b/meta/conf/distro/include/default-providers.inc
> > @@ -37,3 +37,4 @@ PREFERRED_PROVIDER_nativesdk-opkg ?= "nativesdk-opkg"
> >  PREFERRED_PROVIDER_console-tools ?= "kbd"
> >  PREFERRED_PROVIDER_gzip-native ?= "pigz-native"
> >  PREFERRED_PROVIDER_make ?= "make"
> > +PREFERRED_PROVIDER_udev ?= "${@base_contains('DISTRO_FEATURES','sysvinit','udev','',d)}"
> 
> 'systemd' missing in 4th param? or another
> "${@base_contains('DISTRO_FEATURES','systemd','systemd','',d)}"
> ?

Ah ignore this, it's added in 05/10.

Patch

diff --git a/meta/conf/distro/include/default-providers.inc b/meta/conf/distro/include/default-providers.inc
index 8ed703c..89b6465 100644
--- a/meta/conf/distro/include/default-providers.inc
+++ b/meta/conf/distro/include/default-providers.inc
@@ -37,3 +37,4 @@  PREFERRED_PROVIDER_nativesdk-opkg ?= "nativesdk-opkg"
 PREFERRED_PROVIDER_console-tools ?= "kbd"
 PREFERRED_PROVIDER_gzip-native ?= "pigz-native"
 PREFERRED_PROVIDER_make ?= "make"
+PREFERRED_PROVIDER_udev ?= "${@base_contains('DISTRO_FEATURES','sysvinit','udev','',d)}"