[Libreoffice-qa] minutes of ESC call ...

Bjoern Michaelsen bjoern.michaelsen at canonical.com
Fri Feb 15 13:01:01 PST 2013


On Fri, Feb 15, 2013 at 02:30:33PM -0600, Norbert Thiebaud wrote:
> -2 means I oppose the 'idea' of the patch. no amount of tweaking can
> address my concerns.. one could use a -2 on a technically perfectly
> valid/correct patch that remove dbaccess from libreoffice for
> instance. -2 is effectively a veto because it is a sticky state that
> remain even if a new version of the patch is uploaded... so as long as
> there is a -2 one cannot Submit the patch, short of hacking gerrit or
> pushing the patch directly to master, by-passsing gerrit-review
> altogether.

Can we maybe make the text:
"There are concerns about the intend or implications of this change that need
clarification. Please do not merge this before consensus is reached by
discussion."

Having a 'please' makes this much more friendly (although as Norbert said to
push the change you would need to bypass gerrit anyway, but we dont have to rub
that it) and 'consensus'/'discussion' gives a clear road on how to proceed for
the contributor.

Best,

Bjoern


More information about the LibreOffice mailing list