Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH] navit: enable qml build
Date: Fri, 11 Jun 2010 08:18:38 +0200	[thread overview]
Message-ID: <20100611061838.GS1403@jama> (raw)
In-Reply-To: <4C11CF60.5060106@freyther.de>

On Fri, Jun 11, 2010 at 01:53:36PM +0800, Holger Freyther wrote:
> On 06/11/2010 04:07 AM, Martin Jansa wrote:
> > * but for now without DEPENDS on qt4-x11-free, because 4.7.0 is not yet
> >   released and building qt4-x11-free just because of navit (for someone
> >   not using it for something else) takes a lot of time
> 
> I might not understand everything here, but this looks dangerous. A
> package rebuild should always generate the same kind of package.

Yes it's dangerous indeed.

Python DEPENDS contains 
${@base_contains('DISTRO_FEATURES', 'tk', 'tk', '', d)}

Then it's ensured that if you have tk in DISTRO_FEATURES then it's
picked by automatic in do_configure. But if you don't have it in
DISTRO_FEATURES then it depends if you build something depending on tk
before trying to build python or not. Even worse case is when you build
an image (ie kernel, python (without tk), th, something_depending_on_tk)
and then someone PR bump python and now you'll get python with tk).

I guess there is plenty of examples where configure has "automatic" by
default and we don't DEPEND on optional stuff and also don't force it
with --disable-some-optional-feature :/.

I think that best solution would be something like
DEPENDS += "${@base_contains('DISTRO_FEATURES', 'qt', 'qt4-x11-free', '', d)}"
EXTRA_OECONF += "${@base_contains('DISTRO_FEATURES', 'qt', '--enable-gui-qml', '--disable-gui-qml', d)}"

which is not so common in OE AFAIK and looks like how USE flags are
handled in gentoo (but they have better syntax to make it shorter).

Regards,

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



      reply	other threads:[~2010-06-11  6:23 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-10 20:07 [PATCH] navit: enable qml build Martin Jansa
2010-06-11  5:53 ` Holger Freyther
2010-06-11  6:18   ` Martin Jansa [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=20100611061838.GS1403@jama \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-devel@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