Re: patchwork.sourceware.org is live!

[Thomas Schwinge <thomas@codesourcery.com>]

[-- Attachment #1: Type: text/plain, Size: 1125 bytes --]

Hi!

On Sat, 24 May 2014 02:43:38 +0530, Siddhesh Poyarekar <siddhesh@redhat.com> wrote:
> patchwork.sourceware.org

> close patches you know you have already pushed into master.

Does that mean changing the state to »Accepted« (or »Committed«, once
that has been added), or ticking the »Archived« box, or both?  What's the
meaning of the »Archived« box?

> Please also take time to review the workflow for patch reviews:
> 
> https://sourceware.org/glibc/wiki/Patch%20Review%20Workflow

So, does patchwork's »Delegate to« Patch Property equal Bugzilla's
»Assigned To« field?  And, it is not totally clear to my why the Patch
Review Workflow suggests to »not change the status of the patch« if
»changing the Delegate to value to their patchwork username«?

"In a perfect world", after a patch review, during which »Delegate to«
has been set to the reviewer(s), should it then be changed to the person
who is responsible for committing the patch (which may be the submitter,
or the reviewer, or someone else), together with changing the state to
»Accepted«?


Grüße,
 Thomas

[-- Attachment #2: Type: application/pgp-signature, Size: 472 bytes --]