Bug 108750

Summary: Add Antoine Quint to the list of committers
Product: WebKit Reporter: Antoine Quint <graouts>
Component: New BugsAssignee: Antoine Quint <graouts>
Status: RESOLVED INVALID    
Severity: Normal CC: abarth, dino, dpranke, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Description Antoine Quint 2013-02-02 05:24:55 PST
Add Antoine Quint to the list of committers
Comment 1 Antoine Quint 2013-02-02 05:25:51 PST
Created attachment 186226 [details]
Patch
Comment 2 WebKit Review Bot 2013-02-02 06:11:24 PST
Comment on attachment 186226 [details]
Patch

Clearing flags on attachment: 186226

Committed r141693: <http://trac.webkit.org/changeset/141693>
Comment 3 WebKit Review Bot 2013-02-02 06:11:27 PST
All reviewed patches have been landed.  Closing bug.
Comment 4 Alexey Proskuryakov 2013-02-02 09:36:33 PST
Rolled out in r141698. It is not allowed to use commit queue to become a committer, this change must be landed manually (no review needed, no bugzilla bug needed).

This proves that Apple actually received committer agreement, and created an account. This admittedly makes less sense for Apple employees, but doing this differently is misleading.
Comment 5 Dean Jackson 2013-02-02 14:51:23 PST
(In reply to comment #4)
> Rolled out in r141698. It is not allowed to use commit queue to become a committer, this change must be landed manually (no review needed, no bugzilla bug needed).
> 
> This proves that Apple actually received committer agreement, and created an account. This admittedly makes less sense for Apple employees, but doing this differently is misleading.

No problem. I'll land manually.
Comment 6 Dean Jackson 2013-02-02 15:03:52 PST
Landed in http://trac.webkit.org/changeset/141707
(even though we don't need a bugzilla)
Comment 7 Eric Seidel (no email) 2013-02-02 16:40:54 PST
(In reply to comment #6)
> Landed in http://trac.webkit.org/changeset/141707
> (even though we don't need a bugzilla)

The point is that Antoine is supposed to land it to prove he has rights. :)
Comment 8 Dean Jackson 2013-02-02 23:48:43 PST
(In reply to comment #7)
> (In reply to comment #6)
> > Landed in http://trac.webkit.org/changeset/141707
> > (even though we don't need a bugzilla)
> 
> The point is that Antoine is supposed to land it to prove he has rights. :)

OMG! Now I get it!! I'm not sure what was wrong with my brain that caused me to not understand this :)

Honestly, I think this was a senior moment.