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. --
| Rafael J. Wysocki | [Bug #10493] mips BCM47XX compile error |
| Ingo Molnar | [patch 02/13] syslets: add syslet.h include file, user API/ABI definitions |
| Greg KH | [GIT PATCH] driver core patches against 2.6.24 |
| Andrea Arcangeli | [PATCH 00 of 11] mmu notifier #v16 |
git: | |
| David Miller | Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock(). |
| Gerrit Renker | [PATCH 27/37] dccp: Integration of dynamic feature activation - part 2 (server side) |
| Linus Torvalds | Re: [GIT]: Networking |
| Mark Lord | Re: [BUG] New Kernel Bugs |
