[PATCH 1/2] Use an explicit initial default patch state

Dirk Wallenstein halsmit at t-online.de
Mon Apr 30 19:31:33 EST 2012


On Mon, Apr 30, 2012 at 09:57:04AM +0200, Wolfgang Denk wrote:
> Dear Jeremy,
> 
> In message <4F9E34EF.5030005 at ozlabs.org> you wrote:
> > 
> > > This will make editing states through the admin UI less error-prone, and
> > > will facilitate parsing patches when relying on a default state, as well as
> > > testing.
> > >
> > > Use the explicit default state when parsing mails.
> > >
> > > Signed-off-by: Dirk Wallenstein<halsmit at t-online.de>
> > 
> > 
> > Excellent, thanks for your work on these. I've committed and pushed, 
> > plus a couple of minor changes.
> > 
> > Wolfgang - these changes are now live the patchwork.ozlabs.org site too 
> > now :)
> 
> Thanks a lot!
> 
> Um... but is it really working?
> 
> I added "X-patchwork-state: Accepted" to a reply on a patch; my reply
> was correctly added to the respective PW entry
> (http://patchwork.ozlabs.org/patch/155545/) but the status did not
> change.
> 
> Am I doing something wrong?

You can only request an initial state (e.g. RFC) when submitting the
patch -- in the same mail as the patch.  Changing the state later on by
mail is problematic due to the lack of authentication, I'd say.  But you
can do that with pwclient.

-- 
Cheers,
  Dirk


More information about the Patchwork mailing list