[webkit-dev] Why does the commit queue clear the review flag on every patch?

[Eric Seidel]

I was not aware bugzilla was sending such emails.  That's unfortunate.

Certainly it can be changed.  It used to just close bugs which only
had one patch, but the multi-patch/single-patch code paths were
unified a while back.  The code in question can be found here:
http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/bugzilla-tool#L328

The clear_attachment_flags call is here:
http://trac.webkit.org/browser/trunk/WebKitTools/Scripts/bugzilla-tool#L339

We can make it smarter to just close bugs with one patch.  However
you'll still get an email about commit-queue being canceled when it
removes commit-queue+.  You should feel encouraged to file a bug about
this (and any other) desired change to the commit-queue.


I believe others have expressed a desire not to obsolete patches after
they've been landed, so we went with clearing flags instead.  If folks
have opinions on what approach to take to indicate that a patch has
been landed I'm interested to hear them! :)

-eric

On Fri, Sep 11, 2009 at 10:43 AM, Darin Adler <darin at apple.com> wrote:
> It seems that the commit queue clears the review flag on each patch as it
> lands it. I can see how this could be useful for the cases of one bug with a
> lot of patches, but for the usual case of one bug one patch, it seems
> non-helpful. I find it confusing to look back at such bugs and see the
> patches without the reviewer name in the attachment list. And I get mail
> saying "review canceled" for each one of these.
>
> Can we do better?
>
>    -- Darin
>
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>