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
next prev parent 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.