This is the mail archive of the gdb@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: patch review time


> Date: Fri, 4 Mar 2005 09:26:00 -0500
> From: Bob Rossi <bob@brasko.net>
> Cc: gdb@sources.redhat.com
> 
> On Fri, Mar 04, 2005 at 09:33:25AM +0200, Eli Zaretskii wrote:
> > > Date: Thu, 3 Mar 2005 10:00:22 -0500
> > > From: Bob Rossi <bob@brasko.net>
> > > 
> > > I understand that people are busy, which is the very reason my patches
> > > take so long to create. However, from my perspective it simply takes to long, 
> > > to get patch reviews done, except of course doco reviews from Eli :).
> > 
> > I think I generally review all patches within my responsibility and/or
> > area of expertise with the same speed, not only the doco ones.
> 
> I think you are missing the point here Eli.

No, I'm not: _you_ are.

All I wanted to say that _my_ review time is short no matter if it's
with documentation or code patches.  In other words, I was protesting
against your saying that just my _doco_ patches are fast enough.

> Even in that respect, Andrew himself stated, 
> 
>    It's taken us three months to get through files a-c, simple math tells
>    us that at that rate we'll finish sometime on '07.  I don't think so.

Please don't start that again.  I told you right there and then that
the i18n changes Andrew was referring to were going slow because of
those who made changes, not because of the review process, since my
comments on the i18n-related patches were posted at most a couple of
days after the RFA, and normally just a few hours after the RFA.

> I'm calling for some type of action that will allow me to get some
> real work done on GDB.

Unfortunately, your changes are outside my authority and pretty much
outside my expertise.  Otherwise, they would have been reviewed eons
ago.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]