From: Eli Zaretskii <eliz@is.elta.co.il>
To: Jim Blandy <jimb@zwingli.cygnus.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: Apology, and the patch review process
Date: Wed, 13 Jun 2001 08:59:00 -0000 [thread overview]
Message-ID: <Pine.SUN.3.91.1010613185614.19790F-100000@is> (raw)
In-Reply-To: <20010613145023.5059E5E9CB@zwingli.cygnus.com>
On Wed, 13 Jun 2001, Jim Blandy wrote:
> I understand that the patch review process is losing --- perhaps
> symbol table patches have fared worse than patches to other areas.
> I'd honestly welcome constructive suggestions for how to improve this.
I don't have any magic wand to offer, but one way to improve things is to
(1) reply to posted patches quickly, even if the reply just says ``noted,
will get to it soon''; and (2) review large patches in small chunks and
publish any requests to modify the reviewed parts as soon as you can say
something useful. That still leaves the more general design issues that
cannot be reviewed without allocating significant time, but at least it
gets the minor boring issues such as formatting, ChangeLog entries, etc.
out of your way.
next prev parent reply other threads:[~2001-06-13 8:59 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-06-13 7:49 Jim Blandy
2001-06-13 8:59 ` Eli Zaretskii [this message]
2001-06-13 10:14 ` Patch " Andrew Cagney
2001-06-13 11:28 ` Christopher Faylor
2001-06-13 13:02 ` Andrew Cagney
2001-06-13 13:14 ` Christopher Faylor
2001-06-13 14:49 ` Christopher Faylor
2001-06-13 22:26 ` Daniel Berlin
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=Pine.SUN.3.91.1010613185614.19790F-100000@is \
--to=eliz@is.elta.co.il \
--cc=gdb-patches@sources.redhat.com \
--cc=jimb@zwingli.cygnus.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