From: Leo Famulari <leo@famulari.name>
To: Pjotr Prins <pjotr.public12@thebird.nl>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: gnu-patches back log
Date: Wed, 1 Mar 2017 01:16:25 -0500 [thread overview]
Message-ID: <20170301061625.GA11982@jasmine> (raw)
In-Reply-To: <20170228062531.GA647@mail.thebird.nl>
On Tue, Feb 28, 2017 at 06:25:31AM +0000, Pjotr Prins wrote:
> Now we have debbugs we can see there is a building back-log:
>
> https://debbugs.gnu.org/cgi/pkgreport.cgi?package=guix-patches;max-bugs=100;base-order=1;bug-rev=1
>
> A patch like this one
>
> https://debbugs.gnu.org/cgi/bugreport.cgi?bug=25725
>
> has been two weeks without comment. I think we should not leave patches without
> feedback longer than one week - even 3 days, to be honest. It is the surest way
> to kill enthusiasm.
>
> To move forward with Guix and to recognise the effort new submitters
> put in I would like to ask *all* reviewers to pick an outstanding
> patch on a regular basis. If reviewers split the work it should be doable.
We all know that patch review is important. But it's also real work, and
just as hard as writing patches in many cases. I think we all do it when
we find the motivation.
> Would it be an idea to send out weekly E-mails with patches that had
> no attention to a select list of reviewers? Or maybe to the ML as a
> whole? Basically it would read:
As long as the list of reviewers volunteered for that.
We already get the messages with the patches. I wonder if adding yet
another message to our mail boxes is going to help. At least for me, the
issue is finding the energy to review things, not tools for finding old
patches.
If we are interested in handling submissions more quickly, we could
arrange for package-related changes to be linted and built before they
get sent to the list subscribers. Spending time on a patch series before
learning that the submitter did not even test it reduces my motivation
to review.
next prev parent reply other threads:[~2017-03-01 6:16 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-12 14:21 How to M-x debbugs-gnu with new guix-patches Christopher Allan Webber
2017-02-22 17:26 ` Catonano
2017-02-26 11:10 ` Pjotr Prins
2017-02-26 19:02 ` Christopher Allan Webber
2017-02-28 6:25 ` gnu-patches back log Pjotr Prins
2017-02-28 11:14 ` Hartmut Goebel
2017-03-01 5:23 ` Pjotr Prins
2017-03-01 6:16 ` Leo Famulari [this message]
2017-03-01 8:17 ` Pjotr Prins
2017-03-01 10:42 ` Andy Wingo
2017-03-01 11:17 ` Pjotr Prins
2017-03-01 12:48 ` Thomas Danckaert
2017-03-01 15:06 ` Pjotr Prins
2017-03-01 13:14 ` Leo Famulari
2017-03-01 14:45 ` Pjotr Prins
2017-03-01 15:51 ` Leo Famulari
2017-03-01 16:07 ` Pjotr Prins
2017-03-01 23:08 ` Catonano
2017-03-07 12:09 ` Ricardo Wurmus
2017-03-13 17:37 ` Catonano
2017-03-01 14:16 ` ng0
2017-03-01 13:14 ` John Darrington
2017-03-06 16:14 ` Ludovic Courtès
2017-03-07 11:44 ` Hartmut Goebel
2017-03-11 21:30 ` Ludovic Courtès
2017-03-25 12:37 ` How to M-x debbugs-gnu with new guix-patches Catonano
2017-03-25 21:45 ` Ludovic Courtès
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
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170301061625.GA11982@jasmine \
--to=leo@famulari.name \
--cc=guix-devel@gnu.org \
--cc=pjotr.public12@thebird.nl \
/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 public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).