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

[Petr Mladek]

Bjoern Michaelsen píše v Út 19. 06. 2012 v 13:38 +0200:
> On Tue, Jun 19, 2012 at 11:13:45AM +0200, Petr Mladek wrote:
> > 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.
> 
> Please provide constructive feedback to for example:
> 
>  https://bugs.freedesktop.org/show_bug.cgi?id=51159

Ah, this bug is about a daily digest. I think that we first need to
decide how much we want to modify the current work flow. Do we want to
really move most discussions from the mailing list to gerrit?

How people will be noticed about news in gerrit? Is one day digest
really enough?

Also, you know, there is the "hello/thanks/see you" much more involved
in the mailing list. I think that such things are important for the
community live.

Note that the cooperation with mailing list is even in the plan, see:

	+ setup mail notifications to the developer mailing list
	+ setup patch drop mailbox
	+ scripting to automagically identify patches on the mailing
          list

at
http://wiki.documentfoundation.org/Development/GerritMigration#Action_items


I think that gerrit should send mail to the mailing list immediately
when someone needs review (people without commit access, people that are
not sure about their fix).

For inspiration, I attach mail from the openSUSE Build Service. It
includes:

	+ description of changes
	+ diff of the spec file
	+ commands how to get full diff, approve or reject the change



>  http://nabble.documentfoundation.org/gerrit-for-release-branches-bots-ftw-tp3990804.html

I am going to answer it but I am busy with some other things.

> implementing this "into the blind" just for it to be rejected by those who
> where silent before is not the way to go.

I think that my request is nothing new for you. I think that it was
actually one of the most important requests since the beginning.


Best Regards,
Petr
-------------- next part --------------
An embedded message was scrubbed...
From: fstrba at novell.com
Subject: [obs submit-request 124460] devel:libraries:c_c++/libcdr: created by fstrba
Date: Mon, 11 Jun 2012 12:40:47 +0000
Size: 3314
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120619/1aca1b1f/attachment.mht>