[Gs-code-review] porting the regression test tool to Windows
L. Peter Deutsch
ghost at aladdin.com
Tue Feb 19 07:39:15 PST 2002
>> I like these changes. Please delay implementing them until after Jack
>> has approved the new toolbin/tests directory, so that the changes can be
>> made to the new files.
>
> It's too hard for me to wait a weeks before applying each patch.
> Too many local versions. Too big delays and too slow development.
> I'm tired to accommodate obsolete patches.
> I've committed it, while my patch corresponds to the repository state.
> I think the renaming a directory isn't complicated after this.
Igor,
I did not see any message on gs-code-review indicating that your patches had
been approved. (I am not your official reviewer.) Did I miss it?
I too dislike waiting a week or sometimes longer to apply patches that I
have posted to gs-code-review, especially if I am working on more than one
project, but I consider this a reasonable price for having the review
process work well. Part of the "social contract" for gs-code-review is that
reviewers will review patches quickly. If a patch I have posted is not
reviewed within 1 week, I send a reminder to Raph, who is my official
reviewer.
Since your patches affect the same files that Jack and I have been working
on, I have to handle the merge process. Fortunately, the patches are small,
so even though about half of them create conflicts that I have to fix
manually, it won't take me a lot of time.
In the future, please wait for review before committing patches.
Thanks -
L. Peter Deutsch | Aladdin Enterprises | 203 Santa Margarita Ave.
ghost at aladdin.com | http://www.aladdin.com | Menlo Park, CA 94025
The future of software is at http://www.opensource.org
More information about the gs-code-review
mailing list