[RFC] Patchsets in a single mail? [Re: [PATCH 01/55] Add simple subunit runner outputting subunit v1.]
Ira Cooper
ira at samba.org
Mon Feb 16 05:57:46 MST 2015
On Sun, Feb 15, 2015 at 08:36:28PM +0100, David Disseldorp wrote:
> On Sun, 15 Feb 2015 19:46:00 +0100, Michael Adam wrote:
>
> > Well the granularity is not lost if I attach the whole
> > patchset to a mail. The individual patches are not squashed
> > or so. But it is possible (in contrast to the one mail per
> > commit approach) to reply in a broader context.
>
> The "git send-email --compose" introductory mail is ideal for a broad
> context reply.
Seems sane...
> > I can't emphasise enought the point that for me a patchset
> > sent to the list should be form a logical unit. Some comments
> > may apply to several patches in the series and may provoke
> > changes to several commits. This is just intractable for my
> > poor brain in the multi-mail mode. But I am trying. ;)
> >
> > Again, I guess we should just continue as is. Everybody sends the
> > patches the way (s)he likes (single/multi mail). People are just
> > different, and so are their preferences.
>
> In the end I want to make it as easy for reviewers as possible, so if
> the majority prefer not to have inline patch-sets, then I guess I'll
> have to consider changing.
Have we done a full serious review with comments like LKML (without the
flaming) of a patchset this way?
I'm open to changing my tooling if the new approach beats the old one. :)
Thanks,
-Ira
More information about the samba-technical
mailing list