From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: Denys Dmytriyenko <denis@denix.org>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: dbus: only use x11 if DISTRO_FEATURES has selected it
Date: Thu, 25 Oct 2012 10:58:47 +0100 [thread overview]
Message-ID: <1919390.bnYn62cq03@helios> (raw)
In-Reply-To: <20121025040826.GG3465@denix.org>
On Thursday 25 October 2012 00:08:26 Denys Dmytriyenko wrote:
> On Fri, Aug 12, 2011 at 04:59:43PM +0100, Phil Blundell wrote:
> > This makes it possible to include dbus in a non-X11-equipped DISTRO.
>
> I have a follow up question on this old change...
>
> > Signed-off-by: Phil Blundell <philb@gnu.org>
> > ---
> >
> > meta/recipes-core/dbus/dbus.inc | 7 ++++---
> > 1 files changed, 4 insertions(+), 3 deletions(-)
> >
> > diff --git a/meta/recipes-core/dbus/dbus.inc
> > b/meta/recipes-core/dbus/dbus.inc index 6b76cba..a8ecda8 100644
> > --- a/meta/recipes-core/dbus/dbus.inc
> > +++ b/meta/recipes-core/dbus/dbus.inc
> > @@ -5,7 +5,8 @@ SECTION = "base"
> >
> > LICENSE = "AFL-2 | GPLv2+"
> > LIC_FILES_CHKSUM = "file://COPYING;md5=10dded3b58148f3f1fd804b26354af3e \
> >
> > file://dbus/dbus.h;firstline=6;endline=20;md5=6eea2e0
> > c7750dd8e620dcb1437312fa5">
> > -DEPENDS = "expat virtual/libintl virtual/libx11 libsm"
> > +X11DEPENDS = "virtual/libx11 libsm"
> > +DEPENDS = "expat virtual/libintl ${@base_contains('DISTRO_FEATURES',
> > 'x11', '${X11DEPENDS}', '', d)}">
> > DEPENDS_virtclass-native = "expat-native virtual/libintl-native"
> > DEPENDS_virtclass-nativesdk = "expat-nativesdk virtual/libintl-nativesdk
> > virtual/libx11"
>
> Would there be any objections extending this DISTRO check to nativesdk as
> well?
We did have a request not so long ago to be able to have x11 enabled in the
nativesdk (for development tools that need X) but disabled on the target. I'm
pretty sure we already have a lot of places where DISTRO_FEATURES will
influence nativesdk, but I suspect the configuration may need to be separated
for these two at some point.
Cheers,
Paul
--
Paul Eggleton
Intel Open Source Technology Centre
next prev parent reply other threads:[~2012-10-25 10:12 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-12 15:59 dbus: only use x11 if DISTRO_FEATURES has selected it Phil Blundell
2011-08-15 17:07 ` Saul Wold
2012-10-25 4:08 ` Denys Dmytriyenko
2012-10-25 9:26 ` Burton, Ross
2012-10-25 9:58 ` Paul Eggleton [this message]
2012-10-25 11:44 ` Richard Purdie
-- strict thread matches above, loose matches on Subject: below --
2011-08-16 5:49 James Limbouris
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=1919390.bnYn62cq03@helios \
--to=paul.eggleton@linux.intel.com \
--cc=denis@denix.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox