From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5864 invoked by alias); 27 May 2014 06:27:19 -0000 Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org Received: (qmail 5842 invoked by uid 89); 27 May 2014 06:27:19 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.8 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-Spam-User: qpsmtpd, 2 recipients X-HELO: mail-we0-f172.google.com Received: from mail-we0-f172.google.com (HELO mail-we0-f172.google.com) (74.125.82.172) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Tue, 27 May 2014 06:27:17 +0000 Received: by mail-we0-f172.google.com with SMTP id k48so9096668wev.17 for ; Mon, 26 May 2014 23:27:14 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.180.106.194 with SMTP id gw2mr34805566wib.47.1401172034220; Mon, 26 May 2014 23:27:14 -0700 (PDT) Received: by 10.195.12.43 with HTTP; Mon, 26 May 2014 23:27:14 -0700 (PDT) In-Reply-To: <87a9a3spu4.fsf@kepler.schwinge.homeip.net> References: <20140523211338.GK12497@spoyarek.pnq.redhat.com> <87a9a3spu4.fsf@kepler.schwinge.homeip.net> Date: Tue, 27 May 2014 06:53:00 -0000 Message-ID: Subject: Re: patchwork.sourceware.org is live! From: Siddhesh Poyarekar To: Thomas Schwinge Cc: Siddhesh Poyarekar , gdb@sourceware.org, GNU C Library , carlos@redhat.com, fche@redhat.com, gbenson@redhat.com Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-SW-Source: 2014-05/txt/msg00074.txt.bz2 On 27 May 2014 11:52, Thomas Schwinge wrote: >> close patches you know you have already pushed into master. > > Does that mean changing the state to =C2=BBAccepted=C2=AB (or =C2=BBCommi= tted=C2=AB, once > that has been added), or ticking the =C2=BBArchived=C2=AB box, or both? = What's the > meaning of the =C2=BBArchived=C2=AB box? Only change to Accepted for now, since nobody has commented on my suggestion to add a new Committed status. I have no idea what the Archived checkbox does :) > So, does patchwork's =C2=BBDelegate to=C2=AB Patch Property equal Bugzill= a's > =C2=BBAssigned To=C2=AB field? And, it is not totally clear to my why th= e Patch > Review Workflow suggests to =C2=BBnot change the status of the patch=C2= =AB if > =C2=BBchanging the Delegate to value to their patchwork username=C2=AB? That suggestion is no longer necessary. I have added the 'Under Review' and 'Change Requested' statuses to the default view, so changing state should not make those patches disappear from the queue unless the state is Accepted, Superseded, RFC or Rejected. > "In a perfect world", after a patch review, during which =C2=BBDelegate t= o=C2=AB > 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 > =C2=BBAccepted=C2=AB? Yes, that would be good idea. Siddhesh --=20 http://siddhesh.in