Re: git-send-email

[Junio C Hamano]

lsorense@xxxxxxxxxxxxxxxxxxx (Lennart Sorensen) writes:

> Split: 25 patch sets.
> Shallow: 56 patch sets.
> Deep: 6 patch sets.

This is an interesting datapoint.

FWIW, for the git mailing list, the stats are like Shallow=30 vs Deep=50
for a series longer than 3 patches (I stopped counting after looking at
5200 messages).

But I think these numbers are flawed, as it is very likely that much more
people on the git mailing list are using git-send-email, while on the
kernel list, a lot of patches do not even come from git.  To put it
another way, we shouldn't take the numbers from the git mailing list
samples as an indicator on the style preferred by the *readers*.  The
numbers that can be counted only show what the *senders* thought would be
acceptable to the readers, nothing more.

If you are advocating to change the default, please take the discussion to
the git mailing list.  As I already said, I personally am in favor of the
shallow kind, but I do not run dictatorship over there.

Also we cannot make a change based solely on what the kernel people have
recently done these days; it is not year 2005 anymore and git is used by
other projects as well.

Just like the kernel folks take regressions seriously to the point to say
breaking one person's working setup is worse than fixing a known bug that
affects many more people, I take changing any default seriously.  Most
discussions to change the default come from poeple who do not like the
default for obvious reasons, and it is difficult to judge if there is a
silent majority that is content with the current behaviour, or everybody
is unhappy but only some care deeply enough to make loud noises about it.
We cannot tell these two cases apart very easily by only listening to the
loudness of voices of complaining people.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/