Re: [OE-core] [PATCH 0/7] De-globalization of 'devtool'

["Mathieu Dubois-Briand" <mathieu.dubois-briand@bootlin.com>]

On Sun Jan 12, 2025 at 3:53 PM CET, Chris Laplante via lists.openembedded.org wrote:
> From: Chris Laplante <chris.laplante@agilent.com>
>
> This patchset removes global variables from the 'devtool' script. It
> also changes `Context` to be a dataclass.
>
> All devtool self-tests pass, tested via: oe-selftest -r devtool
>
> Chris Laplante (7):
>   devtool: un-globalize the 'basepath' variable
>   devtool: un-globalize 'workspace' variable
>   devtool: un-globalize 'context' variable and convert it to a dataclass
>   devtool: un-globalize 'config' variable
>   devtool: un-globalize 'plugins' variable
>   devtool: misc cleanups
>   devtool: remove unused 'config' param from '_create_workspace' method
>
>  scripts/devtool | 88 +++++++++++++++++++++++++------------------------
>  1 file changed, 45 insertions(+), 43 deletions(-)
>
> --
> 2.43.0

Hi Chris,

I believe this series is breaking some builds on the autobuilder. We
got the following error:

devtool: error: argument <subcommand>: invalid choice: 'sdk-install' (choose from create-workspace, add, modify, extract, sync, rename, update-recipe, status, reset, finish, edit-recipe, find-recipe, configure-help, import, deploy-target, undeploy-target, build-image, ide-sdk, search, menuconfig, upgrade, latest-version, check-upgrade-status, build, export)

https://valkyrie.yoctoproject.org/#/builders/80/builds/725/steps/13/logs/stdio

Can you have a look at this issue please ?

-- 
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com