Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH] default-providers: fix PREFERRED_PROVIDER_udev
@ 2013-03-16 11:49 Andreas Oberritter
  2013-03-16 21:52 ` Andreas Oberritter
  0 siblings, 1 reply; 3+ messages in thread
From: Andreas Oberritter @ 2013-03-16 11:49 UTC (permalink / raw)
  To: openembedded-core

This must be either systemd or udev, but not both. Prefer
systemd whenever systemd is set in DISTRO_FEATURES.

Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
---
 meta/conf/distro/include/default-providers.inc |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/distro/include/default-providers.inc b/meta/conf/distro/include/default-providers.inc
index ebf2a09..f7ee789 100644
--- a/meta/conf/distro/include/default-providers.inc
+++ b/meta/conf/distro/include/default-providers.inc
@@ -38,4 +38,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)}${@base_contains('DISTRO_FEATURES','systemd','systemd','',d)}"
+PREFERRED_PROVIDER_udev ?= "${@base_contains('DISTRO_FEATURES','systemd','systemd','udev',d)}"
-- 
1.7.10.4




^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] default-providers: fix PREFERRED_PROVIDER_udev
  2013-03-16 11:49 [PATCH] default-providers: fix PREFERRED_PROVIDER_udev Andreas Oberritter
@ 2013-03-16 21:52 ` Andreas Oberritter
  2013-03-17  0:07   ` Burton, Ross
  0 siblings, 1 reply; 3+ messages in thread
From: Andreas Oberritter @ 2013-03-16 21:52 UTC (permalink / raw)
  To: openembedded-core

On 16.03.2013 12:49, Andreas Oberritter wrote:
> --- a/meta/conf/distro/include/default-providers.inc
> +++ b/meta/conf/distro/include/default-providers.inc
> @@ -38,4 +38,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)}${@base_contains('DISTRO_FEATURES','systemd','systemd','',d)}"
> +PREFERRED_PROVIDER_udev ?= "${@base_contains('DISTRO_FEATURES','systemd','systemd','udev',d)}"

Please ignore this patch. Ross already did this is his hybrid
sysvinit/systemd branch.

Regards,
Andreas




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] default-providers: fix PREFERRED_PROVIDER_udev
  2013-03-16 21:52 ` Andreas Oberritter
@ 2013-03-17  0:07   ` Burton, Ross
  0 siblings, 0 replies; 3+ messages in thread
From: Burton, Ross @ 2013-03-17  0:07 UTC (permalink / raw)
  To: Andreas Oberritter; +Cc: openembedded-core

On 16 March 2013 14:52, Andreas Oberritter <obi@opendreambox.org> wrote:
> Please ignore this patch. Ross already did this is his hybrid
> sysvinit/systemd branch.

For what it's worth that behaviour was intentional, although we
appeared to have neglected it from the commit message.  The intention
at the time was that specifying both would cause the udev provider to
be nonsense, producing an error.

Ross



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-03-17  0:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-16 11:49 [PATCH] default-providers: fix PREFERRED_PROVIDER_udev Andreas Oberritter
2013-03-16 21:52 ` Andreas Oberritter
2013-03-17  0:07   ` Burton, Ross

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox