Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yegor Yefremov <yegor_sub1@visionsystems.de>
To: buildroot@busybox.net
Subject: [Buildroot] [RFC] Gerrit for BR
Date: Fri, 21 Oct 2011 12:17:19 +0200	[thread overview]
Message-ID: <4EA146AF.6090402@visionsystems.de> (raw)
In-Reply-To: <87ehy6942n.fsf@macbook.be.48ers.dk>

Am 21.10.2011 09:49, schrieb Peter Korsgaard:
>>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:
> 
> Hi,
> 
>  >> What do you think about using Gerrit
>  >> (http://code.google.com/p/gerrit/) for Buildroot? Many OSS projects
>  >> seem to switch to this system.
> 
>  Thomas> How does it compare to patchwork (http://patchwork.ozlabs.org/)
>  Thomas> ?  Patchwork works in relation to a mailing-list, from which it
>  Thomas> automatically picks up patches and related review comments, and
>  Thomas> it allows maintainers to track which patches have been
>  Thomas> applied/not applied.
> 
>  Thomas> How does Gerrit work in that respect?
> 
> I don't have any hands on experience (but I am following the openocd
> list, so I see it used), and my initial feeling is that something like
> patchwork would fit better our current way of working.
> 
> But that's just an initial feeling, let's discuss at ELC and/or here on
> the list.

Peter, you just mentioned yourself, that you're following openocd list. Please see this thread for pros contra etc.
http://www.mail-archive.com/openocd-development at lists.berlios.de/msg18509.html

As far as I understand Gerrit integrates very good with Git and when used properly and by all contributors, you'll have all the patches in one place. The patches can be then commented online and you'll also get history for reworked patches. As soon as patches are verified, maintainer (you :-) ) can merge them with one click. These were the benefits Gerrit would bring Buidroot community. I think such approach makes sense, since lots of patches will be sent almost every day.

And as far as I understand from the openocd list thread mentioned above, patchwork still requires more handwork from maintainer than Gerrit.

Yegor

  reply	other threads:[~2011-10-21 10:17 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-20 14:11 [Buildroot] [RFC] Gerrit for BR Yegor Yefremov
2011-10-20 15:44 ` C.A, Subramaniam
2011-10-21  6:57 ` Thomas Petazzoni
2011-10-21  7:38   ` Bryan Hundven
2011-10-21  7:43     ` Bryan Hundven
2011-10-21  7:49   ` Peter Korsgaard
2011-10-21 10:17     ` Yegor Yefremov [this message]
2011-10-21 11:19     ` Thomas De Schampheleire
2011-10-21 11:48       ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EA146AF.6090402@visionsystems.de \
    --to=yegor_sub1@visionsystems.de \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox