Re: [yocto] [layerindex-web][PATCH v2 05/12] layerindex: Add distro to web interface and model.

2016-10-11 Thread Mark Hatle
On 10/10/16 3:20 AM, Paul Eggleton wrote: > This looks good, just a few notes about this bit: > > On Fri, 07 Oct 2016 11:57:14 Liam R. Howlett wrote: >> +def update_distro_conf_file(path, distro): >> +logger.debug('Updating distro %s' % path) >> +desc = "" >> +with open(path, 'r') as f

Re: [yocto] [layerindex-web][PATCH v2 05/12] layerindex: Add distro to web interface and model.

2016-10-09 Thread Paul Eggleton
This looks good, just a few notes about this bit: On Fri, 07 Oct 2016 11:57:14 Liam R. Howlett wrote: > +def update_distro_conf_file(path, distro): > +logger.debug('Updating distro %s' % path) > +desc = "" > +with open(path, 'r') as f: > +for line in f: > +if line.s

[yocto] [layerindex-web][PATCH v2 05/12] layerindex: Add distro to web interface and model.

2016-10-07 Thread Liam R. Howlett
Add the distros to the index. This looks a lot like the machines and allows users to search for a particular distro. Signed-off-by: Liam R. Howlett --- layerindex/admin.py| 11 ++ layerindex/models.py | 14 +++ layerindex/recipeparse.py | 5 +++ l