From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: Hongxu Jia <hongxu.jia@windriver.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 1/1] list-packageconfig-flag.py: add a script to list all PACKAGECONFIG's flags
Date: Fri, 02 Aug 2013 11:38:20 +0100 [thread overview]
Message-ID: <2141715.fZzytAtgVl@helios> (raw)
In-Reply-To: <51FB787A.2040406@windriver.com>
On Friday 02 August 2013 17:14:34 Hongxu Jia wrote:
> On 08/02/2013 04:24 PM, Paul Eggleton wrote:
> > So, this is fairly simple and works for most cases; however it would
> > probably be a bit more robust if the script used bitbake's own parser to
> > extract the flags. The "tinfoil" module should help with this. We can
> > always address this in a follow-up patch later on, though - I can
> > certainly show you some examples of how to do this or I can do it,
> > whichever you prefer.
>
> Great, this is what I hope to do in the follow-up, I am very pleased to
> do the investigate on bitbake parser, very thankful to share some
> examples of how to do this with me.
Sure - we use the parser in the layer index. The latest in-development code
has a module that does most of the work:
http://git.yoctoproject.org/cgit/cgit.cgi/layerindex-web/tree/layerindex/recipeparse.py?h=paule/features
Specifically, you can call bb.cache.Cache.loadDataFull() for each recipe file
and that will give you a datastore you can query for information.
I'm not sure, but you may wish to use bitbake's cache to get the list of recipes
rather than just walking the directory tree so the utility has the same
understanding of PN and preferred versions as bitbake. If you do want to
go down this route I've put together this example:
https://gist.github.com/bluelightning/6138980
Hope this helps.
> Should this simple patch be accepted? And I will include bitbake parser
> in follow-up patch.
> Or
> Drop this patch and waiting till bitbake parser has been included.
I'd say let's accept this simple version for now after the naming/message
corrections (especially since it's a contrib script) and then follow up
with another patch when you have the bitbake-based version working.
Cheers,
Paul
--
Paul Eggleton
Intel Open Source Technology Centre
next prev parent reply other threads:[~2013-08-02 10:38 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-01 11:04 [PATCH 0/1] list-packageconfig-flag.py: add a script to list all PACKAGECONFIG's flags Hongxu Jia
2013-08-01 11:04 ` [PATCH 1/1] " Hongxu Jia
2013-08-01 16:46 ` Otavio Salvador
2013-08-02 1:50 ` Hongxu Jia
2013-08-02 1:54 ` Otavio Salvador
2013-08-02 8:24 ` Paul Eggleton
2013-08-02 9:14 ` Hongxu Jia
2013-08-02 10:38 ` Paul Eggleton [this message]
2013-08-02 10:03 ` Hongxu Jia
2013-08-02 10:40 ` Paul Eggleton
2013-08-03 3:15 ` Hongxu Jia
-- strict thread matches above, loose matches on Subject: below --
2013-08-02 2:20 [PATCH V2 0/1] " Hongxu Jia
2013-08-02 2:20 ` [PATCH 1/1] " Hongxu Jia
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=2141715.fZzytAtgVl@helios \
--to=paul.eggleton@linux.intel.com \
--cc=hongxu.jia@windriver.com \
--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