[webkit-dev] 194 bugs in pending-commit

Antonio Gomes tonikitoo at gmail.com
Sat Jun 18 21:01:22 PDT 2011


IIRC, Mozilla's bugzilla can "hide" obsolete patches (?). If so, why can not
webkit's bugzilla?

I actually do not like the way the review flags are cleared today only in
order to make the tools and pending-xxx pages happier. IMO the review flags
give much about the history of the bug. In that matter, I dislike
webkit-patch's ways of clearing "r-" flags of patches while it marks it as
obsolete and uploads a new one. Reason: an easy-to-see r-'ed patch is very
helpful to me to understand the chronological progresses in the bug.

What is the reason for clearing r- flag while uploading a new one, instead
of only making it obsolete?

On Sat, Jun 18, 2011 at 2:23 AM, Adam Barth <abarth at webkit.org> wrote:

> On Fri, Jun 17, 2011 at 11:13 PM, Peter Kasting <pkasting at chromium.org>
> wrote:
> > On Fri, Jun 17, 2011 at 10:56 PM, Adam Barth <abarth at webkit.org> wrote:
> >> 2) Mark the patch as obsolete / clear the review flag if we're not
> >> going to land the patch.
> >
> > Does the slash mean "do both"?  I
> > have https://bugs.webkit.org/show_bug.cgi?id=47036 on that list and the
> only
> > r+ed patch on it is already marked obsolete.
>
> Yeah, Bugzilla kind of sucks.  That page isn't smart enough to hide
> the obsolete patches.  If you have EditBugs, you can run "webkit-patch
> clean-pending-commit", which will automatically remove the review
> flags from obsolete patches.  Eric and I have been meaning to having
> one of the bots do that periodically, but we haven't set that up yet.
>
> Adam
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>



-- 
--Antonio Gomes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20110619/59ff09bb/attachment.html>


More information about the webkit-dev mailing list