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

Submitted by Ross Burton on Jan. 19, 2013, 10:47 p.m.

Details

Message ID 7c26b8f7417dfbe245926137acfa61d0f659488c.1358635193.git.ross.burton@intel.com
State Accepted
Commit 4c8b2ddbecfbddd3e12c04dc39caea9212358276
Headers show

Commit Message

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(+)

Patch hide | download patch | download mbox

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)}"

Comments

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.