Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Anders Darander <anders@chargestorm.se>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] libx11: skip self if x11 not in DISTRO_FEATURES
Date: Mon, 15 Aug 2011 14:23:27 +0200	[thread overview]
Message-ID: <20110815122327.GA7862@chargestorm.se> (raw)
In-Reply-To: <1313164659.6733.163.camel@phil-desktop>


Although, it is already mergerd by Richard, I really like this patch
series (after testing it).

* Phil Blundell <philb@gnu.org> [110812 17:57]:
> This stops (most) x11 packages leaking into a non-x11 distro by mistake.

Will you submit this to meta-oe also?
This question might apply to a few of the other patches also.

Cheers,
Anders

> Signed-off-by: Phil Blundell <philb@gnu.org>
> ---
>  meta/recipes-graphics/xorg-lib/libx11.inc |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)

> diff --git a/meta/recipes-graphics/xorg-lib/libx11.inc b/meta/recipes-graphics/xorg-lib/libx11.inc
> index c156ce6..455753f 100644
> --- a/meta/recipes-graphics/xorg-lib/libx11.inc
> +++ b/meta/recipes-graphics/xorg-lib/libx11.inc
> @@ -42,3 +42,8 @@ do_compile() {
>  # Multiple libx11 derivatives from from this file and are selected by virtual/libx11
>  # A world build should only build the correct version, not all of them.
>  EXCLUDE_FROM_WORLD = "1"
> +
> +python () {
> +       if not oe.utils.contains ('DISTRO_FEATURES', 'x11', True, False, d):
> +		  raise bb.parse.SkipPackage("X11 not enabled for this DISTRO")
> +}
-- 
Anders Darander
ChargeStorm AB		Tel: +46 702 44 84 36
Laxholmstorget 3	Email: anders@chargestorm.se
602 21 Norrköping	Web: www.chargestorm.se



  reply	other threads:[~2011-08-15 12:27 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-12 15:57 [PATCH] libx11: skip self if x11 not in DISTRO_FEATURES Phil Blundell
2011-08-15 12:23 ` Anders Darander [this message]
2011-08-15 13:07   ` Phil Blundell
2011-08-15 13:15     ` Anders Darander
2011-08-15 13:24       ` Paul Eggleton
2011-08-15 13:32         ` Anders Darander
2011-08-15 13:43           ` Paul Eggleton
2011-08-22 22:32 ` Saul Wold

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=20110815122327.GA7862@chargestorm.se \
    --to=anders@chargestorm.se \
    --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