Teach validate-committer-lists how to validate committers bugzilla_email()

[Eric Seidel (no email)]

Teach validate-committer-lists how to validate committers bugzilla_email()

[Eric Seidel (no email)]

Created attachment 93165 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 93165 [details]
Patch

Found a bug, new patch coming.

[Eric Seidel (no email)]

Created attachment 93174 [details]
Patch

[David Levin]

Comment on attachment 93174 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=93174&action=review

You could cq+ it. Depends on how you feel about the typo.

> Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py:195
> +    # This only works if your account has edituser privilages.

typo: privileges

[Eric Seidel (no email)]

(In reply to comment #4)
> (From update of attachment 93174 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=93174&action=review
> 
> You could cq+ it. Depends on how you feel about the typo.
> 
> > Tools/Scripts/webkitpy/common/net/bugzilla/bugzilla.py:195
> > +    # This only works if your account has edituser privilages.
> 
> typo: privileges

Turns out it was spelled wrong in a second place!  The message that the commit-queue spams to bugs when users don't have cq privileges.  I guess I never knew how to spell the word. :(

[Eric Seidel (no email)]

Created attachment 93179 [details]
Patch for landing

[Eric Seidel (no email)]

Thankfully we have apply-from-bug and land-safely, so fixing typos is easy. :)

[WebKit Commit Bot]

Comment on attachment 93179 [details]
Patch for landing

Clearing flags on attachment: 93179

Committed r86288: <http://trac.webkit.org/changeset/86288>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.