Re: Adding Series Dependency to Patchwork

[Patrick Robb <probb@iol.unh.edu>]

On Thu, Jul 25, 2024 at 5:57 AM Ferruh Yigit <ferruh.yigit@amd.com> wrote:
>
> On 7/24/2024 4:07 PM, Adam Hassick wrote:
> > On Tue, Jul 23, 2024 at 12:08 PM Ferruh Yigit <ferruh.yigit@amd.com> wrote:
> > An unrelated note about the v2: Earlier I mentioned that we might
> > support cover letter IDs to reference a series dependency. I've
> > decided to forgo that feature because developers do not always
> > resubmit cover letters when they submit new versions of a patch
> > series. For example, if they depend on the v3 of a patch and they use
> > the cover letter message ID submitted with v1, that will introduce a
> > dependency on v1. This could lead to some confusion.
> >
>
> As far as I can see if a patch series has cover letter, more likely it
> will have cover letter in next version.

Okay, this was not my impression, so thanks for setting the record straight.

Sorry Adam for me giving you the wrong idea about common cover letter
resubmission practices. :)

>
> If cover letter is not supported, and dependent series has a cover
> letter, is the expectation to add first patch of the series as
> dependency skipping cover letter? This maybe also confusing.

Yes this would work, but I can see how having this expectation may be
confusing. So, we can abandon this idea.

>
> For the sample you give, if dependent patch decided to drop cover letter
> in v3, won't it work to provide first patch of v3 as dependency?
>

Yes.