patchwork feature request: support for patch series cover letters?

Dirk Wallenstein halsmit at t-online.de
Fri Jan 7 22:02:27 EST 2011


On Fri, Jan 07, 2011 at 06:43:58PM +0800, Jeremy Kerr wrote:
> Hi Peter,
> 
> > Most qemu patches get sent to the list as multi-patch sets with
> > an initial "[PATCH 00/nn]" "cover letter". Unfortunately patchwork
> > doesn't do anything with the cover letter, so it just shows the
> > individual sub-patches.
> > 
> > Would it be possible for patchwork to also handle the 'cover letter'
> > email, so that:
> >  * there's a single URL you can refer to for the whole patchset
> > (which could link to the patch URLs for all its parts)
> >  * patchwork recorded the overall rationale for the patchset as
> > documented in the cover letter (and discussion on it which was
> > sent as a reply to the cover letter)
> > ?
> > 
> > (Perhaps patchsets should be automatically turned into
> > patchwork bundles? I'm not sure.)
> 
> Yeah, this is something I'd like to do too. I see bundles as user-created, so 
> would prefer some other system of grouping.
> 
> I've previously thought this could be solved by adding 'relations' between 
> patches, that can be used to express all sorts of stuff, like:
> 
>  * patch X replaces patch Y,
>  * patch X is a add-on to patch Y, or
>  * patch X is in the same series as patch y
> 
> However, the patch-cover-letter case might break the last case (and isn't 
> really the best way to express grouping).
> 
> So how does this sound (just taking some initial notes here):
> 
> Add patch 'groups', which have:
> 
>   * A group leader - the cover letter, or the first patch in the series if
>       there is none.
> 
>   * Member patches
> 
> Some bits to consider:
> 
> Parsing: currently, if a mail does not contain a patch, we see if it's a 
> follow up to an existing patch, and discard it if not. This probably won't 
> work for cover letters, as it's likely that we haven't parsed any patches from 
> the series yet. For the cover letters, we'd need to keep a record of 
> 'potential' cover letters, and try and match it up with patches that come in 
> later.

Would it be a problem/overkill to record every mail thread? For example,
a user notifies about a bug and then there are several patches attached
to the thread. It would be nice to have a grouping for those, too.

-- 
Greetings,
Dirk


More information about the Patchwork mailing list