[OE-core] [PATCH V5 0/2] bitbake-layers: add a ability to query layer dependencies from layer index

Paul Eggleton paul.eggleton at linux.intel.com
Fri Feb 13 10:10:13 UTC 2015


On Thursday 12 February 2015 09:50:21 Chong Lu wrote:
> Change since V4:
> Change commit log for two variables.
> 
> The following changes since commit 35c9fa0588ed8e88b541a6c80cc1517324616cea:
> 
>   maintainers: Update for non-maintained recipes (2015-01-20 21:39:41 +0000)
> 
> are available in the git repository at:
> 
>   git://git.pokylinux.org/poky-contrib chonglu/layerindex
>   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=chonglu/layerindex
> 
> Chong Lu (2):
>   bitbake.conf: Add two variables for layer index
>   bitbake-layers: add a ability to query layer dependencies from layer index

Thanks for fixing nearly all of the issues from v3 - I've tested it and it 
works pretty well. However, there is one that is still not addressed:

> > * I think the new subcommand would make more sense if it were switched
> > around so that it was fetching by default (and renamed as appropriate).
> > Perhaps "layerindex-fetch" (with -n to avoid fetching and just report what
> > it would fetch).

FWIW I'm happy if we still have a "layerindex-show-deps" that is just an alias 
for "layerindex-fetch -n", so it's really about the naming.

Additionally I'd really rather we did not hardcode the branch names - there's 
an API call available to get these in the layer index and we should use it. I 
also think we could improve the formatting of the output to make it a bit 
easier for the user to understand.

I know you'll be out of office for the next few weeks - perhaps I could address 
these issues while you're out and send a new series if you have no objections.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



More information about the Openembedded-core mailing list