Re: [yocto] [layerindex-web] Add SPDX Identifier

2019-10-28 Thread Paul Eggleton
Hi Yann On Wednesday, 23 October 2019 2:25:48 AM NZDT Yann CARDAILLAC wrote: > On Mon, Oct 21, 2019 at 9:37 PM Paul Eggleton > wrote: > > On Tuesday, 22 October 2019 4:15:03 AM NZDT Yann CARDAILLAC wrote: > > > https://bugzilla.yoctoproject.org/show_bug.cgi?id=13527 > > > > > > I'm beginning the

Re: [yocto] [layerindex-web] Add SPDX Identifier

2019-10-22 Thread Yann CARDAILLAC
On Mon, Oct 21, 2019 at 9:37 PM Paul Eggleton wrote: > On Tuesday, 22 October 2019 4:15:03 AM NZDT Yann CARDAILLAC wrote: > > https://bugzilla.yoctoproject.org/show_bug.cgi?id=13527 > > > > I'm beginning the work of updating the layerindex-web repo, this one is a > > bit tricky, how should I trea

Re: [yocto] [layerindex-web] Add SPDX Identifier

2019-10-21 Thread Paul Eggleton
Hi Yann Thanks for looking into this. On Tuesday, 22 October 2019 4:15:03 AM NZDT Yann CARDAILLAC wrote: > https://bugzilla.yoctoproject.org/show_bug.cgi?id=13527 > > I'm beginning the work of updating the layerindex-web repo, this one is a > bit tricky, how should I treat the files generated us

Re: [yocto] [layerindex-web] Enabled zeus, not working...

2019-10-14 Thread Mark Hatle
On 10/14/19 5:08 PM, Paul Eggleton wrote: > Hi Mark > > On Tuesday, 15 October 2019 10:21:29 AM NZDT Mark Hatle wrote: >> I added the zeus branch on the layers.openembedded.org today and it's not >> showing up and being indexed. Any idea why? > > The bitbake branch wasn't correctly specified

Re: [yocto] [layerindex-web] Enabled zeus, not working...

2019-10-14 Thread Paul Eggleton
Hi Mark On Tuesday, 15 October 2019 10:21:29 AM NZDT Mark Hatle wrote: > I added the zeus branch on the layers.openembedded.org today and it's not > showing up and being indexed. Any idea why? The bitbake branch wasn't correctly specified - it needed to be set to "1.44" (grabbed from https://wi

Re: [yocto] [layerindex-web] [PATCH 0/3] Some misc changes/fixes..

2019-10-13 Thread Paul Eggleton
Hi Mark On Sunday, 13 October 2019 2:56:30 PM NZDT Mark Hatle wrote: > A few misc changes/fixes. The first two are well tested. However, I suspect > the 3/3 may be incorrect and I've labeled it an RFC due to this. > > 1/3 - '.' wasn't allowed in branch names w/o an error. This turned out > to

Re: [yocto] [layerindex-web] Having problems instantiating a docker image

2019-10-09 Thread Paul Eggleton
Hi Mark On Wednesday, 9 October 2019 11:59:35 AM NZDT Mark Hatle wrote: > I've setup the layerindex in the past (without docker).. but I'm attempting to > follow the current instructions w/o much success. > > I'm trying to use: > > ./dockersetup.py -m 8080:80 --no-https > > It asks me for my em

Re: [yocto] [layerindex-web][PATCH] These changes will add the metadata for product revision and epoch if present in the bitbake recipe.

2019-09-30 Thread Paul Eggleton
Hi Pranay, On Tuesday, 1 October 2019 3:52:59 PM NZDT Pranay Mankad wrote: > Signed-off-by: Pranay Mankad > --- > .../migrations/0042_recipe_property_addition.py | 20 > > layerindex/models.py | 2 ++ > layerindex/update_layer.py

Re: [yocto] [layerindex-web] PR / PE in layer index

2019-09-23 Thread Paul Eggleton
Hi Pranay On Tuesday, 24 September 2019 10:00:46 AM NZST Pranay Mankad wrote: > Thank you for supporting the recipesExtended view over the last couple of > days. Here’s a small patch that includes values from two more variables from > the Bitbake cooker – PR and PE. > > I read through Paul’s expl

Re: [yocto] Layerindex - actions required for deleted repos?

2018-10-05 Thread Khem Raj
On Thu, Oct 4, 2018 at 12:34 AM Andreas Müller wrote: > > Hi, > > I am cleaning my github and want to remove some repos. One of them is > meta-gumstix-community - it is not maintained any more. > This layer is listed in layer-index. Will it be removed automatically > or are there steps I have to t

Re: [yocto] Layerindex - actions required for deleted repos?

2018-10-05 Thread akuster808
On 10/05/2018 07:02 AM, Mark Hatle wrote: > On 10/4/18 2:33 AM, Andreas Müller wrote: >> Hi, >> >> I am cleaning my github and want to remove some repos. One of them is >> meta-gumstix-community - it is not maintained any more. >> This layer is listed in layer-index. Will it be removed automatica

Re: [yocto] Layerindex - actions required for deleted repos?

2018-10-05 Thread Mark Hatle
On 10/4/18 2:33 AM, Andreas Müller wrote: > Hi, > > I am cleaning my github and want to remove some repos. One of them is > meta-gumstix-community - it is not maintained any more. > This layer is listed in layer-index. Will it be removed automatically > or are there steps I have to take? Just let

Re: [yocto] [layerindex-web][PATCH 000/242] Integrate and improve Recipe Reporting System (cover letter only)

2018-10-02 Thread Anibal Limon
Hi Paul, Good to know that is integrated now. Cheers, Anibal On Tue, 1 May 2018 at 15:09, Paul Eggleton wrote: > Hi Alex, > > On Tuesday, 1 May 2018 9:58:03 PM NZST Alexander Kanavin wrote: > > On 05/01/2018 05:23 AM, Paul Eggleton wrote: > > > The Recipe Reporting System (RRS - live instance

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-10-02 Thread Konrad Scherer
On 09/05/2018 08:04 AM, Paul Eggleton wrote: On Thursday, 30 August 2018 11:44:10 AM NZST Paul Eggleton wrote: On Wednesday, 29 August 2018 10:46:27 PM NZST Paul Eggleton wrote: I know Konrad promised to update the docker-compose setup to be closer to what we'd done recently with the docker set

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-09-05 Thread Paul Eggleton
On Thursday, 30 August 2018 11:44:10 AM NZST Paul Eggleton wrote: > On Wednesday, 29 August 2018 10:46:27 PM NZST Paul Eggleton wrote: > > I know Konrad promised to update the docker-compose setup to be closer > > to what we'd done recently with the docker setup, but I needed something > > urgently

Re: [yocto] [layerindex-web][PATCH 1/1] import_layer.py: add -t option for layer_type

2018-08-29 Thread Robert Yang
On 08/30/2018 02:20 PM, Paul Eggleton wrote: Hi Robert Sorry I meant to reply earlier. I'm not especially happy with moving the constants to utils.py. Is there any way we can avoid that? I will try to import models.py, but that may make things complicated. // Robert Thanks Paul On Thu

Re: [yocto] [layerindex-web][PATCH 1/1] import_layer.py: add -t option for layer_type

2018-08-29 Thread Paul Eggleton
Hi Robert Sorry I meant to reply earlier. I'm not especially happy with moving the constants to utils.py. Is there any way we can avoid that? Thanks Paul On Thursday, 30 August 2018 2:05:11 PM NZST Robert Yang wrote: > Hi Paul, > > Do you have any comments on this, please ? > > // Robert >

Re: [yocto] [layerindex-web][PATCH 1/1] import_layer.py: add -t option for layer_type

2018-08-29 Thread Robert Yang
Hi Paul, Do you have any comments on this, please ? // Robert On 07/31/2018 03:57 PM, Robert Yang wrote: Hi Paul, On 07/24/2018 04:33 PM, Paul Eggleton wrote: Hi Robert, On Monday, 23 July 2018 12:30:01 PM CEST Robert Yang wrote: Now the logic is: Use options.layer_type if specified, guess

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-08-29 Thread Paul Eggleton
On Wednesday, 29 August 2018 10:46:27 PM NZST Paul Eggleton wrote: > Hi folks > > I know Konrad promised to update the docker-compose setup to be closer > to what we'd done recently with the docker setup, but I needed something > urgently and wanted to learn how to use docker-compose anyway, so I'

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-08-29 Thread Paul Eggleton
Hi folks I know Konrad promised to update the docker-compose setup to be closer to what we'd done recently with the docker setup, but I needed something urgently and wanted to learn how to use docker-compose anyway, so I've put this together: http://git.yoctoproject.org/cgit/cgit.cgi/layerinde

Re: [yocto] [layerindex-web][PATCH 1/1] import_layer.py: add -t option for layer_type

2018-07-31 Thread Robert Yang
Hi Paul, On 07/24/2018 04:33 PM, Paul Eggleton wrote: Hi Robert, On Monday, 23 July 2018 12:30:01 PM CEST Robert Yang wrote: Now the logic is: Use options.layer_type if specified, guess if not, default to 'M'. Note choices=['A', 'B', 'S', 'D', 'M', ''], the '' is for default='', we can't use

Re: [yocto] [layerindex-web][PATCH 1/1] import_layer.py: add -t option for layer_type

2018-07-26 Thread Paul Eggleton
Hi Robert, On Monday, 23 July 2018 12:30:01 PM CEST Robert Yang wrote: > Now the logic is: > Use options.layer_type if specified, guess if not, default to 'M'. > > Note choices=['A', 'B', 'S', 'D', 'M', ''], the '' is for default='', we can't > use default='M' here, otherwise we don't know whethe

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-07-17 Thread Konrad Scherer
On 07/17/2018 06:16 AM, Paul Eggleton wrote: On Monday, 16 July 2018 10:16:55 PM CEST Paul Eggleton wrote: On Monday, 16 July 2018 8:24:53 PM CEST Konrad Scherer wrote: I recently noticed that if gcc is removed in the 'apt-get purge' step, the bitbake sanity checker will fail due to missing 'gc

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-07-17 Thread Paul Eggleton
On Monday, 16 July 2018 10:16:55 PM CEST Paul Eggleton wrote: > On Monday, 16 July 2018 8:24:53 PM CEST Konrad Scherer wrote: > > I recently noticed that if gcc is removed in the 'apt-get purge' step, > > the bitbake sanity checker will fail due to missing 'gcc' when doing a > > layer update. >

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-07-16 Thread Paul Eggleton
On Monday, 16 July 2018 8:24:53 PM CEST Konrad Scherer wrote: > On 07/13/2018 03:29 AM, Paul Eggleton wrote: > > On Thursday, 12 July 2018 10:16:12 PM CEST you wrote: > >> I just noticed that the image doesn't contain git. Was this > >> deliberate or an oversight? > > > > Ah, no, it wasn't deliber

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-07-16 Thread Konrad Scherer
On 07/13/2018 03:29 AM, Paul Eggleton wrote: On Thursday, 12 July 2018 10:16:12 PM CEST you wrote: I just noticed that the image doesn't contain git. Was this deliberate or an oversight? Ah, no, it wasn't deliberate - perhaps it was there earlier with buildpack- deps? It doesn't appear that we

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-07-13 Thread Paul Eggleton
On Thursday, 12 July 2018 10:16:12 PM CEST you wrote: > I just noticed that the image doesn't contain git. Was this > deliberate or an oversight? Ah, no, it wasn't deliberate - perhaps it was there earlier with buildpack- deps? It doesn't appear that we ever explicitly had it in there. I've added

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-07-12 Thread Konrad Scherer
On 07/12/2018 02:34 AM, Paul Eggleton wrote: Hi Konrad On Wednesday, 11 July 2018 7:03:28 PM CEST Konrad Scherer wrote: I have attached a patch to reduce the layerindex-web image size further. Docker image layers are like git objects and RUN commands can only hide files, not remove them once th

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-07-11 Thread Paul Eggleton
Hi Konrad On Wednesday, 11 July 2018 7:03:28 PM CEST Konrad Scherer wrote: > I have attached a patch to reduce the layerindex-web image size further. > Docker image layers are like git objects and RUN commands can only hide > files, not remove them once they have been committed. The pip upgrade

Re: [yocto] [layerindex-web] RFC: layer index docker fixes

2018-07-11 Thread Konrad Scherer
On 07/10/2018 12:17 PM, Paul Eggleton wrote: Hi folks I've been trying to get the OE layer index example docker setup in shape, based upon improvements from myself, Michael and Konrad. For the moment I've taken most of what Michael and I did (with some fixes and tweaks, and broken up into smalle

Re: [yocto] [layerindex-web][PATCH 1/6] Add docker-compose file to create full layerindex stack of MariaDB, RabbitMQ and Nginx

2018-07-10 Thread Paul Eggleton
Hi Konrad On Tuesday, 26 June 2018 7:41:30 PM CEST Konrad Scherer wrote: > Lots of new features added: > > - Layerindex runs as unprivileged user inside container > > - Celery worker is started before gunicorn > > - Entrypoint script supports changing RabbitMQ location > > - Entrypoint script

Re: [yocto] [layerindex-web][PATCH 4/4] update.py: check whether branch existed when nocheckout

2018-07-09 Thread Paul Eggleton
On Monday, 9 July 2018 10:45:58 AM CEST Robert Yang wrote: > On 07/09/2018 04:36 PM, Paul Eggleton wrote: > > On Monday, 9 July 2018 10:34:07 AM CEST Paul Eggleton wrote: > >> On Monday, 9 July 2018 6:11:30 AM CEST Robert Yang wrote: > >>> Fixed: > >>> Assume there is no master branch in hello laye

Re: [yocto] [layerindex-web][PATCH 4/4] update.py: check whether branch existed when nocheckout

2018-07-09 Thread Robert Yang
On 07/09/2018 04:36 PM, Paul Eggleton wrote: On Monday, 9 July 2018 10:34:07 AM CEST Paul Eggleton wrote: Hi Robert, On Monday, 9 July 2018 6:11:30 AM CEST Robert Yang wrote: Fixed: Assume there is no master branch in hello layer: $ update.py -l hello -b master INFO: Skipping update of laye

Re: [yocto] [layerindex-web][PATCH 4/4] update.py: check whether branch existed when nocheckout

2018-07-09 Thread Robert Yang
Hi Paul, On 07/09/2018 04:34 PM, Paul Eggleton wrote: Hi Robert, On Monday, 9 July 2018 6:11:30 AM CEST Robert Yang wrote: Fixed: Assume there is no master branch in hello layer: $ update.py -l hello -b master INFO: Skipping update of layer hello - branch master doesn't exist This is correct

Re: [yocto] [layerindex-web][PATCH 4/4] update.py: check whether branch existed when nocheckout

2018-07-09 Thread Paul Eggleton
On Monday, 9 July 2018 10:34:07 AM CEST Paul Eggleton wrote: > Hi Robert, > > On Monday, 9 July 2018 6:11:30 AM CEST Robert Yang wrote: > > Fixed: > > Assume there is no master branch in hello layer: > > $ update.py -l hello -b master > > INFO: Skipping update of layer hello - branch master doesn'

Re: [yocto] [layerindex-web][PATCH 4/4] update.py: check whether branch existed when nocheckout

2018-07-09 Thread Paul Eggleton
Hi Robert, On Monday, 9 July 2018 6:11:30 AM CEST Robert Yang wrote: > Fixed: > Assume there is no master branch in hello layer: > $ update.py -l hello -b master > INFO: Skipping update of layer hello - branch master doesn't exist > > This is correct since hello layer doesn't have master branch,

Re: [yocto] [layerindex-web][PATCH 5/7] update: ignore recommends when ordering layers

2018-07-08 Thread Robert Yang
Hi Paul, I've sent the patches to mailing list: [yocto] [layerindex-web][PATCH 0/4] update.py: several fixes The one for recommends is: update.py: add layers when RECOMMENDS isn't satisfied Now you can drop [PATCH 5/7], others are still needed. // Robert On 07/06/2018 02:49 PM, Robert Yang

Re: [yocto] [layerindex-web][PATCH 5/7] update: ignore recommends when ordering layers

2018-07-05 Thread Robert Yang
On 07/06/2018 01:28 PM, Paul Eggleton wrote: Hi Robert On Wednesday, 4 July 2018 7:52:05 PM NZST you wrote: I'm sorry to say that I met layerindex' loaddata problems yesterday and today, I still didn't find the root cause. Have you tried dumpdata and loaddata recently, please ? What I did w

Re: [yocto] [layerindex-web][PATCH 5/7] update: ignore recommends when ordering layers

2018-07-05 Thread Paul Eggleton
Hi Robert On Wednesday, 4 July 2018 7:52:05 PM NZST you wrote: > I'm sorry to say that I met layerindex' loaddata problems yesterday and > today, > I still didn't find the root cause. Have you tried dumpdata and loaddata > recently, please ? > > What I did was: > > $ python3 manage.py dumpdata -

Re: [yocto] [layerindex-web][PATCH 5/7] update: ignore recommends when ordering layers

2018-07-04 Thread Robert Yang
Hi Paul, I'm sorry to say that I met layerindex' loaddata problems yesterday and today, I still didn't find the root cause. Have you tried dumpdata and loaddata recently, please ? What I did was: $ python3 manage.py dumpdata --settings settings --exclude=contenttypes --exclude=auth.Permission

Re: [yocto] [layerindex-web][PATCH 5/7] update: ignore recommends when ordering layers

2018-07-02 Thread Robert Yang
On 07/03/2018 10:58 AM, Paul Eggleton wrote: Hi Robert On Tuesday, 3 July 2018 2:45:11 PM NZST Robert Yang wrote: Thanks for let me know this, this patch might be incorrect, suppose we have two layers: core and hello: 1) LAYERRECOMMENDS_core = "hello" 2) $ update.py -l hello,core Then core

Re: [yocto] [layerindex-web][PATCH 5/7] update: ignore recommends when ordering layers

2018-07-02 Thread Paul Eggleton
Hi Robert On Tuesday, 3 July 2018 2:45:11 PM NZST Robert Yang wrote: > Thanks for let me know this, this patch might be incorrect, suppose we have > two > layers: core and hello: > > 1) LAYERRECOMMENDS_core = "hello" > 2) $ update.py -l hello,core > > Then core maybe added before hello layer si

Re: [yocto] [layerindex-web][PATCH 5/7] update: ignore recommends when ordering layers

2018-07-02 Thread Robert Yang
Hi Paul, Thanks for let me know this, this patch might be incorrect, suppose we have two layers: core and hello: 1) LAYERRECOMMENDS_core = "hello" 2) $ update.py -l hello,core Then core maybe added before hello layer since it ignores recs on hello, and if hello is a new layer, it would not be i

Re: [yocto] [layerindex-web][PATCH 2/2] rrs_upstream_history.py: fix set_regexes function

2018-06-05 Thread Paul Eggleton
Hi Yi On Friday, 1 June 2018 7:00:11 PM NZST Yi Zhao wrote: > REGEX, REGEX_URI and GITTAGREGEX are replaced by UPSTREAM_CHECK_REGEX, > UPSTREAM_CHECK_URI and UPSTREAM_CHECK_GITTAGREGEX Merged both of these, thanks. Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre -- __

Re: [yocto] [layerindex-web][PATCH 2/2] Add CSV export for layer recipes

2018-05-07 Thread Mark Hatle
On 5/6/18 10:35 PM, Paul Eggleton wrote: > Add the ability to export the recipe listing for a layer to a CSV file > for importing into external tools. At the moment we include name, > version and license, but there is a parameter that lets you specify the > fields to include in the URL if desired.

Re: [yocto] [layerindex-web][PATCH 000/242] Integrate and improve Recipe Reporting System (cover letter only)

2018-05-01 Thread Paul Eggleton
Hi Alex, On Tuesday, 1 May 2018 9:58:03 PM NZST Alexander Kanavin wrote: > On 05/01/2018 05:23 AM, Paul Eggleton wrote: > > The Recipe Reporting System (RRS - live instance at > > http://recipes.yoctoproject.org) was originally developed as a fork of > > the layer index (live instance at http://la

Re: [yocto] [layerindex-web][PATCH 000/242] Integrate and improve Recipe Reporting System (cover letter only)

2018-05-01 Thread Alexander Kanavin
On 05/01/2018 05:23 AM, Paul Eggleton wrote: The Recipe Reporting System (RRS - live instance at http://recipes.yoctoproject.org) was originally developed as a fork of the layer index (live instance at http://layers.openembedded.org) - this set of changes rebases it on top of master and makes the

Re: [yocto] [layerindex-web][PATCH 2/2] requirements.txt: use the most recent Django 1.8 version

2018-03-27 Thread Paul Eggleton
Hi Armin, On Tuesday, 27 March 2018 4:14:25 AM NZDT akuster808 wrote: > Have you seen this ? > > https://www.djangoproject.com/download/#supported-versions > > 1.8 will EOL this April Ah yes, I was aware this was looming but had forgotten how close it was. Thanks for the reminder, I guess I ha

Re: [yocto] [layerindex-web][PATCH 2/2] requirements.txt: use the most recent Django 1.8 version

2018-03-26 Thread akuster808
Paul, Have you seen this ? https://www.djangoproject.com/download/#supported-versions 1.8 will EOL this April - armin On 03/25/2018 12:31 PM, Paul Eggleton wrote: > Use <1.9 to always get the most recent 1.8 point release. > > Signed-off-by: Paul Eggleton > --- > requirements.txt | 2 +- >

Re: [yocto] [layerindex-web][PATCH 4/5] update_layer.py: use it as a module

2018-02-04 Thread Robert Yang
Hi Paul, Thank you very much, and please see my comments inline. On 02/05/2018 07:27 AM, Paul Eggleton wrote: Hi Robert, On Wednesday, 3 January 2018 6:42:25 PM NZDT Robert Yang wrote: It had been split to a separate tool and use subprocess to run it to avoid tinfoil issues (e.g., can't be sh

Re: [yocto] [layerindex-web] Updating layers?

2018-02-04 Thread Paul Eggleton
Hi Steve On Friday, 12 January 2018 4:52:23 AM NZDT Steve Bedford wrote: > Is the layerindex not running the update script? I see end of September as > last updates, and a bunch of branches in our layer (meta-timesys) were > never picked up. FYI we've fixed this as of a few hours after your emai

Re: [yocto] [layerindex-web][PATCH 0/5] update.py: performance improve and 2 fixes

2018-02-04 Thread Paul Eggleton
Hi Robert On Thursday, 1 February 2018 5:59:42 PM NZDT Robert Yang wrote: > On 01/03/2018 01:42 PM, Robert Yang wrote: > > This email contains two parts: > > - The first 2 are bug fixes > > - The other 3 are performance improvements, I have 124 layers, > >the "update.py -b " needs about 9m20s

Re: [yocto] [layerindex-web][PATCH 4/5] update_layer.py: use it as a module

2018-02-04 Thread Paul Eggleton
Hi Robert, On Wednesday, 3 January 2018 6:42:25 PM NZDT Robert Yang wrote: > It had been split to a separate tool and use subprocess to run it to avoid > tinfoil issues (e.g., can't be shutdown correctly in old version), but the > utils.setup_django() cost a lot of time which made it a little slow

Re: [yocto] [layerindex-web][PATCH 0/5] update.py: performance improve and 2 fixes

2018-01-31 Thread Robert Yang
Hi Paul, Any comments, please ? // Robert On 01/03/2018 01:42 PM, Robert Yang wrote: Hi Paul, This email contains two parts: - The first 2 are bug fixes - The other 3 are performance improvements, I have 124 layers, the "update.py -b " needs about 9m20s to finish the running when everythi

Re: [yocto] [layerindex-web][PATCH 0/5] update.py: performance improve and 2 fixes

2018-01-15 Thread Robert Yang
Hi Paul, Sorry, this is the cover letter, I should reply to this one just now. // Robert On 01/03/2018 01:42 PM, Robert Yang wrote: Hi Paul, This email contains two parts: - The first 2 are bug fixes - The other 3 are performance improvements, I have 124 layers, the "update.py -b " needs a

Re: [yocto] [layerindex-web][PATCH 1/5] utils.py: fix remove obsolete dependencies

2018-01-15 Thread Robert Yang
Hi Paul, Do you have any comments, please ? // Robert On 01/03/2018 01:42 PM, Robert Yang wrote: The obsolete dependency is the one which is in database but not in conf/layer.conf anymore. The old code had a problem for newly created layerbranch, the new layerbranch has no dependencies, so no

Re: [yocto] [layerindex-web][patch v3 1/1] recipes.html: Require keyword for recipe search

2017-11-08 Thread Mark Hatle
On 11/8/17 1:08 PM, Paul Eggleton wrote: > On Thursday, 9 November 2017 4:42:50 AM NZDT Mark Hatle wrote: >> On 11/7/17 8:43 PM, Paul Eggleton wrote: >>> On Wednesday, 8 November 2017 11:47:49 AM NZDT Mark Hatle wrote: On 11/7/17 4:31 PM, Amanda Brindle wrote: > Use JavaScript to check if

Re: [yocto] [layerindex-web][patch v3 1/1] recipes.html: Require keyword for recipe search

2017-11-08 Thread Paul Eggleton
On Thursday, 9 November 2017 4:42:50 AM NZDT Mark Hatle wrote: > On 11/7/17 8:43 PM, Paul Eggleton wrote: > > On Wednesday, 8 November 2017 11:47:49 AM NZDT Mark Hatle wrote: > >> On 11/7/17 4:31 PM, Amanda Brindle wrote: > >>> Use JavaScript to check if the search box for recipe search is > >>> em

Re: [yocto] [layerindex-web][patch v3 1/1] recipes.html: Require keyword for recipe search

2017-11-08 Thread Mark Hatle
On 11/7/17 8:43 PM, Paul Eggleton wrote: > On Wednesday, 8 November 2017 11:47:49 AM NZDT Mark Hatle wrote: >> On 11/7/17 4:31 PM, Amanda Brindle wrote: >>> Use JavaScript to check if the search box for recipe search is >>> empty before querying the database. This will prevent the "502 >>> Bad Gate

Re: [yocto] [layerindex-web][patch v3 1/1] recipes.html: Require keyword for recipe search

2017-11-07 Thread Paul Eggleton
On Wednesday, 8 November 2017 11:47:49 AM NZDT Mark Hatle wrote: > On 11/7/17 4:31 PM, Amanda Brindle wrote: > > Use JavaScript to check if the search box for recipe search is > > empty before querying the database. This will prevent the "502 > > Bad Gateway" error that occurs when the query takes

Re: [yocto] [layerindex-web][patch v3 1/1] recipes.html: Require keyword for recipe search

2017-11-07 Thread Mark Hatle
On 11/7/17 4:31 PM, Amanda Brindle wrote: > Use JavaScript to check if the search box for recipe search is > empty before querying the database. This will prevent the "502 > Bad Gateway" error that occurs when the query takes too long due > to the large list of recipes. Since there are so many reci

Re: [yocto] [layerindex-web][patch v2 1/1] recipes.html: Require keyword for recipe search

2017-11-06 Thread Paul Eggleton
On Tuesday, 7 November 2017 8:27:03 PM NZDT Paul Eggleton wrote: > I'm not sure is a valid HTML tag. Looking at the bootstrap docs I > think we should actually be using its error state and help text > functionality (see "Validation states" on http://getbootstrap.com/2.3.2/ > base-css.html#forms ).

Re: [yocto] [layerindex-web][patch v2 1/1] recipes.html: Require keyword for recipe search

2017-11-06 Thread Paul Eggleton
Hi Amanda, A few comments below. On Tuesday, 7 November 2017 1:21:23 PM NZDT Amanda Brindle wrote: > Use JavaScript to check if the search box for recipe search is > empty before querying the database. This will prevent the "502 > Bad Gateway" error that occurs when the query takes too long due >

Re: [yocto] [layerindex-web][patch v3 1/1] Send email notification on publication

2017-10-30 Thread Paul Eggleton
On Wednesday, 25 October 2017 10:49:42 AM NZDT Amanda Brindle wrote: > When publishing a layer, send an email notification to all of that > layer's maintainers. > > Fixes [YOCTO #11208] > > Signed-off-by: Amanda Brindle > --- > layerindex/views.py | 34 +

Re: [yocto] [layerindex-web][patch v3 1/1] Send email notification on publication

2017-10-24 Thread Paul Eggleton
Hi Amanda, On Wednesday, 25 October 2017 10:49:42 AM NZDT Amanda Brindle wrote: > When publishing a layer, send an email notification to all of that > layer's maintainers. > > Fixes [YOCTO #11208] This change looks good to me - I'll wait a day or so for other reviews and merge after that. Than

Re: [yocto] [layerindex][PATCH] Indicate if layer has YP Compatible certification

2017-09-26 Thread Paul Eggleton
Hi Amanda, On Tuesday, 26 September 2017 12:41:32 PM NZDT Amanda Brindle wrote: > Allow admin to create Yocto Project versions with the version name, > description, an icon link that represents that version, and a link that > contains more information about the version. > > Admins who have the "s

Re: [yocto] [layerindex][PATCH] Indicate if layer has YP Compatible certification

2017-09-25 Thread Paul Eggleton
On Tuesday, 26 September 2017 2:23:15 PM NZDT akuster wrote: > On 09/25/2017 04:41 PM, Amanda Brindle wrote: > > Allow admin to create Yocto Project versions with the version name, > > description, an icon link that represents that version, and a link that > > contains more information about the ve

Re: [yocto] [layerindex][PATCH] Indicate if layer has YP Compatible certification

2017-09-25 Thread akuster
On 09/25/2017 04:41 PM, Amanda Brindle wrote: > Allow admin to create Yocto Project versions with the version name, > description, an icon link that represents that version, and a link that > contains more information about the version. Who are the Admins ? - armin > > Admins who have the "set_

Re: [yocto] [layerindex-web][PATCH v4] Asynchronous email notifications, task execution

2017-08-31 Thread Paul Eggleton
Hi Diana, On Friday, 25 August 2017 5:20:30 PM NZST Diana Thayer wrote: > This patch adds asynchronous task execution using a Celery backend > and RabbitMQ task queue, so that the layer submission process to > proceed even in the event that sending the notification email fails, > and establishing

Re: [yocto] [layerindex-web][PATCH v2] Asynchronous email notifications, task execution

2017-08-24 Thread Paul Eggleton
Hi Diana, On Thursday, 24 August 2017 12:58:38 PM NZST Diana Thayer wrote: > This patch adds asynchronous task execution using a Celery backend > and RabbitMQ task queue. > > It updates the README to reflect the installation and configuration > of a basic RabbitMQ setup, adds a 'tasks.py' file to

Re: [yocto] [layerindex-web][PATCH] layerindex-web : adding asynchronous task execution

2017-08-22 Thread Paul Eggleton
On Wednesday, 23 August 2017 12:06:09 PM NZST Paul Eggleton wrote: > Hi Diana > > I did finally manage to get to it today - sorry about the delay. I've > sorted out the settings issue and dropped the related item from the todo list. Oops, I neglected to mention I pushed these minor changes to the

Re: [yocto] [layerindex-web][PATCH] layerindex-web : adding asynchronous task execution

2017-08-22 Thread Paul Eggleton
Hi Diana I did finally manage to get to it today - sorry about the delay. I've sorted out the settings issue and dropped the related item from the todo list. Assuming that didn't break anything on your end, could you please squash all of these into one commit with you as the author and an appropri

Re: [yocto] [layerindex-web][PATCH] Redirect user to correct url after editing a layer

2017-08-17 Thread Paul Eggleton
Hi Amanda, Thanks - a few issues noted below. On Tuesday, 15 August 2017 10:50:56 AM NZST Amanda Brindle wrote: > Before, if a user edited a layer's name, they would be redirected > to a url utilizing the old name and then receive a 404 Page not > found error. Now, the url utilizes the new name.

Re: [yocto] [layerindex-web][PATCH] layerindex-web : adding asynchronous task execution

2017-08-11 Thread Diana Thayer
Hello! Did you get a chance to look at the PR ? On Wed, Jul 26, 2017 at 6:02 PM, Diana Thayer wrote: > Great! Thanks for testing it out and identifying that issue. > > I've put together a PR on GitHub >

Re: [yocto] [layerindex-web][PATCH] layerindex-web : adding asynchronous task execution

2017-07-26 Thread Diana Thayer
Great! Thanks for testing it out and identifying that issue. I've put together a PR on GitHub that includes some changes that appear to fix the issue you're describing. It also updates the readme to document how to configure the connection to the

Re: [yocto] [layerindex-web][PATCH] layerindex-web : adding asynchronous task execution

2017-07-24 Thread Paul Eggleton
On Friday, 21 July 2017 4:15:26 PM CEST Paul Eggleton wrote: > On Tuesday, 27 June 2017 1:59:23 AM CEST Diana Thayer wrote: > > This is a work-in-progress solution to issue 11197 within layerindex-web: > > adding asynchronous task execution. > > I used RabbitMQ and Celery. I'm still working with P

Re: [yocto] [layerindex-web][PATCH] layerindex-web : adding asynchronous task execution

2017-07-21 Thread Paul Eggleton
Hi Diana, Sorry it's taken me a ridiculous amount of time to get back to you on this! On Tuesday, 27 June 2017 1:59:23 AM CEST Diana Thayer wrote: > This is a work-in-progress solution to issue 11197 within layerindex-web: > adding asynchronous task execution. > I used RabbitMQ and Celery. I'm s

Re: [yocto] [layerindex-web][PATCH V5 00/10] Several fixes for layerindex

2017-07-20 Thread Robert Yang
Hi Paul, Thank you very much. // Robert On 07/21/2017 02:19 PM, Paul Eggleton wrote: Hi Robert / all, This patchset finally got merged. I tweaked some of the commit messages, messages and comments but otherwise they went in unchanged. Thanks for sorting out these issues! Cheers, Paul On Fri

Re: [yocto] [layerindex-web][PATCH V5 00/10] Several fixes for layerindex

2017-07-20 Thread Paul Eggleton
Hi Robert / all, This patchset finally got merged. I tweaked some of the commit messages, messages and comments but otherwise they went in unchanged. Thanks for sorting out these issues! Cheers, Paul On Friday, 23 June 2017 9:44:59 AM CEST Robert Yang wrote: > * V5 > - Fix Paul's comments >

Re: [yocto] [layerindex-web][PATCH v2] layerindex/views: support querying by layer name

2017-06-15 Thread Paul Eggleton
Hi Jose, Sorry for the delay. The patch looks good except it displays two errors if there's no layer after the layer: prefix. I've fixed that myself and merged to layerindex-web master. Thanks, Paul On Thursday, 15 June 2017 3:19:48 PM CEST Jose Lamego wrote: > ping > > On 05/31/2017 03:47 PM

Re: [yocto] [layerindex-web][PATCH v2] layerindex/views: support querying by layer name

2017-06-15 Thread Jose Lamego
ping On 05/31/2017 03:47 PM, Jose Lamego wrote: > This change supports querying recipes that belong to a specific layer > by using the prefix "layer:" + the desired layer name, for example: > "layer: openembedded-core" and this string can be used by itself or > combined with other supported option

Re: [yocto] [layerindex-web][PATCH] views: support querying by layer name

2017-05-24 Thread Paul Eggleton
Hi Jose, Sorry for the extreme delay reviewing this. On Tuesday, 4 April 2017 7:31:29 AM NZST Jose Lamego wrote: > This change supports querying recipes that belong to a specific layer > by using the prefix "layer:" + the desired layer name, for example: > "layer: openembedded-core" and this stri

Re: [yocto] [layerindex-web][PATCH 2/2] views.py: Consider layer name in recipe search

2017-04-03 Thread Jose Lamego
On 03/13/2017 08:01 PM, Paul Eggleton wrote: > Hi Jose, > > Sorry for the delay in responding to this. > > I'd prefer if this were implemented like the recently added "inherits:" where > we have a special prefix e.g. "layer:" in order to search for things in a > particular layer. Then it's mu

Re: [yocto] [layerindex-web][PATCH 1/2] views.py: Redirect to recipe details for single results

2017-03-13 Thread Paul Eggleton
This should be using a reverse() call instead of a hardcoded URL path suffix. However, since that was a trivial fix I've done that and merged it - it'll appear in the layer index once we do the next upgrade (fairly soon). Cheers, Paul On Tuesday, 14 March 2017 9:35:16 AM NZDT Jose Lamego wrote:

Re: [yocto] [layerindex-web][PATCH 2/2] views.py: Consider layer name in recipe search

2017-03-13 Thread Paul Eggleton
Hi Jose, Sorry for the delay in responding to this. I'd prefer if this were implemented like the recently added "inherits:" where we have a special prefix e.g. "layer:" in order to search for things in a particular layer. Then it's much clearer as to what's going on. Cheers, Paul On Tuesday,

Re: [yocto] [layerindex-web][PATCH 2/2] views.py: Consider layer name in recipe search

2017-03-13 Thread Jose Lamego
Ping. On 01/18/2017 04:19 PM, Jose Lamego wrote: > From: Alex Franco > > Recipe search cannot be narrowed to a specific layer, producing > unnecessary results. > > This change allows including a layers name when searching for recipes to > narrow the yielded results by adding a layer name to the

Re: [yocto] [layerindex-web][PATCH 1/2] views.py: Redirect to recipe details for single results

2017-03-13 Thread Jose Lamego
Ping. On 01/18/2017 04:19 PM, Jose Lamego wrote: > From: Alex Franco > > When a recipe search returns only one result, clicking the single > result at the results view is still required to view further > recipe detail. > > This change automatically redirects to the recipe details view > when th

Re: [yocto] [layerindex-web] issue with webinterface

2017-01-24 Thread Paul Eggleton
he Nachricht --- > Von: Paul Eggleton > Datum: 22.01.2017 20:23:43 > An: Peter Tulpen > Betreff: Re: [yocto] [layerindex-web] issue with webinterface > > > Hi Peter, > > > > On Thu, 19 Jan 2017 18:05:12 Peter Tulpen wrote: > > > Hello, I installed the

Re: [yocto] [layerindex-web][PATCH 2/2] views.py: Consider layer name in recipe search

2017-01-24 Thread Jose Lamego
ping On 01/18/2017 04:31 PM, Jose Lamego wrote: > From: Alex Franco > > Recipe search cannot be narrowed to a specific layer, producing > unnecessary results. > > This change allows including a layers name when searching for recipes to > narrow the yielded results by adding a layer name to the

Re: [yocto] [layerindex-web][PATCH 1/2] views.py: Redirect to recipe details for single results

2017-01-24 Thread Jose Lamego
ping On 01/18/2017 04:31 PM, Jose Lamego wrote: > From: Alex Franco > > When a recipe search returns only one result, clicking the single > result at the results view is still required to view further > recipe detail. > > This change automatically redirects to the recipe details view > when the

Re: [yocto] [layerindex-web] issue with webinterface

2017-01-24 Thread Peter Tulpen
um: 22.01.2017 20:23:43 An: Peter Tulpen Betreff: Re: [yocto] [layerindex-web] issue with webinterface > Hi Peter, > > On Thu, 19 Jan 2017 18:05:12 Peter Tulpen wrote: > > Hello, I installed the webinterface locally and configured it with several > > > branches But when I

Re: [yocto] [layerindex-web] issue with webinterface

2017-01-22 Thread Paul Eggleton
Hi Peter, On Thu, 19 Jan 2017 18:05:12 Peter Tulpen wrote: > Hello, I installed the webinterface locally and configured it with several > branches But when I want to browse it, I can only choose master: > > https://postimg.org/image/dq1vj6v7h/ > > When I browse to http://MYSERVER/layerindex/bran

Re: [yocto] [layerindex-web] issue with webinterface

2017-01-20 Thread Mark Hatle
On 1/19/17 11:05 AM, Peter Tulpen wrote: > Hello, I installed the webinterface locally and configured it with several > branches > But when I want to browse it, I can only choose master: > > https://postimg.org/image/dq1vj6v7h/ I've been using a local instance of the layer index for a while now.

Re: [yocto] [layerindex-web][PATCH v2 00/12] Add Distro, dependency and recommends detection, and import_project

2016-10-11 Thread Mark Hatle
On 10/10/16 10:24 PM, Paul Eggleton wrote: > On Tue, 11 Oct 2016 09:19:15 Paul Eggleton wrote: >> On Mon, 10 Oct 2016 12:47:11 Mark Hatle wrote: >>> On 10/10/16 12:37 PM, Paul Eggleton wrote: On Mon, 10 Oct 2016 04:48:41 Hatle, Mark wrote: >> On Oct 10, 2016, at 2:54 AM, Paul Eggleton

Re: [yocto] [layerindex-web][PATCH v2 09/12] layerindex: Add collection and version to layerbranch

2016-10-11 Thread Mark Hatle
On 10/10/16 3:32 AM, Paul Eggleton wrote: > On Fri, 07 Oct 2016 11:57:18 Liam R. Howlett wrote: >> Collection and version will be pulled from the layer.conf if it exists >> and dependencies will be resolved by first checking for layers with the >> dependency name and then checking for collections.

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 00/12] Add Distro, dependency and recommends detection, and import_project

2016-10-10 Thread Paul Eggleton
On Tue, 11 Oct 2016 09:19:15 Paul Eggleton wrote: > On Mon, 10 Oct 2016 12:47:11 Mark Hatle wrote: > > On 10/10/16 12:37 PM, Paul Eggleton wrote: > > > On Mon, 10 Oct 2016 04:48:41 Hatle, Mark wrote: > > >>> On Oct 10, 2016, at 2:54 AM, Paul Eggleton > > >>> > > >>> > > >>> wrote: > > On Fri

Re: [yocto] [layerindex-web][PATCH v2 00/12] Add Distro, dependency and recommends detection, and import_project

2016-10-10 Thread Paul Eggleton
On Mon, 10 Oct 2016 12:47:11 Mark Hatle wrote: > On 10/10/16 12:37 PM, Paul Eggleton wrote: > > On Mon, 10 Oct 2016 04:48:41 Hatle, Mark wrote: > >>> On Oct 10, 2016, at 2:54 AM, Paul Eggleton > >>> > >>> > >>> wrote: > On Fri, 07 Oct 2016 13:20:50 Mark Hatle wrote: > FYI, I have made s

Re: [yocto] [layerindex-web][PATCH v2 00/12] Add Distro, dependency and recommends detection, and import_project

2016-10-10 Thread Mark Hatle
On 10/10/16 12:37 PM, Paul Eggleton wrote: > On Mon, 10 Oct 2016 04:48:41 Hatle, Mark wrote: >>> On Oct 10, 2016, at 2:54 AM, Paul Eggleton >>> wrote: On Fri, 07 Oct 2016 13:20:50 Mark Hatle wrote: FYI, I have made sure these are re-based on top of paule/django18 and pushed to:

Re: [yocto] [layerindex-web][PATCH v2 00/12] Add Distro, dependency and recommends detection, and import_project

2016-10-10 Thread Paul Eggleton
On Mon, 10 Oct 2016 04:48:41 Hatle, Mark wrote: > > On Oct 10, 2016, at 2:54 AM, Paul Eggleton > > wrote: > >> On Fri, 07 Oct 2016 13:20:50 Mark Hatle wrote: > >> FYI, I have made sure these are re-based on top of paule/django18 and > >> pushed to: > >> > >> git://git.yoctoproject.org/layerindex-

  1   2   >