On Tue, 7 Nov 2023 at 16:27, Jermain Horsman <[email protected]> wrote:
> + def register_commands(self, sp):
> + parser_update_layers = self.add_command(sp, 'update-layers-setup',
> self.do_update_setup, parserecipes=False)
> + parser_update_layers.add_argument('json',
> + help='File location of the setup-layers json config.')
> + parser_update_layers.add_argument('repo',
> + help='Name of the repository to be updated.')
> + parser_update_layers.add_argument('--branch', '-b',
> + help='store branch',
> + action='store_true')
I'm afraid I can't accept this. The UI should be:
bitbake-layers create-layers-setup --update poky --reference <ref>
where ref is a revision, tag or branch.
A separate top level plugin is problematic because create-layers-setup
has specifically a sub-plugin mechanism in itself to support various
ways of writing layer setup, json writer is just one implementation,
and all extensions should be contained there.
Also --branch without parameters to say 'use current branch' is too
limiting. It's better to explicitly provide the reference, and write
it out into the json without validation for existence.
Alex
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#190313):
https://lists.openembedded.org/g/openembedded-core/message/190313
Mute This Topic: https://lists.openembedded.org/mt/102444605/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-
