-----Original Message-----
From: Alexander Kanavin <alex.kana...@gmail.com> 
Sent: Wednesday, November 8, 2023 11:39 AM

> 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.

You are absolutely right, I had considered including the writer,
but decided to ignore that for now.

> 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.

I was working with the assumption that not validating would not be
acceptable.
I guess it comes down to whether we (want to) trust the users to validate
their input, or if we think it is more important to make sure that it is.

Sincerely,

Jermain Horsman
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#190353): 
https://lists.openembedded.org/g/openembedded-core/message/190353
Mute This Topic: https://lists.openembedded.org/mt/102444605/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to