[Webkit-unassigned] [Bug 71690] Add cc-bugs group to watch changes in chromium graphics
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Nov 9 11:36:08 PST 2011
https://bugs.webkit.org/show_bug.cgi?id=71690
--- Comment #19 from David Levin <levin at chromium.org> 2011-11-09 11:36:07 PST ---
(In reply to comment #18)
> (From update of attachment 114319 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=114319&action=review
>
> > Tools/Scripts/webkitpy/common/config/committers.py:90
> > +# This is a list of email addresses who have bugzilla accounts but are not
>
> A list is not a "who", is it?
>
> > Tools/Scripts/webkitpy/common/config/committers.py:95
> > + Contributor("Chromium Compositor Bugs", ["cc-bugs at google.com"], "cc-bugs"),
>
> This should be Account(...)
>
> > Tools/Scripts/webkitpy/common/watchlist/watchlistparser.py:158
> > - if not contributors.contributor_by_email(email):
> > + if not accounts.account_by_email(email):
> > cc_rule.remove_instruction(email)
> > self._log_error("The email alias %s which is in the watchlist is not listed as a contributor in committers.py" % email)
> > continue
>
> This code should additionally check that it's the first email, the bugzilla_email in the account. account_by_email will give you any account with that email, which might not be the bugzilla email for the account.
And please add a simple test to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers_unittest.py?
Thanks so much for doing this!
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list