..snip..
> I'm less concerned with the capabilities of other clients than I am for
> how this negatively impacts the current workflow of people on this
> list, which is what I'm trying to figure out. How does doing this
> making things more difficult for people here? And specifically Daniel.
Don't know what flow Daniel is using, but my email reader is mutt.
Once I am comfortable with the patches I usually save them in a mailbox
(.mbox) and do 'git am -s' on them and then kickoff a workflow.
Having multiple threads within threads makes my life as maintainer
more difficult as I have to manually split out the right version and
save it.
I suppose I can use the 'tag-save' option and save the ones
with a certain version to a mailbox which can add another step in this.
And then I am worried I missed something :-(
>
> If this causes tools to break, then that's certainly a good reason to
> avoid changing things. If it makes Daniel's workflow more burdensome
> because he has a client that can not collapse threads at all, then
> that's a valid concern too. I'm currently failing to see as valid
> concerns: "because traditionally we've never done it that way" or "I
> don't like seeing patchset revisions together".
>
> From a structuring data point of view, it makes more sense (to me) to
> link patchset revisions together. Now it might also be true that
> looking at previous patchsets is very uncommon and so its not a huge
Maintainers trust that folks add right after the ---
v6: Fixed up spaces
Fixed up XYZ
and there is no need to go back to v5 or earlier.
> benefit to link them together. But that's not a reason why it shouldn't
> be done.
So ... making the life of maintainer and as simple as possible makes
it easier and faster to get patches upstream.
That is what LKML has taught a lot of us and that is probably why
many of us are used to this workflow.
>
> Perhaps with a better understanding of the harm, I could come to a
> different opinion. And sincerely, thank you for the feedback.
>
> Glenn
>
> > Kind regards,
> >
> > Paul
> >
> >
> > [1]:
> > https://lists.gnu.org/archive/html/grub-devel/2021-03/threads.html
>
> _______________________________________________
> Grub-devel mailing list
> [email protected]
> https://lists.gnu.org/mailman/listinfo/grub-devel
_______________________________________________
Grub-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/grub-devel
