Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: "Burton, Ross" <ross.burton@intel.com>
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 2/4] recipes: add x11 to required DISTRO_FEATURES
Date: Thu, 24 Jul 2014 17:22:18 +0200	[thread overview]
Message-ID: <20140724152218.GM2492@jama> (raw)
In-Reply-To: <CAJTo0La2AZo=nxBJGnkQrC+=jB9hoO+tsYyuCMbfAT0vwwd-qg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1937 bytes --]

On Thu, Jul 24, 2014 at 02:52:45PM +0100, Burton, Ross wrote:
> On 24 July 2014 14:42, Martin Jansa <martin.jansa@gmail.com> wrote:
> > +REQUIRED_DISTRO_FEATURES = "x11"
> 
> Now I'm wondering why this is the solution.
> 
> If you attempt to build e.g. gnome-desktop explicitly without the x11
> distro feature you understandably get an error message, because
> gnome-desktop depends on libx11 which sanity checks the distro
> features.  This seems correct.
> 
> Presumably you're problem is that you're running world builds and
> they're producing errors on gnome-desktop because they can't satisfy a
> dependency on libx11.  It seems that bubbling up the
> REQUIRED_DISTRO_FEATURES tests isn't the right thing to do here - why
> can't SkipPackage be handled specially, so if you do bitbake -k world
> and libx11 emits SkipPackage, anything that has unsatisfiable
> dependencies because of this is also skipped?

We discussed this many months ago and IIRC the conclusion was that user
should explicitly say that he wants to skip the recipes which depend on
something skipped (so that he is aware of what he is missing).

At that time there wasn't REQUIRED_DISTRO_FEATURES support, so I've
created huge list of PNBLACKLISTs to blacklist everything not available
in our setup - so I can do world builds without ERRORs at the beginning.

REQUIRED_DISTRO_FEATURES seems to me like reasonable compromise, that's
why I've sent this patchset to replace small part of my huge blacklist.

This is the list:
https://github.com/openwebos/meta-webos/blob/master/conf/distro/include/webos-recipe-blacklist-world.inc

If someone has time to improve SkipPackage and we really want to skip
all depending packages, I would be glad to test such patch (because it
allows to easily drop all those blacklists for "depends-on-broken"
components)

Regards,

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]

  reply	other threads:[~2014-07-24 15:22 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-24 13:42 [PATCH 1/4] xorg-app: add x11 to required DISTRO_FEATURES and cleanup dependencies Martin Jansa
2014-07-24 13:42 ` [PATCH 2/4] recipes: add x11 to required DISTRO_FEATURES Martin Jansa
2014-07-24 13:52   ` Burton, Ross
2014-07-24 15:22     ` Martin Jansa [this message]
2014-07-28 13:18       ` Richard Purdie
2014-08-06 11:48         ` Martin Jansa
2014-10-30 13:20           ` Martin Jansa
2014-10-30 13:28             ` Richard Purdie
2014-10-30 14:18               ` Koen Kooi
2014-11-11 17:13                 ` Martin Jansa
2014-12-10 14:18                   ` [PATCHv2 1/4] xorg-app: add x11 to required DISTRO_FEATURES and cleanup dependencies Martin Jansa
2014-12-10 14:18                     ` [PATCHv2 2/4] recipes: add x11 to required DISTRO_FEATURES Martin Jansa
2015-01-20 21:17                       ` Burton, Ross
2015-01-21 12:27                         ` Martin Jansa
2014-12-10 14:18                     ` [PATCHv2 3/4] xorg-driver: " Martin Jansa
2015-01-20 21:17                       ` Burton, Ross
2015-01-22 16:34                         ` Martin Jansa
2015-01-23 12:46                           ` Burton, Ross
2014-12-10 14:18                     ` [PATCHv2 4/4] recipes-qt: " Martin Jansa
2015-01-20 21:19                       ` Burton, Ross
2015-01-22 16:34                         ` Martin Jansa
2015-01-18 17:06                     ` [PATCHv2 1/4] xorg-app: add x11 to required DISTRO_FEATURES and cleanup dependencies Martin Jansa
2015-01-20 20:12                       ` Burton, Ross
2014-07-24 13:42 ` [PATCH 3/4] xorg-driver: add x11 to required DISTRO_FEATURES Martin Jansa
2014-07-24 13:47   ` Burton, Ross
2014-07-24 13:42 ` [PATCH 4/4] recipes-qt: " Martin Jansa

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=20140724152218.GM2492@jama \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=ross.burton@intel.com \
    /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