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 -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
