[OE-core] [PATCH] combo-layer: make Signed-off-by optional
Patrick Ohly
patrick.ohly at intel.com
Thu Mar 12 19:45:32 UTC 2015
On Thu, 2015-03-12 at 18:21 +0000, Paul Eggleton wrote:
> Hi Patrick,
>
> On Monday 09 March 2015 13:56:39 Patrick Ohly wrote:
> > +# global options
> > +[DEFAULT]
> > +
> > +# Add 'Signed-off-by' to all commits that get imported automatically.
> > +signoff = True
> > +
> > # component name
> > [bitbake]
> > # mandatory options
>
> So I'm OK with adding this in as an option. However to me a name like DEFAULT
> implies you're establishing a general section to apply default settings for
> all components where the component can override those defaults if it chooses,
> which doesn't really represent what this does - so a different name might be
> more appropriate (GLOBAL or _global_ perhaps?)
"DEFAULT" is the special string that Python's ConfigParser uses, well,
by default for that special section which does not show up in the list
of sections. I don't know how to rename that.
I'm probably abusing this concept a bit here: it seems that special
section is meant to provide default values that get returned also for
the other sections when they don't have their own value.
Here's a cleaner solution:
* Get rid of the [DEFAULT] section in the file.
* When reading it, on-the-fly prepend the string
'[this-is-not-really-a-repo'].
* When reading global properties, get it from that section.
* When listing sections to find repos, ignore it.
How about that?
--
Best Regards, Patrick Ohly
The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.
More information about the Openembedded-core
mailing list