From: Daniel Jacobowitz <drow@false.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: gdb@sourceware.org
Subject: Re: Maintainer policy for GDB
Date: Fri, 25 Nov 2005 21:03:00 -0000 [thread overview]
Message-ID: <20051125204347.GA7107@nevyn.them.org> (raw)
In-Reply-To: <u7jawze0w.fsf@gnu.org>
On Fri, Nov 25, 2005 at 10:08:31PM +0200, Eli Zaretskii wrote:
> > Date: Fri, 25 Nov 2005 11:04:54 -0500
> > From: Daniel Jacobowitz <drow@false.org>
> >
> > You see, I was thinking a couple of days, or up to a week.
>
> Two days is awfully too few, IMO. I could think of many reasons why I
> could be away of my mail for two days. Not everyone hacks GCC and GDB
> for their living and have an opportunity to read gdb-patches during
> office hours.
I apologize. I realize this is grammatically busted usage of English,
but I've never been able to break myself of it. Here's the one I was
using:
couple
n 1: a small indefinite number; "he's coming for a couple of
days"
i.e. I meant 3-5, not two, which I agree is far too short. I'm more
than comfortable with Joel's 7-10 days, also.
[By the way, I don't generally have an opportunity to read gdb-patches
during office hours either. I do it more than I feel I ought to.]
> > Do you want to be the one to explain to all the latter group "no,
> > sorry, we can't look at your patch for three weeks"?
>
> I think there's a misunderstanding: 3 weeks was suggested as a
> _timeout_, i.e. an extreme value beyond which we behave as if the
> responsible maintainer were not there. It is not suggested as the
> _average_ value. If, several months from now, we see that the average
> delay is anywhere near 3 weeks, I will be the first one to suggest we
> do something about it.
What are you suggesting doing with the current set of maintainers,
then? The fact remains that for most patch review, three weeks is
currently optimistic.
> > With just a week, it's easy to give the contributor feedback on the
> > style et cetera - which often takes a week anyway - while waiting
> > for comments from the responsible party.
>
> That's another misunderstanding: there's no need for the other
> maintainers to wait before they post comments about the proposed
> patches, not even for a minute. They could do that right away. One
> needs to wait only for the approval. Any other comments, style or
> otherwise, need not wait.
>
> In other words, the timeout is not a silence period during which no
> one can say anything about the proposed patch. It's the max time we
> give the responsible maintainer to review the patch and make up her
> mind whether to approve it.
Of course. But when the contributor asks us "OK, can it be applied
now", we have to answer "please let's wait 19 more days".
> > But alternatively, we could use a long timeout and an aggressive
> > policy for maintainers who time out repeatedly - politely remove
> > them from responsibility (shift into the authorized section). How
> > do you feel about that?
>
> Responsible maintainers that time out repeatedly should be asked to
> do better or to step down.
Great, I agree.
--
Daniel Jacobowitz
CodeSourcery, LLC
next prev parent reply other threads:[~2005-11-25 20:43 UTC|newest]
Thread overview: 101+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-11-17 4:48 Daniel Jacobowitz
2005-11-17 20:14 ` Eli Zaretskii
2005-11-17 21:10 ` Jim Blandy
2005-11-18 3:07 ` Daniel Jacobowitz
2005-11-18 3:26 ` Joel Brobecker
2005-11-18 3:30 ` Daniel Jacobowitz
2005-11-18 3:33 ` Joel Brobecker
2005-11-18 3:46 ` Wu Zhou
2005-11-18 11:09 ` Andrew STUBBS
2005-11-18 11:46 ` Eli Zaretskii
2005-11-18 11:59 ` Andrew STUBBS
2005-11-18 13:15 ` Eli Zaretskii
2005-11-18 15:26 ` Daniel Jacobowitz
2005-11-18 18:24 ` Eli Zaretskii
2005-11-18 18:44 ` Ian Lance Taylor
2005-11-18 18:51 ` Daniel Jacobowitz
2005-11-18 21:40 ` Eli Zaretskii
2005-11-18 21:46 ` Daniel Jacobowitz
2005-11-18 22:33 ` Eli Zaretskii
2005-11-18 22:41 ` Daniel Jacobowitz
2005-11-19 9:34 ` Eli Zaretskii
2005-11-18 21:51 ` Jim Blandy
2005-11-18 22:29 ` Eli Zaretskii
2005-11-19 0:34 ` Jim Blandy
2005-11-19 10:54 ` Eli Zaretskii
2005-11-21 7:52 ` Jim Blandy
2005-11-21 22:35 ` Eli Zaretskii
2005-11-18 22:46 ` David Carlton
2005-11-19 10:38 ` Eli Zaretskii
2005-11-23 1:28 ` David Carlton
2005-11-23 19:56 ` Eli Zaretskii
2005-11-23 20:13 ` Joel Brobecker
2005-11-24 4:51 ` Eli Zaretskii
2005-11-24 20:36 ` Joel Brobecker
2005-11-24 20:47 ` Eli Zaretskii
2005-11-24 21:20 ` Joel Brobecker
2005-11-25 3:07 ` Daniel Jacobowitz
2005-11-25 8:36 ` Christopher Faylor
2005-11-25 8:37 ` Eli Zaretskii
2005-11-25 17:07 ` Daniel Jacobowitz
2005-11-25 19:53 ` Joel Brobecker
2005-11-25 20:43 ` Eli Zaretskii
2005-11-25 20:10 ` Eli Zaretskii
2005-11-25 21:03 ` Daniel Jacobowitz [this message]
2005-11-25 21:38 ` Eli Zaretskii
2005-11-25 23:04 ` Daniel Jacobowitz
2005-11-25 23:42 ` Mark Kettenis
2005-11-26 0:03 ` Daniel Jacobowitz
2005-11-26 9:38 ` Eli Zaretskii
2005-11-26 9:31 ` Eli Zaretskii
2005-11-27 15:07 ` Daniel Jacobowitz
2005-11-28 8:51 ` Eli Zaretskii
2005-11-25 9:23 ` Eli Zaretskii
2005-11-25 16:04 ` Daniel Jacobowitz
2005-11-25 20:08 ` Eli Zaretskii
2005-11-26 7:28 ` Christopher Faylor
2005-11-26 15:18 ` Eli Zaretskii
2005-11-26 16:38 ` Daniel Jacobowitz
2005-11-23 20:41 ` Christopher Faylor
2005-11-24 4:56 ` Eli Zaretskii
2005-11-24 2:05 ` David Carlton
2005-11-24 6:17 ` Eli Zaretskii
2005-11-18 21:09 ` Eli Zaretskii
2005-11-18 21:32 ` Daniel Jacobowitz
2005-11-18 12:14 ` Eli Zaretskii
[not found] ` <8f2776cb0511162240q6f550008udda9803b5253fd88@mail.gmail.com>
2005-11-17 6:44 ` Fwd: " Jim Blandy
2005-11-17 14:04 ` Daniel Jacobowitz
2005-11-17 17:07 ` Jim Blandy
2005-11-17 20:38 ` Jim Blandy
2005-11-17 20:15 ` Eli Zaretskii
2005-11-17 20:16 ` Eli Zaretskii
2005-11-17 23:10 ` Joel Brobecker
2005-11-18 12:42 ` Eli Zaretskii
2005-11-18 15:05 ` Daniel Jacobowitz
2005-11-18 18:11 ` Eli Zaretskii
2005-11-18 17:53 ` Paul Gilliam
2005-11-18 18:36 ` Eli Zaretskii
2005-11-18 19:25 ` Joel Brobecker
2005-11-18 21:02 ` Paul Gilliam
2005-11-19 2:44 ` Christopher Faylor
2005-11-19 10:56 ` Eli Zaretskii
2005-11-19 17:05 ` Christopher Faylor
2005-11-19 19:39 ` Eli Zaretskii
2005-11-19 22:21 ` Christopher Faylor
2005-11-19 22:23 ` Christopher Faylor
2005-11-19 22:25 ` Christopher Faylor
2005-11-19 22:54 ` Eli Zaretskii
2005-11-19 22:55 ` Eli Zaretskii
2005-11-20 5:28 ` Joel Brobecker
2005-11-20 19:22 ` Eli Zaretskii
2005-11-20 21:55 ` Christopher Faylor
2005-11-20 22:01 ` Daniel Jacobowitz
2005-11-18 19:50 ` Joel Brobecker
2005-11-18 21:41 ` Eli Zaretskii
2005-11-17 23:52 ` Mark Kettenis
2005-11-18 21:51 ` David Carlton
2005-11-27 4:50 Michael Snyder
2005-11-27 4:59 ` Eli Zaretskii
2005-11-27 5:00 ` Ian Lance Taylor
2005-11-27 19:22 ` Eli Zaretskii
2005-11-27 19:18 ` Christopher Faylor
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=20051125204347.GA7107@nevyn.them.org \
--to=drow@false.org \
--cc=eliz@gnu.org \
--cc=gdb@sourceware.org \
/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).