All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Joe Slater <jslater@windriver.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/1] vala: add dependencies on native packages
Date: Tue, 08 Apr 2014 17:54:01 +0100	[thread overview]
Message-ID: <1396976041.24597.107.camel@ted> (raw)
In-Reply-To: <1396912345-7880-1-git-send-email-jslater@windriver.com>

On Mon, 2014-04-07 at 16:12 -0700, Joe Slater wrote:
> We do not want to run the host xsltproc one time, and ours
> another, so we add a dependency on libxslt-native.  We also
> need to add a dependency on pkgconfig-native because the host
> pkg-config will not see native packages we have built.
> 
> Signed-off-by: Joe Slater <jslater@windriver.com>
> ---
>  meta/recipes-devtools/vala/vala.inc |   11 +++++++++--
>  1 files changed, 9 insertions(+), 2 deletions(-)

Could you send something against master please?

Specifically, master has:

DEPENDS = "bison-native flex-native libxslt-native"

so this is half fixed there...

Cheers,

Richard

> diff --git a/meta/recipes-devtools/vala/vala.inc b/meta/recipes-devtools/vala/vala.inc
> index a8927d7..9d273aa 100644
> --- a/meta/recipes-devtools/vala/vala.inc
> +++ b/meta/recipes-devtools/vala/vala.inc
> @@ -3,7 +3,13 @@ Vala compiles to plain C and has no runtime environment nor penalities whatsoeve
>  SECTION = "devel"
>  DEPENDS = "glib-2.0 dbus gtk+"
>  BBCLASSEXTEND = "native"
> -DEPENDS_virtclass-native = "bison-native glib-2.0-native dbus-native"
> +
> +# libxslt provides xsltproc, which is probably on the host, but we want to use ours.
> +# We need our pkg-config, too.  (Inheriting pkgconfig supplies that dependency, but
> +# we clobber it here).
> +#
> +DEPENDS_virtclass-native = "bison-native glib-2.0-native dbus-native libxslt-native pkgconfig-native"
> +
>  HOMEPAGE = "http://vala-project.org"
>  LICENSE = "LGPLv2.1"
>  LIC_FILES_CHKSUM = "file://COPYING;md5=fbc093901857fcd118f065f900982c24"
> @@ -12,7 +18,8 @@ INC_PR = "r1"
>  SHRT_VER = "${@d.getVar('PV',1).split('.')[0]}.${@d.getVar('PV',1).split('.')[1]}"
>  
>  SRC_URI = "http://ftp.gnome.org/pub/GNOME/sources/${BPN}/${SHRT_VER}/${BP}.tar.xz"
> -inherit autotools
> +
> +inherit autotools pkgconfig
>  
>  EXTRA_OECONF = "--disable-vapigen"
>  
> -- 
> 1.7.3.4
> 




      reply	other threads:[~2014-04-08 16:54 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-07 23:12 [PATCH 1/1] vala: add dependencies on native packages Joe Slater
2014-04-08 16:54 ` Richard Purdie [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1396976041.24597.107.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=jslater@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.