Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


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!


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.