Re: whomto.pl -- finding out whom to send patches to

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Vegard Nossum <vegard.nossum@...>
Cc: <linux-kernel@...>, Jan Engelhardt <jengelh@...>, Sverre Rabbelier <alturin@...>, Joe Perches <joe@...>, <git@...>
Date: Thursday, May 29, 2008 - 7:33 pm

Vegard Nossum <vegard.nossum@gmail.com> writes:


"git blame" does not give irrelevant commits in its output, with or
without --incremental.  Perhaps you were thinking about the "oops, earlier
one was wrong, here are the corrections" behaviour of "git log
--early-output", which is an unrelated mechanism in a different command.

But I have to wonder why you used --incremental and not --porcelain
format, the latter of which is more compact and is designed for parsing by
tools.

I also have to wonder why you did not use -M, -C, and/or -w, if you used
blame to find the true origin of lines that are involved.

Unless the patch is truly about a narrow region of a handful files
(e.g. micro-optimizing the implementation of a single function without
changing its external interface at all, or fixing an off-by-one error in a
group of functions that do similar things), I suspect that it would make
more sense to use "git shortlog --no-merges -- paths" to get the list of
people who are involved in the general area, even though they may not have
been involved in particular _lines_ that the patch touches.  For example,
if a patch changes the lines in a function's implementation, you would
want input not only from the people who improved the implementation of the
function over the years, but more from the people who care about the
calling sites of that function the patch is touching.

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
whomto.pl -- finding out whom to send patches to, Vegard Nossum, (Thu May 29, 5:00 pm)
Re: whomto.pl -- finding out whom to send patches to, Andrea Righi, (Fri May 30, 3:58 am)
Re: whomto.pl -- finding out whom to send patches to, Vegard Nossum, (Sat May 31, 6:03 am)
Re: whomto.pl -- finding out whom to send patches to, Junio C Hamano, (Thu May 29, 7:33 pm)
Re: whomto.pl -- finding out whom to send patches to, Vegard Nossum, (Sat May 31, 6:33 am)
Re: whomto.pl -- finding out whom to send patches to, Jesper Juhl, (Thu May 29, 6:19 pm)
Re: whomto.pl -- finding out whom to send patches to, Joe Perches, (Thu May 29, 2:20 pm)