Bug 64165
| Summary: | Rollout patches created by sheriffbot say that sheriffbot is the patch author, but should say that some contributor is the author | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Adam Roben (:aroben) <aroben> |
| Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | abarth, eric |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Adam Roben (:aroben)
When you ask sheriffbot to create a rollout patch (via "sheriffbot rollout"), sheriffbot itself is used as the patch author in the ChangeLog. Since sheriffbot is really just a proxy for the requester, it would make more sense if the requester were used as the author.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Adam Barth
Historically, we've had the perspective that your IRC is completely untrusted because spoofing identity in IRC is very easy. Another option is to have the person who marks the patch cq+ be the patch author.
Adam Barth
*your IRC identity
Adam Roben (:aroben)
(In reply to comment #1)
> Historically, we've had the perspective that your IRC is completely untrusted because spoofing identity in IRC is very easy.
Makes sense.
> Another option is to have the person who marks the patch cq+ be the patch author.
Maybe that's the best option.