On Wed, Apr 5, 2023 at 11:19 AM Jakub Kicinski <kuba@xxxxxxxxxx> wrote: > > On Wed, 5 Apr 2023 10:22:16 -0700 Andrii Nakryiko wrote: > > So I'm exclusively using `pw-apply -c <patchworks-url>` to apply > > everything locally. > > I think you can throw -M after -c $url? It can only help... :) > > > I'd expect that at this time the script would > > detect any Acked-by replies on *cover letter patch*, and apply them > > across all patches in the series. Such that we (humans) can look at > > them, fix them, add them, etc. Doing something like this in git hook > > seems unnecessary? > > Maybe mb2q can do it, IDK. I don't use the mb2q thing. > I don't think git has a way of doing git am and insert these tags if > they don't exist, in a single command. > > > So I think the only thing that's missing is the code that would fetch > > all replies on the cover letter "patch" (e.g., like on [0]) and just > > apply it across everything. We must be doing something like this for > > acks on individual patches, so I imagine we are not far off to make > > this work, but I haven't looked at pw-apply carefully enough to know > > for sure. > > The individual patches are handled by patchwork. > Ah, I see, so yeah, it would be harder to do in pw-apply then. > Don't get me wrong, I'm not disagreeing with you. Just trying to help > and point out existing workarounds.. Oh, absolutely, I think we are all on the same page here. Daniel mentioned -a in another reply, I didn't know about that and will use that for the time being, still will save a bunch of time next time. When I feel inspired to deal with a bit of bash, curl, and stuff, I'll see if we can extend `pw-apply -c` to do this automatically. Thanks for the discussion!
