all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Thomas Koch <thomas@koch.ro>
To: emacs-devel@gnu.org
Cc: Ted Zlatanov <tzz@lifelogs.com>
Subject: Re: Patch queue management systems
Date: Mon, 08 Dec 2014 22:49:15 +0100	[thread overview]
Message-ID: <14630069.9yU0C9RVCL@x121e> (raw)
In-Reply-To: <87zjb0ncqo.fsf@lifelogs.com>

We use gerrit at work so I can comment for it.

On Saturday, December 06, 2014 08:12:31 AM Ted Zlatanov wrote:
> My wishlist for a patch queue management system (PR/pull request system
> for those familiar with the Github model):
> 
> * targeted hydra builds for a specific branch
I don't know hydra. We use jenkins which has a very good plugin to integrate 
with gerrit. Every commit that lands in gerrit triggers a jenkins build and 
jenkins leaves a vote afterwards in gerrit. Thus a human doesn't need to do a 
review as long as the jenkins build isn't good.

> * a click-and-done way to merge a specific branch into master or emacs-24
This is called 'submit' in gerrit and it's click-and-done.

> * maybe a comment system
In gerrit you can comment and reply on every line of the commit, the commit 
message or comment on the whole change.

> * maybe debbugs integration (but I do hate debbugs... really hate it...)
- easy: link from gerrit changes to debbugs if a bug is referenced in the 
commit message
- a bit more involved: trigger debbugs actions for actions in gerrit (new 
change, change merged...)

> * ability to review the potential ChangeLog entries that will be
>   generated from the commit
The commit message is part of the review. We generate changelogs from commit 
messages and add an 'ignore:' trailer if a commit should not be included in 
the changelog.

A massive advantage of gerrit over reviewboard as used by the Apache 
Foundation is the frictionless integration with git.

On the other hand gerrit is extremely powerful and customizable which might be 
a problem for somebody setting it up for the first time.

Regards, Thomas Koch





  reply	other threads:[~2014-12-08 21:49 UTC|newest]

Thread overview: 85+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-19 23:57 bug#19113: Generate a ChangeLog file from commit logs Paul Eggert
2014-11-20 15:51 ` Eli Zaretskii
2014-11-20 17:43   ` Paul Eggert
2014-11-20 17:44   ` Paul Eggert
2014-11-26  5:38     ` Paul Eggert
2014-11-26  6:21       ` Jan D.
2014-11-26  7:50         ` Paul Eggert
2014-11-26 15:12       ` Stefan Monnier
2014-11-26 19:16         ` Paul Eggert
2014-11-26 22:15           ` Stefan Monnier
2014-12-05  8:07           ` Glenn Morris
2014-12-05  8:38             ` Eli Zaretskii
2014-12-05 14:33             ` Ted Zlatanov
2014-12-05 18:49               ` Stefan Monnier
2014-12-05 20:08                 ` Ted Zlatanov
2014-12-06  4:59                   ` Stefan Monnier
2014-12-06 13:17                     ` Ted Zlatanov
2014-12-06  2:27                 ` Dmitry Gutov
2014-12-06  5:06                   ` Stefan Monnier
2014-12-06  7:26                     ` Eli Zaretskii
2014-12-09 13:46                     ` Dmitry Gutov
2014-12-06 10:08                   ` Patch queue management systems Eli Zaretskii
2014-12-06 13:12                     ` Ted Zlatanov
2014-12-08 21:49                       ` Thomas Koch [this message]
2014-12-08 23:10                       ` Lars Magne Ingebrigtsen
2014-12-09  0:00                         ` Lars Magne Ingebrigtsen
2014-12-09  0:32                           ` Lars Magne Ingebrigtsen
2014-12-09  2:57                         ` Stefan Monnier
2014-12-09  3:16                           ` Lars Magne Ingebrigtsen
2014-12-09 12:37                         ` Dmitry Gutov
2014-12-09 17:20                           ` Lars Magne Ingebrigtsen
2014-12-09 20:40                             ` Dmitry Gutov
2014-12-09 20:44                               ` Lars Magne Ingebrigtsen
2014-12-09 21:32                                 ` Dmitry Gutov
2014-12-09 21:58                                   ` Lars Magne Ingebrigtsen
2014-12-09 21:57                                 ` Lars Magne Ingebrigtsen
2014-12-10 14:35                                   ` Ted Zlatanov
2014-12-08 22:24                     ` Lars Magne Ingebrigtsen
2014-12-08 22:54                       ` joakim
2014-12-09  5:31                         ` Ivan Andrus
2014-12-09  0:06                       ` Thien-Thi Nguyen
2014-12-09  2:54                       ` Stefan Monnier
2014-12-09  3:15                         ` Lars Magne Ingebrigtsen
2014-12-09  8:13                       ` Stephen Leake
2014-12-09 13:03                     ` Dmitry Gutov
2014-12-09 17:09                       ` Eli Zaretskii
2014-12-09 17:17                         ` Dmitry Gutov
2014-12-09 17:36                           ` Eli Zaretskii
2014-12-09 19:31                             ` Dmitry Gutov
2014-12-10 14:39                             ` Ted Zlatanov
2014-12-11  8:59                               ` Steinar Bang
2014-12-11 11:52                                 ` Ted Zlatanov
2014-12-11 12:38                                   ` Steinar Bang
2014-12-11 14:31                                     ` Ted Zlatanov
2014-12-11 14:39                                       ` David Kastrup
2014-12-11 14:50                                         ` Ted Zlatanov
2014-12-11 14:55                                         ` Andreas Schwab
2014-12-11 15:00                                           ` Ted Zlatanov
2014-12-11 15:26                                             ` David Kastrup
2014-12-11 15:33                                               ` Ted Zlatanov
2014-12-11 15:49                                                 ` David Kastrup
2014-12-11 15:19                                           ` David Kastrup
2014-12-11 18:43                                 ` Achim Gratz
2014-12-05 15:38             ` bug#19113: Generate a ChangeLog file from commit logs Stefan Monnier
2014-12-06  4:42               ` Paul Eggert
2014-12-06  7:40                 ` Eli Zaretskii
2014-12-09  5:14                 ` Glenn Morris
2014-12-09  6:42                   ` Paul Eggert
2015-01-03  1:37             ` Glenn Morris
2014-11-20 18:00   ` Achim Gratz
2014-11-20 17:25 ` Pádraig Brady
2015-01-12  7:38 ` Paul Eggert
2015-01-15 14:58   ` Stefan Monnier
2015-01-15 21:46   ` Glenn Morris
2015-01-16  1:27     ` Paul Eggert
2015-01-30 16:34       ` Stefan Monnier
2015-01-30 20:14         ` Paul Eggert
2015-01-31  6:33           ` Stefan Monnier
2015-01-31  8:18             ` Paul Eggert
2015-01-31  9:06           ` Andreas Schwab
2015-03-21  2:37 ` Paul Eggert
2015-03-21  8:33   ` Eli Zaretskii
2015-03-21 14:38     ` Stefan Monnier
2015-03-22 23:53   ` Glenn Morris
2015-04-07  7:06   ` Paul Eggert

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=14630069.9yU0C9RVCL@x121e \
    --to=thomas@koch.ro \
    --cc=emacs-devel@gnu.org \
    --cc=tzz@lifelogs.com \
    /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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.