All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Hatle <mark.hatle@windriver.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: RDEPENDS of packages created by PACKAGES_DYNAMIC not built before image rootfs
Date: Wed, 28 Nov 2012 10:24:13 -0600	[thread overview]
Message-ID: <50B63AAD.4080506@windriver.com> (raw)
In-Reply-To: <1354080360.21863.111.camel@ted>

On 11/27/12 11:26 PM, Richard Purdie wrote:
> On Tue, 2012-11-27 at 15:07 -0600, Mark Hatle wrote:
>> On 11/27/12 2:23 PM, Martin Jansa wrote:
>>>> Bitbake doesn't know which modules this package will produce since its
>>>> using PACKAGES_DYNAMIC. It does its best effort to try and find DEPENDS
>>>> but it can't take the regexps and come up with
>>>> "pulseaudio-module-console-kit", hence it doesn't see the RDEPENDS.
>>>> There isn't any generic way we can teach it to do that either. If it
>>>> did, it would simply always build consolekit which you say you don't
>>>> want so its not relevant anyway.
>>>>
>>>> So basically, you need to decide whether you want consolekit enabled or
>>>> not and build accordingly, there is nothing you can do within bitbake to
>>>> "mind read" what the image is going to want...
>>
>> For the resolver, it has to use the RPROVIDES, PACKAGES and PACKAGES_DYNAMIC...
>
> and it does to the limits that are reasonable...
>
>>> No, you know that you will need pulseaudio-module-console-kit because
>>> pulseaudio-server RDEPENDs on it (not on any pulseaudio-module-* but
>>> pulseaudio-module-console-kit exactly). So it's not really mind reading.
>>
>> But after the recipe has been built, the required item from PACKAGES_DYNAMIC
>> should now be in PACKAGES... and if it doesn't end up in the PACKAGES list, it
>> would be a good idea to at a minimum trigger a warning...  The catch is, is
>> there any way to tell if something from PACKAGES_DYNAMIC was used..
>
> "After it is built" is no good to bitbake. It has to know what its doing
> without building things.
>
> All Bitbake can see is that there is some regexp in PACKAGES_DYNAMIC.
>
> Yes, Bitbake could take all the keys in the datastore, expand each one
> (since it could be constructed using another variable name) and find the
> ones starting with RDEPENDS, then apply the rexexp to each key and see
> if it matches. We one tried something like this and it is extremely
> slow, parsing would take a massive speed hit.

I was more thinking in the oe-core recipe QA stuff..  if there was a way to know 
that the following items were matched in PACKAGES_DYNAMIC for this item, and 
then check to see if they ended up in PACKAGES.  Then warn/error as appropriate 
if they don't match.

> The rule is therefore in corner cases like this (which are rather
> unusual), you add something to DEPENDS and be done with it.
>
> It is not a reasonable expectation for bitbake to figure this kind of
> thing out.

Ya, I wasn't advocating a change in bitbake for the detection, just a way to 
pass the data to the recipe and do it at that point.

> I agree that if it builds things and discovers some kind of missing
> dependency, that would be good to tell the user about.

--Mark

> Cheers,
>
> Richard
>
>
>




  reply	other threads:[~2012-11-28 16:38 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-27 15:41 RDEPENDS of packages created by PACKAGES_DYNAMIC not built before image rootfs Martin Jansa
2012-11-27 17:08 ` Otavio Salvador
2012-11-27 20:11 ` Richard Purdie
2012-11-27 20:23   ` Martin Jansa
2012-11-27 21:07     ` Mark Hatle
2012-11-28  5:26       ` Richard Purdie
2012-11-28 16:24         ` Mark Hatle [this message]
2012-11-28  5:35     ` Richard Purdie

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=50B63AAD.4080506@windriver.com \
    --to=mark.hatle@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=richard.purdie@linuxfoundation.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.