[OE-core] [PATCH 1/1] vala.bbclass: add dependency on vala
Richard Purdie
richard.purdie at linuxfoundation.org
Tue Sep 24 08:24:09 UTC 2013
On Mon, 2013-09-23 at 21:42 +0000, Slater, Joseph wrote:
> From: Joe Slater [mailto:jslater at windriver.com]
> Sent: Monday, September 23, 2013 1:53 PM
> To: MacLeod, Randy; Polk, Jeffrey
> Cc: lpd-eng-rr
> Subject: [PATCH 1/1] vala.bbclass: add dependency on vala
>
> This class points the inheritor to directories in
> the target sysroot, so we want to be sure things
> like .vapi files are there.
>
> Signed-off-by: Joe Slater <jslater at windriver.com>
> ---
> meta/classes/vala.bbclass | 6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/meta/classes/vala.bbclass b/meta/classes/vala.bbclass
> index c7db08c..0a98dab 100644
> --- a/meta/classes/vala.bbclass
> +++ b/meta/classes/vala.bbclass
> @@ -1,9 +1,9 @@
> # Vala has problems with multiple concurrent invocations
> PARALLEL_MAKE = ""
>
> -# Vala needs vala-native
> -DEPENDS += "vala-native"
> -DEPENDS_virtclass-native += "vala-native"
> +# Vala needs vala-native (and we need vala)
> +DEPENDS += "vala-native vala"
> +DEPENDS_class-native += "vala-native"
In class code this is a really bad idea. Why? Imagine a recipe which
does:
DEPENDS =+ "x"
DEPENDS_class-native = "y"
The += on the class-native above will not do what you think.
I'd recommend something like:
VALADEPENDS = "vala vala-native"
VALADEPENDS_class-native = "vala-native"
DEPENDS += "${VALADEPENDS}"
Most other classes actually use _append on DEPENDS to ensure consistent
behaviour.
Cheers,
Richard
More information about the Openembedded-core
mailing list