[OE-core] [PATCH 1/2] devtool: standard: update-recipe/finish: fix update localfile in another layer

2024-04-15 Thread Julien Stephan
object has no attribute 'get' This was introduced when adding support for git submodules. No selftest case exists to catch this, so a selftest will be added in another commit. Signed-off-by: Julien Stephan --- scripts/lib/devtool/standard.py | 23 ++- 1 file changed, 14 inserti

[OE-core] [PATCH 2/2] oeqa/selftest/devtool: add test for updating local files into another layer

2024-04-15 Thread Julien Stephan
We don't have a test to check if we can correctly devtool update-recipe/finish into another layer. So update the existing test_devtool_update_recipe_local_files to also check the updates into another layer. Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py | 20

[OE-core] [PATCH] oeqa: selftest: context: run tests serially if testtools/subunit modules are not found

2024-04-15 Thread Julien Stephan
t;<..>/poky/meta/lib/oeqa/core/utils/concurrencytest.py", line 22, in import testtools ModuleNotFoundError: No module named 'testtools' Fix this by adding a custom callback on -j/--num-processes parameter to check testtools and subunit modules. Fallback to serial testing if m

Re: [OE-core] [PATCH v2] devtool: standard: throws appropriate error if source is in detached HEAD

2024-04-11 Thread Julien Stephan
Le mer. 10 avr. 2024 à 13:23, Peter Kjellerstedt a écrit : > > > -Original Message- > > From: openembedded-core@lists.openembedded.org > > On Behalf Of Julien Stephan > > Sent: den 10 april 2024 09:07 > > To: openembedded-core@lists.openembedded.org &g

[OE-core] [PATCH v2] devtool: standard: throws appropriate error if source is in detached HEAD

2024-04-10 Thread Julien Stephan
in/sh: -c: line 1: `git checkout (HEAD detached at 9bbf87e)' Check this and throws an appropriate error in this case Signed-off-by: Julien Stephan --- Changes in v2: - update error message to avoid confusion with a potentially non existing "main" branch --- scripts/lib/devtool/standa

Re: [OE-core] [PATCH] devtool: standard: throws appropriate error if source is in detached HEAD

2024-04-10 Thread Julien Stephan
Le mer. 10 avr. 2024 à 01:11, Peter Kjellerstedt a écrit : > > > -Original Message- > > From: openembedded-core@lists.openembedded.org > > On Behalf Of Julien Stephan > > Sent: den 9 april 2024 09:19 > > To: openembedded-core@lists.openembedded.org &g

[OE-core] [PATCH] devtool: standard: throws appropriate error if source is in detached HEAD

2024-04-09 Thread Julien Stephan
in/sh: -c: line 1: `git checkout (HEAD detached at 9bbf87e)' Check this and throws an appropriate error in this case Signed-off-by: Julien Stephan --- scripts/lib/devtool/standard.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devto

[OE-core] [PATCH v3 3/3] oeqa/selftest/devtool: add test for recipes with multiple sources in SRC_URI

2024-01-23 Thread Julien Stephan
add a non regression test for devtool modify/build on recipe having several sources in SRC_URI Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py | 26 + 1 file changed, 26 insertions(+) diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta

[OE-core] [PATCH v3 1/3] externalsrc: fix task dependency for do_populate_lic

2024-01-23 Thread Julien Stephan
t;...>/poky/meta/recipes-core/init-ifupdown/init-ifupdown_1.0.bb:do_populate_lic) failed with exit code '1' Fix this by forcing the do_populate_lic task to run after do_unpack Signed-off-by: Julien Stephan --- meta/classes/externalsrc.bbclass | 1 + 1 file changed, 1 insertion(

[OE-core] [PATCH v3 2/3] devtool: modify: add support for multiple source in SRC_URI

2024-01-23 Thread Julien Stephan
not be able to generate patches for the secondary sources * type="git-dependency" is added for secondary sources even on non git sources, so we may want to rename this parameter Signed-off-by: Julien Stephan --- scripts/lib/devtool/standard.py | 20 1 file chan

[OE-core] [PATCH v3 0/3] devtool: modify: add support for multiple source

2024-01-23 Thread Julien Stephan
ace/appends/) Doing that, I tested my work with the bzip2 recipe and found another bug on do_populate_lic dependency. I added a patch to this series because otherwise the selftest will not work, but it can be applied separately. Cheers Julien Julien Stephan (3): externalsrc: fix task dependency

[OE-core][PATCH] oeqa/selftest/recipetool: fix metadata corruption on meta layer

2023-12-11 Thread Julien Stephan
-by: Julien Stephan --- depends on https://lists.openembedded.org/g/openembedded-core/message/192176 --- meta/lib/oeqa/selftest/cases/recipetool.py | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest

[OE-core][PATCH] oeqa/selftest/recipetool: remove left over from development

2023-12-11 Thread Julien Stephan
Remove leftover code needed for development that was unintentionally committed Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa

[OE-core][PATCH] oeqa/selftest/recipetool: add back checksum checks on pypi tests

2023-12-11 Thread Julien Stephan
include the expected SRC_URI checksums so add only the needed ones. Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest/cases

Re: [OE-core] [RESEND][PATCH 1/7] bitbake: utils: remove spaces on empty lines

2023-12-06 Thread Julien Stephan
Le mar. 5 déc. 2023 à 17:27, Julien Stephan via lists.openembedded.org a écrit : > > Le mar. 5 déc. 2023 à 16:45, Alexandre Belloni > a écrit : > > > > On 05/12/2023 15:57:58+0100, Julien Stephan wrote: > > > Le mar. 5 déc. 2023 à 08:50, Alexandre Belloni > >

Re: [OE-core] [RESEND][PATCH 1/7] bitbake: utils: remove spaces on empty lines

2023-12-05 Thread Julien Stephan
Le mar. 5 déc. 2023 à 16:45, Alexandre Belloni a écrit : > > On 05/12/2023 15:57:58+0100, Julien Stephan wrote: > > Le mar. 5 déc. 2023 à 08:50, Alexandre Belloni > > a écrit : > > > > > > Please do not send bitbake patches in the same series as oe-core pat

Re: [OE-core] [RESEND][PATCH 1/7] bitbake: utils: remove spaces on empty lines

2023-12-05 Thread Julien Stephan
the bitbake patch? Cheers Julien > > On 04/12/2023 16:59:28+0100, Julien Stephan wrote: > > Signed-off-by: Julien Stephan > > --- > > bitbake/lib/bb/utils.py | 16 > > 1 file changed, 8 insertions(+), 8 deletions(-) > > > > diff --g

[OE-core] [PATCH v3 07/10] recipeutils: bbappend_recipe: allow to patch the recipe itself

2023-12-05 Thread Julien Stephan
Add a new parameter update_original_recipe to allow to patch a recipe instead of creating/updating a bbappend Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 35 +-- 1 file changed, 25 insertions(+), 10 deletions(-) diff --git a/meta/lib/oe

[OE-core] [PATCH v3 06/10] recipetool: appendsrcfile(s): use params instead of extraline

2023-12-05 Thread Julien Stephan
appendsrcfiles used to not add new src_uri entry if the entry already exist even with different parameters while oe.recipeutils.bbappend_recipe adds it if parameters are different (and remove the old entry) Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 21

[OE-core] [PATCH v3 10/10] oeqa/selftest/recipetool: appendsrc: add test for update mode

2023-12-05 Thread Julien Stephan
add a basic test for testing the update mode of recipetool appendsrcfile(s) Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 28 ++ 1 file changed, 28 insertions(+) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa

[OE-core] [PATCH v3 09/10] oeqa/selftest/recipetool: appendsrfile: add test for machine

2023-12-05 Thread Julien Stephan
Add a new test for machine specific bbappend override Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 44 +++--- 1 file changed, 38 insertions(+), 6 deletions(-) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest

[OE-core] [PATCH v3 08/10] recipetool: appendsrcfile(s): add a mode to update the recipe itself

2023-12-05 Thread Julien Stephan
Add a new option -u/--update-recipe to update the recipe itself instead of creating/updating a bbappend file Signed-off-by: Julien Stephan --- scripts/lib/recipetool/append.py | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/scripts/lib/recipetool/append.py b/scripts/lib

[OE-core] [PATCH v3 05/10] recipeutils: bbappend_recipe: remove old srcuri entry if parameters are different

2023-12-05 Thread Julien Stephan
-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 8 1 file changed, 8 insertions(+) diff --git a/meta/lib/oe/recipeutils.py b/meta/lib/oe/recipeutils.py index 1d793693bfd..d86873056fe 100644 --- a/meta/lib/oe/recipeutils.py +++ b/meta/lib/oe/recipeutils.py @@ -776,14 +776,22 @@ def

[OE-core] [PATCH v3 03/10] recipeutils: bbappend_recipe: fix docstring

2023-12-05 Thread Julien Stephan
Add missing patchdir key in docstring description for srcfiles parameters. Also fix typo: value --> key in srcfile docstring description Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/meta/lib

[OE-core] [PATCH v3 04/10] recipeutils: bbappend_recipe: add a way to specify the name of the file to add

2023-12-05 Thread Julien Stephan
bbappend_recipe can take a dict of source files to add to SRC_URI where the key is the full path to the file to be added and the value is a dict Add a new optionnal entry "newname" to specify the name of the newly added file Signed-off-by: Julien Stephan --- meta/lib/oe/recipeut

[OE-core] [PATCH v3 02/10] recipeutils: bbappend_recipe: fix undefined variable

2023-12-05 Thread Julien Stephan
In the case get_bbappend_path returns None (could not find the layer containing the recipe) the error message tries to print the recipefile, but it is not defined. Fix it. Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/lib

[OE-core] [PATCH v3 00/10] Several fixes around recipetool appendsrcfile(s) and oe.recipeutils.bbappend_recipe

2023-12-05 Thread Julien Stephan
different parameters already exists to avoid duplicate entries with different parameters Link to v1: https://lists.openembedded.org/g/openembedded-core/message/191544 Link to v2: https://lists.openembedded.org/g/openembedded-core/message/191744 Julien Stephan (10): recipetool: appendsrcfile(s

[OE-core] [PATCH v3 01/10] recipetool: appendsrcfile(s): add dry-run mode

2023-12-05 Thread Julien Stephan
Add dry-run mode for recipetool appendsrcfile and appendsrcfiles, with if necessary, a diff of changes Signed-off-by: Julien Stephan --- scripts/lib/recipetool/append.py | 33 +++- 1 file changed, 32 insertions(+), 1 deletion(-) diff --git a/scripts/lib/recipetool

[OE-core] [RESEND][PATCH 0/7] Devtool/Recipetool: adding pypi support

2023-12-04 Thread Julien Stephan
also contains some bug fixes I found during my testing. Pushed my dev branch here: https://git.yoctoproject.org/poky-contrib/log/?h=jstephan/devtool-add-pypi-support Cheers Julien Julien Stephan (7): bitbake: utils: remove spaces on empty lines recipetool: create_buildsys_python.py:

[OE-core] [RESEND][PATCH 7/7] oeqa/selftest/recipetool/devtool: add test for pypi class

2023-12-04 Thread Julien Stephan
https://files.pythonhosted.org/packages/<...> So add tests for the new syntax and modify old tests Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py| 4 +- meta/lib/oeqa/selftest/cases/recipetool.py | 112 +++-- 2 files changed, 85 insertions(

[OE-core] [RESEND][PATCH 6/7] oeqa/selftest/recipetool: remove spaces on empty lines

2023-12-04 Thread Julien Stephan
Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest/cases/recipetool.py index 55cbba9ca74..4bc28a4f2ee 100644 --- a/meta/lib/oeqa

[OE-core] [RESEND][PATCH 5/7] recipetool: create_buildsys_python: add pypi support

2023-12-04 Thread Julien Stephan
iles.pythonhosted.org/packages/<...> Signed-off-by: Julien Stephan --- scripts/lib/devtool/standard.py | 3 + scripts/lib/recipetool/create.py | 1 + .../lib/recipetool/create_buildsys_python.py | 72 +++ 3 files changed, 76 insertions(+) diff --git a/scrip

[OE-core] [RESEND][PATCH 4/7] recipetool: create: add new optional process_url callback for plugins

2023-12-04 Thread Julien Stephan
' to the handled list and must return the fetchuri No functional changes expected for plugins non implementing this optional callback Signed-off-by: Julien Stephan --- scripts/lib/recipetool/create.py | 54 +++- 1 file changed, 32 insertions(+), 22 deletions(-) diff --git

[OE-core] [RESEND][PATCH 1/7] bitbake: utils: remove spaces on empty lines

2023-12-04 Thread Julien Stephan
Signed-off-by: Julien Stephan --- bitbake/lib/bb/utils.py | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/bitbake/lib/bb/utils.py b/bitbake/lib/bb/utils.py index b401fa5ec7a..61ffad92ce3 100644 --- a/bitbake/lib/bb/utils.py +++ b/bitbake/lib/bb/utils.py

[OE-core] [RESEND][PATCH 2/7] recipetool: create_buildsys_python.py: initialize metadata

2023-12-04 Thread Julien Stephan
In the case pyproject.toml doesn't contains metadatas, the metadata variable is not initialized and the plugin throws an error and falls back to another plugin, which is not the desired behaviour. So just ignore metadata if we don't have them Signed-off-by: Julien Stephan --- scripts/lib

[OE-core] [RESEND][PATCH 3/7] recipetool: create: add trailing newlines

2023-12-04 Thread Julien Stephan
if there are two consecutive blanks after a removal Signed-off-by: Julien Stephan --- scripts/lib/recipetool/create.py | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/scripts/lib/recipetool/create.py b/scripts/lib/recipetool/create.py index 293198d1c88..f5d541eb6c1 100644

[OE-core] [PATCH v2 7/9] recipetool: appendsrcfile(s): add a mode to update the recipe itself

2023-12-04 Thread Julien Stephan
Add a new option -u/--update-recipe to update the recipe itself instead of creating/updating a bbappend file Signed-off-by: Julien Stephan --- scripts/lib/recipetool/append.py | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/scripts/lib/recipetool/append.py b/scripts/lib

[OE-core] [PATCH v2 0/9] Several fixes around recipetool appendsrcfile(s) and oe.recipeutils.bbappend_recipe

2023-12-04 Thread Julien Stephan
v1: https://lists.openembedded.org/g/openembedded-core/message/191544 Julien Stephan (9): recipetool: appendsrcfile(s): add dry-run mode recipeutils: bbappend_recipe: fix undefined variable recipeutils: bbappend_recipe: add a way to specify the name of the file to add recipeutils: bbappend_recipe:

[OE-core] [PATCH v2 8/9] oeqa/selftest/recipetool: appendsrfile: add test for machine

2023-12-04 Thread Julien Stephan
Add a new test for machine specific bbappend override Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 44 +++--- 1 file changed, 38 insertions(+), 6 deletions(-) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest

[OE-core] [PATCH v2 5/9] recipetool: appendsrcfile(s): use params instead of extraline

2023-12-04 Thread Julien Stephan
appendsrcfiles used to not add new src_uri entry if the entry already exist even with different parameters while oe.recipeutils.bbappend_recipe adds it if parameters are different (and remove the old entry) Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 21

[OE-core] [PATCH v2 9/9] oeqa/selftest/recipetool: appendsrc: add test for update mode

2023-12-04 Thread Julien Stephan
add a basic test for testing the update mode of recipetool appendsrcfile(s) Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 28 ++ 1 file changed, 28 insertions(+) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa

[OE-core] [PATCH v2 6/9] recipeutils: bbappend_recipe: allow to patch the recipe itself

2023-12-04 Thread Julien Stephan
Add a new parameter update_original_recipe to allow to patch a recipe instead of creating/updating a bbappend Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 35 +-- 1 file changed, 25 insertions(+), 10 deletions(-) diff --git a/meta/lib/oe

[OE-core] [PATCH v2 4/9] recipeutils: bbappend_recipe: remove old srcuri entry if parameters are different

2023-12-04 Thread Julien Stephan
-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 8 1 file changed, 8 insertions(+) diff --git a/meta/lib/oe/recipeutils.py b/meta/lib/oe/recipeutils.py index 9784ae3b031..8aa82396c86 100644 --- a/meta/lib/oe/recipeutils.py +++ b/meta/lib/oe/recipeutils.py @@ -774,14 +774,22 @@ def

[OE-core] [PATCH v2 2/9] recipeutils: bbappend_recipe: fix undefined variable

2023-12-04 Thread Julien Stephan
In the case get_bbappend_path returns None (could not find the layer containing the recipe) the error message tries to print the recipefile, but it is not defined. Fix it. Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/lib

[OE-core] [PATCH v2 3/9] recipeutils: bbappend_recipe: add a way to specify the name of the file to add

2023-12-04 Thread Julien Stephan
to add Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 20 scripts/lib/devtool/standard.py | 6 +++--- scripts/lib/recipetool/append.py | 4 ++-- 3 files changed, 17 insertions(+), 13 deletions(-) diff --git a/meta/lib/oe/recipeutils.py b/meta/lib/oe

[OE-core] [PATCH v2 1/9] recipetool: appendsrcfile(s): add dry-run mode

2023-12-04 Thread Julien Stephan
Add dry-run mode for recipetool appendsrcfile and appendsrcfiles, with if necessary, a diff of changes Signed-off-by: Julien Stephan --- scripts/lib/recipetool/append.py | 33 +++- 1 file changed, 32 insertions(+), 1 deletion(-) diff --git a/scripts/lib/recipetool

[OE-core] [PATCH 8/8] oeqa/selftest/recipetool: appendsrc: add test for update mode

2023-11-30 Thread Julien Stephan
add a basic test for testing the update mode of recipetool appendsrcfile(s) Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 28 ++ 1 file changed, 28 insertions(+) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa

[OE-core] [PATCH 7/8] oeqa/selftest/recipetool: appendsrfile: add test for machine

2023-11-30 Thread Julien Stephan
Add a new test for machine specific bbappend override Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 23 +++--- 1 file changed, 20 insertions(+), 3 deletions(-) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest

[OE-core] [PATCH 6/8] recipetool: appendsrcfile(s): add a mode to update the recipe itself

2023-11-30 Thread Julien Stephan
Add a new option -u/--update-recipe to update the recipe itself instead of creating/updating a bbappend file Signed-off-by: Julien Stephan --- scripts/lib/recipetool/append.py | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/scripts/lib/recipetool/append.py b/scripts/lib

[OE-core] [PATCH 5/8] recipeutils: bbappend_recipe: allow to patch the recipe itself

2023-11-30 Thread Julien Stephan
Add a new parameter update_original_recipe to allow to patch a recipe instead of creating/updating a bbappend Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 30 +- 1 file changed, 21 insertions(+), 9 deletions(-) diff --git a/meta/lib/oe

[OE-core] [PATCH 4/8] recipetool: appendsrcfile(s): use params instead of extraline

2023-11-30 Thread Julien Stephan
, and update the test or remove it. Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 10 -- scripts/lib/recipetool/append.py | 23 -- 2 files changed, 16 insertions(+), 17 deletions(-) diff --git a/meta/lib/oeqa/selftest/cases

[OE-core] [PATCH 3/8] recipeutils: bbappend_recipe: add a way to specify the name of the file to add

2023-11-30 Thread Julien Stephan
to add Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 20 scripts/lib/devtool/standard.py | 6 +++--- scripts/lib/recipetool/append.py | 4 ++-- 3 files changed, 17 insertions(+), 13 deletions(-) diff --git a/meta/lib/oe/recipeutils.py b/meta/lib/oe

[OE-core] [PATCH 1/8] recipetool: appendsrcfile(s): add dry-run mode

2023-11-30 Thread Julien Stephan
Add dry-run mode for recipetool appendsrcfile and appendsrcfiles, with if necessary, a diff of changes Signed-off-by: Julien Stephan --- scripts/lib/recipetool/append.py | 33 +++- 1 file changed, 32 insertions(+), 1 deletion(-) diff --git a/scripts/lib/recipetool

[OE-core] [PATCH 2/8] recipeutils: bbappend_recipe: fix undefined variable

2023-11-30 Thread Julien Stephan
In the case get_bbappend_path returns None (could not find the layer containing the recipe) the error message tries to print the recipefile, but it is not defined. Fix it. Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/lib

[OE-core] [RFC] [PATCH 0/7] Several fixes around recipetool appendsrcfile(s) and oe.recipeutils.bbappend_recipe

2023-11-30 Thread Julien Stephan
parameters are different then we should add the new src_entry (so old behaviour was not correct) and remove old entry so either use "SRC_URI:remove" in bbapend file or completely remove the old entry in patch mode What do you think? Cheers Julien Julien Stephan (7): recipetool: appendsrc

[OE-core] [RFC][PATCH 0/1] recipetool: add plugin to create recipe for rust based application

2023-11-27 Thread Julien Stephan
ock case... NOTE: created recipes are built tested but not tested at runtime for now. Dev branch on poky-contrib: [2] [1]: https://git.yoctoproject.org/poky/tree/meta/classes-recipe/cargo.bbclass#n45 [2]: https://git.yoctoproject.org/poky-contrib/log/?h=jstephan/devtool-add-rust-support Julien S

[OE-core] [PATCH] recipetool: create: add support for crates

2023-11-27 Thread Julien Stephan
This commit adds a new plugin for creating recipes for crates (rust based applications). TODO: add more description here on final version of the patch Signed-off-by: Julien Stephan --- scripts/lib/recipetool/create_cargo.py | 125 + 1 file changed, 125 insertions

Re: [OE-core] [PATCH 0/6] devtool: add support of submodules

2023-11-23 Thread Julien Stephan
> we'll deal with it :) > Hi Alex, yes I do agree.. but I was not able to split it, because I didn't want to introduce "non working" commits in the middle of the series :/ Let me know if some rework is needed! Cheers Julien > Alex > > On Wed, 22 Nov 2023 at 12:08, Julien Ste

[OE-core] [PATCH] bitbake: fetch2: git: add missing destsuffix and subpath parameters in docstrings

2023-11-22 Thread Julien Stephan
Docstring for git fetcher is missing destsuffix and subpath parameters, so add them Signed-off-by: Julien Stephan --- bitbake/lib/bb/fetch2/git.py | 8 1 file changed, 8 insertions(+) diff --git a/bitbake/lib/bb/fetch2/git.py b/bitbake/lib/bb/fetch2/git.py index 27a0d05144c

[OE-core] [PATCH 6/6] lib/oe/recipeutils.py: remove trailing white-spaces

2023-11-22 Thread Julien Stephan
Remove useless trailing white-spaces Signed-off-by: Julien Stephan --- meta/lib/oe/recipeutils.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/lib/oe/recipeutils.py b/meta/lib/oe/recipeutils.py index 58776f4a9a7..25b159bc1bc 100644 --- a/meta/lib/oe/recipeutils.py

[OE-core] [PATCH 5/6] oeqa/selftest/devtool: add test for git submodules

2023-11-22 Thread Julien Stephan
Add a test for gitsm recipes. This tests that we can do changes on submodules, commit them and properly extract the patches Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py | 47 + 1 file changed, 47 insertions(+) diff --git a/meta/lib/oeqa

[OE-core] [PATCH 4/6] devtool: add support for git submodules

2023-11-22 Thread Julien Stephan
iated to it, and all following "git submodule foreach" commands will fail. So detect that a git tree was extracted inside S and correctly add it using "git submodule add ", so that it will be considered as a regular git submodule Signed-off-by: Julien Stephan --- meta/lib/o

[OE-core] [PATCH 3/6] devtool: tag all submodules

2023-11-22 Thread Julien Stephan
In the case of a repository with submodules, we need to add the "devtool-base" and "devtool-patched" tag on all submodules in order to properly detect the added/removed/modified patches Signed-off-by: Julien Stephan --- meta/classes/devtool-source.bbclass | 3 +++ scripts/

[OE-core] [PATCH 0/6] devtool: add support of submodules

2023-11-22 Thread Julien Stephan
is available here : [1]. It passed full recipetool/devtool oeselftest locally Cheers Julien [1]: https://git.yoctoproject.org/poky-contrib/log/?h=jstephan/devtool-submodule-fix Julien Stephan (6): devtool: fix update-recipe dry-run mode devtool: finish/update-recipe: restrict mode srcrev to recipes

[OE-core] [PATCH 2/6] devtool: finish/update-recipe: restrict mode srcrev to recipes fetched from SCM

2023-11-22 Thread Julien Stephan
a check and abort with a proper error message. Signed-off-by: Julien Stephan --- scripts/lib/devtool/standard.py | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/scripts/lib/devtool/standard.py b/scripts/lib/devtool/standard.py index cd79c7802cb..55fa38ccfb8 100644 --- a

[OE-core] [PATCH 1/6] devtool: fix update-recipe dry-run mode

2023-11-22 Thread Julien Stephan
When running devtool update-recipe with --mode=srcrev AND --append switch in dry-run, we get the following error: Traceback (most recent call last): [...] Exception: destpath should be set here Fix this by removing a misplaced else statement in _update_recipe_srcrev Signed-off-by: Julien

[OE-core] [PATCH 7/7] oeqa/selftest/recipetool/devtool: add test for pypi class

2023-11-17 Thread Julien Stephan
https://files.pythonhosted.org/packages/<...> So add tests for the new syntax and modify old tests Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py| 4 +- meta/lib/oeqa/selftest/cases/recipetool.py | 112 +++-- 2 files changed, 85 insertions(

[OE-core] [PATCH 6/7] oeqa/selftest/recipetool: remove spaces on empty lines

2023-11-17 Thread Julien Stephan
Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest/cases/recipetool.py index 55cbba9ca74..4bc28a4f2ee 100644 --- a/meta/lib/oeqa

[OE-core] [PATCH 5/7] recipetool: create_buildsys_python: add pypi support

2023-11-17 Thread Julien Stephan
tax: * recipetool create https://files.pythonhosted.org/packages/<...> Signed-off-by: Julien Stephan --- scripts/lib/devtool/standard.py | 3 + scripts/lib/recipetool/create.py | 1 + .../lib/recipetool/create_buildsys_python.py | 72 +++ 3 files changed, 76 ins

[OE-core] [PATCH 2/7] recipetool: create_buildsys_python.py: initialize metadata

2023-11-17 Thread Julien Stephan
In the case pyproject.toml doesn't contains metadatas, the metadata variable is not initialized and the plugin throws an error and falls back to another plugin, which is not the desired behaviour. So just ignore metadata if we don't have them Signed-off-by: Julien Stephan --- scripts/lib

[OE-core] [PATCH 3/7] recipetool: create: add trailing newlines

2023-11-17 Thread Julien Stephan
if there are two consecutive blanks after a removal Signed-off-by: Julien Stephan --- scripts/lib/recipetool/create.py | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/scripts/lib/recipetool/create.py b/scripts/lib/recipetool/create.py index 293198d1c88..f5d541eb6c1 100644

[OE-core] [PATCH 4/7] recipetool: create: add new optional process_url callback for plugins

2023-11-17 Thread Julien Stephan
' to the handled list and must return the fetchuri No functional changes expected for plugins non implementing this optional callback Signed-off-by: Julien Stephan --- scripts/lib/recipetool/create.py | 54 +++- 1 file changed, 32 insertions(+), 22 deletions(-) diff --git

[OE-core] [PATCH 0/7] Devtool/Recipetool: adding pypi support

2023-11-17 Thread Julien Stephan
also contains some bug fixes I found during my testing. Pushed my dev branch here: https://git.yoctoproject.org/poky-contrib/log/?h=jstephan/devtool-add-pypi-support Cheers Julien Julien Stephan (7): bitbake: utils: remove spaces on empty lines recipetool: create_buildsys_python.py:

[OE-core] [PATCH 1/7] bitbake: utils: remove spaces on empty lines

2023-11-17 Thread Julien Stephan
Signed-off-by: Julien Stephan --- bitbake/lib/bb/utils.py | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/bitbake/lib/bb/utils.py b/bitbake/lib/bb/utils.py index b401fa5ec7a..61ffad92ce3 100644 --- a/bitbake/lib/bb/utils.py +++ b/bitbake/lib/bb/utils.py

[OE-core] [PATCH] oeqa/selftest/devtool: fix test_devtool_modify_overrides test

2023-11-16 Thread Julien Stephan
machine configuration Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta/lib/oeqa/selftest/cases/devtool.py index b5c488be8e8..ab58971fec7 100644 --- a/meta

Re: [OE-core] [RESEND PATCH 2/2] oeqa/selftest/devtool: remove spaces on empty line

2023-10-25 Thread Julien Stephan
Le mer. 25 oct. 2023 à 17:01, Luca Ceresoli a écrit : > > Hello Julien, > > On Tue, 24 Oct 2023 16:17:54 +0200 > "Julien Stephan" wrote: > > > Signed-off-by: Julien Stephan > > --- > > meta/lib/oeqa/selftest/cases/devtool.py | 2 +- &

[OE-core][RESEND][PATCH 2/2] oeqa/selftest/devtool: remove spaces on empty line

2023-10-25 Thread Julien Stephan
Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta/lib/oeqa/selftest/cases/devtool.py index c36b1efa78b..b5c488be8e8 100644 --- a/meta/lib/oeqa/selftest

[OE-core][RESEND][PATCH 1/2] oeqa/selftest/devtool: abort if a local workspace already exist

2023-10-25 Thread Julien Stephan
bblayers.conf or layer.conf to fix it. Found duplicated BBFILE_COLLECTIONS 'workspacelayer', check bblayers.conf or layer.conf to fix it. Check if a workspacelayer exists, warn the user and abort the tests Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py | 3 +++ 1 file changed

[OE-core] [PATCH v4 5/5] oeqa/selftest/recipetool: add selftest for PEP-517 recipe creation

2023-10-25 Thread Julien Stephan
of python starting from 3.11 Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 102 + 1 file changed, 102 insertions(+) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest/cases/recipetool.py index d3aea74228f

Re: [OE-core] [PATCH v3 0/4] devtool/recipetool: add support of PEP-517

2023-10-25 Thread Julien Stephan
Le mer. 25 oct. 2023 à 16:11, Richard Purdie a écrit : > > On Tue, 2023-10-24 at 16:40 +0200, Julien Stephan wrote: > > Hello all, > > > > This series adds basic support for PEP-517 in recipetool/devtool, plus somes > > others fixes. > > > >

[OE-core] [PATCH v4 4/5] scripts:recipetool:create_buildsys_python: add PEP517 support

2023-10-25 Thread Julien Stephan
. Some features may be missing, such as the extra dependencies. [YOCTO #14737] [1]: https://peps.python.org/pep-0517/ Signed-off-by: Julien Stephan --- .../lib/recipetool/create_buildsys_python.py | 268 +- 1 file changed, 267 insertions(+), 1 deletion(-) diff --git a/scripts/lib

[OE-core] [PATCH v4 0/5] devtool/recipetool: add support of PEP-517

2023-10-25 Thread Julien Stephan
" section - add the selftests for the 3 supported backends. Skip the tests if tomliib (python > 3.11) or tomli is not installed - fix self test failing with the new "python3-" prefix added to recipetool for python recipes Julien Stephan (5): scripts:recipetool:create_

[OE-core] [PATCH v4 1/5] scripts:recipetool:create_buildsys_python: fix license note

2023-10-25 Thread Julien Stephan
CENSE =" is not yet present so we can never display this note. Replace the "LICENSE =" condition with "##LICENSE_PLACEHOLDER##" to actually be able to display the note message Signed-off-by: Julien Stephan --- scripts/lib/recipetool/create_buildsys_python.py | 2 +- 1 f

[OE-core] [PATCH v4 3/5] scripts:recipetool:create_buildsys_python: refactor code for futur PEP517 addition

2023-10-25 Thread Julien Stephan
Signed-off-by: Julien Stephan --- .../lib/recipetool/create_buildsys_python.py | 748 +- 1 file changed, 385 insertions(+), 363 deletions(-) diff --git a/scripts/lib/recipetool/create_buildsys_python.py b/scripts/lib/recipetool/create_buildsys_python.py index 502e1dfbc3d

[OE-core] [PATCH v4 2/5] scripts:recipetool:create_buildsys_python: prefix created recipes with python3-

2023-10-25 Thread Julien Stephan
By convention, all python recipes start with "python3-" so update create_buildsys_python to do this This rule doesn't apply for packages already starting with "python" Update recipetool's selftest accordingly Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest

[OE-core] [PATCH v3 3/4] scripts:recipetool:create_buildsys_python: add PEP517 support

2023-10-24 Thread Julien Stephan
. Some features may be missing, such as the extra dependencies. [YOCTO #14737] [1]: https://peps.python.org/pep-0517/ Signed-off-by: Julien Stephan --- .../lib/recipetool/create_buildsys_python.py | 268 +- 1 file changed, 267 insertions(+), 1 deletion(-) diff --git a/scripts/lib

[OE-core] [PATCH v3 2/4] scripts:recipetool:create_buildsys_python: refactor code for futur PEP517 addition

2023-10-24 Thread Julien Stephan
Signed-off-by: Julien Stephan --- .../lib/recipetool/create_buildsys_python.py | 748 +- 1 file changed, 385 insertions(+), 363 deletions(-) diff --git a/scripts/lib/recipetool/create_buildsys_python.py b/scripts/lib/recipetool/create_buildsys_python.py index 502e1dfbc3d

[OE-core] [PATCH v3 4/4] oeqa/selftest/recipetool: add selftest for PEP-517 recipe creation

2023-10-24 Thread Julien Stephan
of python starting from 3.11 Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/recipetool.py | 102 + 1 file changed, 102 insertions(+) diff --git a/meta/lib/oeqa/selftest/cases/recipetool.py b/meta/lib/oeqa/selftest/cases/recipetool.py index d3aea74228f

[OE-core] [PATCH v3 1/4] scripts:recipetool:create_buildsys_python: prefix created recipes with python3-

2023-10-24 Thread Julien Stephan
By convention, all python recipes start with "python3-" so update create_buildsys_python to do this This rule doesn't apply for packages already starting with "python" Update recipetool's selftest accordingly Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest

[OE-core] [PATCH v3 0/4] devtool/recipetool: add support of PEP-517

2023-10-24 Thread Julien Stephan
for the 3 supported backends. Skip the tests if tomliib (python > 3.11) or tomli is not installed - fix self test failing with the new "python3-" prefix added to recipetool for python recipes Julien Stephan (4): scripts:recipetool:create_buildsys_python: prefix

[OE-core] [RESEND PATCH 2/2] oeqa/selftest/devtool: remove spaces on empty line

2023-10-24 Thread Julien Stephan
Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/lib/oeqa/selftest/cases/devtool.py b/meta/lib/oeqa/selftest/cases/devtool.py index c36b1efa78b..b5c488be8e8 100644 --- a/meta/lib/oeqa/selftest

[OE-core] [RESEND PATCH 1/2] oeqa/selftest/devtool: abort if a local workspace already exist

2023-10-24 Thread Julien Stephan
bblayers.conf or layer.conf to fix it. Found duplicated BBFILE_COLLECTIONS 'workspacelayer', check bblayers.conf or layer.conf to fix it. Check if a workspacelayer exists, warn the user and abort the tests Signed-off-by: Julien Stephan --- meta/lib/oeqa/selftest/cases/devtool.py | 3 +++ 1 file changed

Re: [OE-core] [PATCH v2 4/4] scripts:recipetool:create_buildsys_python: add PEP517 support

2023-10-20 Thread Julien Stephan
Le ven. 20 oct. 2023 à 16:04, Richard Purdie a écrit : > > On Fri, 2023-10-20 at 14:57 +0200, Julien Stephan wrote: > > Le jeu. 19 oct. 2023 à 20:34, Alexandre Belloni > > a écrit : > > > > > > On 19/10/2023 20:20:33+0200, Julien Stephan wrote: > > &

Re: [OE-core] [PATCH v2 4/4] scripts:recipetool:create_buildsys_python: add PEP517 support

2023-10-20 Thread Julien Stephan
Le jeu. 19 oct. 2023 à 20:34, Alexandre Belloni a écrit : > > On 19/10/2023 20:20:33+0200, Julien Stephan wrote: > > Le jeu. 19 oct. 2023 à 15:49, Alexandre Belloni > > a écrit : > > > > > > Hello, > > > > > > On 19/10/2023 09:36:53+0200, J

Re: [OE-core] [PATCH v2 3/4] scripts:recipetool:create_buildsys_python: refactor code for futur PEP517 addition

2023-10-20 Thread Julien Stephan
Le ven. 20 oct. 2023 à 08:01, Alexandre Belloni a écrit : > > Hello, > > On 19/10/2023 09:36:52+0200, Julien Stephan wrote: > > In order to prepare the support for pyproject.toml (PEP517 [1]) enabled > > projects, refactor the code and move setup.py specific code i

Re: [OE-core] [PATCH v2 4/4] scripts:recipetool:create_buildsys_python: add PEP517 support

2023-10-19 Thread Julien Stephan
Le jeu. 19 oct. 2023 à 15:49, Alexandre Belloni a écrit : > > Hello, > > On 19/10/2023 09:36:53+0200, Julien Stephan wrote: > > add support for PEP517 [1] > > > > if a pyproject.toml file is found, use it to create the recipe, > > otherwise fallback to the

Re: [OE-core] [PATCH 1/4] scripts:recipetool:create_buildsys_python: fix license note

2023-10-19 Thread Julien Stephan
Le mer. 18 oct. 2023 à 21:58, Alexandre Belloni a écrit : > > Hello, > > On 18/10/2023 14:01:11+0200, Julien Stephan wrote: > > License field of setup is not always standardized, so we usually use the > > classifier to determine the correct license format to use in the r

[OE-core] [PATCH v2 4/4] scripts:recipetool:create_buildsys_python: add PEP517 support

2023-10-19 Thread Julien Stephan
add support for PEP517 [1] if a pyproject.toml file is found, use it to create the recipe, otherwise fallback to the old setup.py method. [YOCTO #14737] [1]: https://peps.python.org/pep-0517/ Signed-off-by: Julien Stephan --- .../lib/recipetool/create_buildsys_python.py | 234

[OE-core] [PATCH v2 3/4] scripts:recipetool:create_buildsys_python: refactor code for futur PEP517 addition

2023-10-19 Thread Julien Stephan
Signed-off-by: Julien Stephan --- .../lib/recipetool/create_buildsys_python.py | 748 +- 1 file changed, 385 insertions(+), 363 deletions(-) diff --git a/scripts/lib/recipetool/create_buildsys_python.py b/scripts/lib/recipetool/create_buildsys_python.py index 502e1dfbc3d

[OE-core] [PATCH v2 2/4] scripts:recipetool:create_buildsys_python: prefix created recipes with python3-

2023-10-19 Thread Julien Stephan
By convention, all python recipes start with "python3-" so update create_buildsys_python to do this This rule doesn't apply for packages already starting with "python" Signed-off-by: Julien Stephan --- scripts/lib/recipetool/create_buildsys_python.py | 5 + 1 file c

  1   2   >