Patchwork [35/37] task-base: allow distribution to define apm provider

login
mail settings
Submitter Saul Wold
Date April 23, 2011, 6:29 a.m.
Message ID <1fb84f7adb3291debd163d273be1555bb77b5447.1303539961.git.sgw@linux.intel.com>
Download mbox | patch
Permalink /patch/2773/
State New, archived
Headers show

Comments

Saul Wold - April 23, 2011, 6:29 a.m.
From: Martin Jansa <Martin.Jansa@gmail.com>

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-core/tasks/task-base.bb |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)
Richard Purdie - April 24, 2011, 9:46 p.m.
On Fri, 2011-04-22 at 23:29 -0700, Saul Wold wrote:
> From: Martin Jansa <Martin.Jansa@gmail.com>
> 
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  meta/recipes-core/tasks/task-base.bb |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/meta/recipes-core/tasks/task-base.bb b/meta/recipes-core/tasks/task-base.bb
> index 599fa4b..af49153 100644
> --- a/meta/recipes-core/tasks/task-base.bb
> +++ b/meta/recipes-core/tasks/task-base.bb
> @@ -179,8 +179,11 @@ RDEPENDS_task-base-acpi = "\
>      acpid \
>      libacpi "
>  
> +# Distro can override apm provider
> +DISTRO_APM ?= "apm"
> +
>  RDEPENDS_task-base-apm = "\
> -    apm \
> +    ${DISTRO_APM} \
>      apmd"

I think I replied about this one previously and Martin suggested we use
a different VIRTUAL* name for the variable. I'd appreciate doing that as
DISTRO* is an overloaded namespace. Can someone repsin the patch
please? :)

Cheers,

Richard

Patch

diff --git a/meta/recipes-core/tasks/task-base.bb b/meta/recipes-core/tasks/task-base.bb
index 599fa4b..af49153 100644
--- a/meta/recipes-core/tasks/task-base.bb
+++ b/meta/recipes-core/tasks/task-base.bb
@@ -179,8 +179,11 @@  RDEPENDS_task-base-acpi = "\
     acpid \
     libacpi "
 
+# Distro can override apm provider
+DISTRO_APM ?= "apm"
+
 RDEPENDS_task-base-apm = "\
-    apm \
+    ${DISTRO_APM} \
     apmd"
 
 RDEPENDS_task-base-ext2 = "\