[PATCH v2 3/6] parser: Add series parsing
Russell Currey
ruscur at russell.cc
Mon Sep 5 14:31:45 AEST 2016
On Mon, 2016-09-05 at 14:25 +1000, Andrew Donnellan wrote:
> On 05/09/16 14:19, Russell Currey wrote:
> >
> > In testing v2 I found a weird issue I didn't find before.
> >
> > If you send a new series in reply to the cover letter of a previous series,
> > it
> > appends the patches to the previous series. This is rather confusing as you
> > would think any patches sent in reply to a cover letter would belong to that
> > series, but they clearly should be treated differently in some cases, as you
> > can
> > see in the screenshot below:
> >
> > https://i.imgur.com/8Yi9IjR.png
>
> Or in reply to patch 1 of the initial series - the one in your
> screenshot doesn't have a cover letter.
Yeah, you're right, good catch.
>
> >
> > Maybe use a reset in numbering? This would be harder to parse if both
> > series
> > had the same number of patches, I'm imagining something like the following:
> >
> > - [1/3] patch sent in reply to cover letter, new series (#1)
> > - [2/3] patch sent in reply to cover letter, append to #1
> > - [3/3] patch sent in reply to cover letter, append to #1
> > - [1/3] patch sent in reply to cover letter, new series (#2)
> > - [2/3] patch sent in reply to cover letter:
> > - see that series #2 doesn't have a 2/3
> > - see that the date of the patch means it's probably for series #2
> > - append to series #2
> > - [3/3] is the same as above.
>
> This is the sanest approach I think. For each 1/N patch, create a new
> series.
>
More information about the Patchwork
mailing list