public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Zack Weinberg <zack@codesourcery.com>
To: Phil Edwards <pedwards@disaster.jaj.com>
Cc: Joe Buck <jbuck@synopsys.COM>, gcc@gcc.gnu.org
Subject: Re: patch tracking
Date: Fri, 14 Sep 2001 15:53:00 -0000	[thread overview]
Message-ID: <20010914155324.H443@codesourcery.com> (raw)
In-Reply-To: <20010914171648.A2363@disaster.jaj.com>

On Fri, Sep 14, 2001 at 05:16:48PM -0400, Phil Edwards wrote:
> On Fri, Sep 14, 2001 at 02:02:54PM -0700, Zack Weinberg wrote:
> > I have not decided whether the tracker should generate one big nag
> > message per week, or follow up to each individual thread.  The former
> > would be less obnoxious, but then obnoxiousness is kind of the point.
> 
> How about individual messages to the people privately, and one big summary
> to the list?  If I were to receive N nag messages, it would be easier to
> delete each message as I review the patch.  With a big summary, I must
> remember which ones I have since reviewed; with individual nags, I can
> simply look at which messages remain in my mailbox.

Good idea, but which people do you mean?  One of the contributing
factors to this problem is that for large parts of the compiler there
is no one person who should be reminded to review patches.  Either
there is a group, and which one do you pick? or there is no one at
all, and you're left flooding the blanket write people, who are
already overloaded.

Better just to post nags to the list, I think.

> Although I must point out that the problem of patch queuing has been solved
> before.  I strongly recommend adopting something used on sourceforge or
> mozilla or whatever for GCC's purposes.  Else we're right back where we
> started:  people needing to be freed up for patch review (Zack) are getting
> stuck with other duties (writing patch queuing nagging programs).  Granted,
> Zack's doing this already on his own, but it made a good example. *grin*

I don't know what sourceforge or mozilla or whatever use, but I'm
willing to bet it's web-centric.  Maybe I'm strange but I infinitely
prefer systems which are email-centric.

Also, I'm not usually the person reviewing patches.  I'm usually the
person waiting for patches to be reviewed.  (We don't get that many
patches for cpplib over the transom, y'know.)  If I can use some of
that waiting time to avoid having to wait so much in the future, well,
why not?

zw

  parent reply	other threads:[~2001-09-14 15:53 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-09-14 12:02 Joe Buck
2001-09-14 12:21 ` Diego Novillo
2001-09-14 12:24 ` egcs
2001-09-14 12:28 ` Joseph S. Myers
2001-09-14 12:35   ` Phil Edwards
2001-09-14 14:03 ` Zack Weinberg
2001-09-14 14:16   ` Phil Edwards
2001-09-14 14:51     ` Joe Buck
2001-09-14 15:53     ` Zack Weinberg [this message]
2001-09-14 14:53   ` Joe Buck
2001-09-14 15:48     ` Zack Weinberg
2001-09-14 14:24 ` Gerald Pfeifer
2001-09-15  3:07 ` Bernd Schmidt
2001-09-16 14:46   ` Joe Buck
2001-09-18 16:18     ` Marc Espie
2001-09-18  9:49 dewar

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=20010914155324.H443@codesourcery.com \
    --to=zack@codesourcery.com \
    --cc=gcc@gcc.gnu.org \
    --cc=jbuck@synopsys.COM \
    --cc=pedwards@disaster.jaj.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.
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).