Hi Paul, Thanks for the review, I've fixed all the issues you commented except the following ones.
On 06/21/2017 05:01 PM, Paul Eggleton wrote:
Hi Robert,
+ # We deliberately exclude status == 'X' ("no update") here + layerquery_all = LayerItem.objects.filter(classic=False).filter(status='P') + if layerquery_all.count() == 0: + logger.info("No published layers to update") + sys.exit(1)So near as I can tell this change isn't required as part of the other changes here, it's just to tell you up front that no layers are published if you've specified layers instead of seeing an error. I don't have an issue with that but this change should be mentioned in the commit message ("additionally ..."), or split out as a separate patch.
Sorry, it should be in another patch, I added it to wrong place when rebase. I add this patch into "update.py: update layers orderly" now. I need layerquery_all to know all the provided collections in database, so that we can know whether the dependencies match or not.
+ # For -a option to update bitbake branch + update_bitbake = False if options.layers: - layerquery = LayerItem.objects.filter(classic=False).filter(name__in=options.layers.split(',')) - if layerquery.count() == 0: - logger.error('No layers matching specified query "%s"' % options.layers) - sys.exit(1) + layers = options.layers.split(',') + if 'bitbake' in layers: + update_bitbake = True + layers.remove('bitbake')So since this means 'bitbake' is a reserved name for layers we should add validation upon layer submission to ensure you can't create a layer with this name. I can take care of that if you like.
Good idea, I will do it later, or you can do it if you have time. // Robert -- _______________________________________________ yocto mailing list [email protected] https://lists.yoctoproject.org/listinfo/yocto
