[libreoffice-projects] [ANN] Please use Gerrit from now on for Patch Review

Petr Mladek pmladek at suse.cz
Tue Jun 19 02:13:45 PDT 2012


Bjoern Michaelsen píše v Po 18. 06. 2012 v 12:09 +0200:
> Hi all,
> 
> with:
> 
>  http://sweetshark.livejournal.com/13298.html
>
> gerrit is documented and ready to go.

Ah, there are several strange and long commands.

Also I miss the cooperation with the mailing list. I remember that the
main request was that mailing list will stay the main communication
point between developers. All patches that need review must be mentioned
there.

It means that gerrit should be able to detect patches for review on the
mailing list, integrate them, and make them ready for review.

My expectation would be that it sends a replay to the mailing list with
a link to diff, link to build results and commands to approve it.

Similar mail should be send for patches that were directly pushed into
gerrit for review. Of course, it must not send mail for every single
patch but just for patches where the developer explicitly asks for
review, so the numbers of mails on the mailing list stays the same.

> Please use it for code review as much as possible now as it simplifies
>  things a lot over manual patch fiddling on mailing lists. I will
>  update the EasyHacks to point to gerrit instead in the next days.

IMHO, we first need to conclude that gerrit is in usable state.


> The last remaining step will be making the repo at gerrit the reference (and
> the one at freedesktop a read-only mirror). I assume that to be prepared and
> done until mid-July(*).

Same here.

Best Regards,
Petr

PS: To make it clear. I am neutral about gerrit. I see some advantages
and also some risks. In each case, we must make Caolan, Kendy, Michael,
Stefan, Eike, Miklos, and all other very active patch reviewers happy
with it. I am excited when I look at the current mailing list and see so
many patches reviewed a pushed within one or two days. It helps to make
contributors active. It helps me a lot to do the tag in time. We must
not break this flow!



More information about the LibreOffice mailing list