From: Johannes Schindelin <Johannes.Schindelin@gmx.de> To: Igor Djordjevic <igor.d.djordjevic@gmail.com> Cc: Jeff King <peff@peff.net>, Eric Sunshine <sunshine@sunshineco.com>, Junio C Hamano <gitster@pobox.com>, Git List <git@vger.kernel.org>, Thomas Rast <tr@thomasrast.ch>, Thomas Gummerer <t.gummerer@gmail.com>, Ævar Arnfjörð Bjarmason <avarab@gmail.com>, Ramsay Jones <ramsay@ramsayjones.plus.com>, Stefan Beller <sbeller@google.com>, Jacob Keller <jacob.keller@gmail.com> Subject: Re: [PATCH v2 02/18] Add a new builtin: branch-diff Date: Mon, 21 May 2018 12:33:59 +0200 (DST) Message-ID: <nycvar.QRO.7.76.6.1805211153370.77@tvgsbejvaqbjf.bet> (raw) In-Reply-To: <3cefc6b3-3dbd-9cb1-20d0-193116191726@gmail.com> Hi Buga, On Mon, 7 May 2018, Igor Djordjevic wrote: > On 07/05/2018 09:48, Jeff King wrote: > > > > > > Let's, please, not fall into the trap of polluting git-branch with > > > > utterly unrelated functionality, as has happened a few times with > > > > other Git commands. Let's especially not do so merely for the sake of > > > > tab-completion. git-branch is for branch management; it's not for > > > > diff'ing. > > > > > > I totally disagree. `git branch` is *the* command to work with branches. > > > Yes, you can manage branches. But you can also list them. And now you can > > > also compare them. > > > > One of the things I don't like about "git branch --diff" is that this > > feature is not _just_ about branches at all. E.g., I could do: > > > > git tbdiff HEAD~10 HEAD~5 foo > > > > Or even: > > > > git tbdiff v2.16.0 v2.17.0 my-rewritten-v2.17.0 > > > > Those arguments really are just commitishes, not necessarily branches. > > One of the current interface rules for "git branch" is that the branch > > names we hand it are interpreted _exactly_ as branch names. You cannot > > "git branch -m v2.16.0", and there is no ambiguity in "git branch -d > > foo" if "foo" is both a tag and a branch. > > > > But this new mode does not fit the pattern at all. > > > > If we were to attach this to an existing command, I think it has more to > > do with "diff" than "branch". But I'm not sure we want to overload > > "diff" either (which has traditionally been about two endpoints, and > > does not really traverse at all, though arguably "foo...bar" is a bit of > > a cheat :) ). > > > > > > Of the suggestions thus far, Junio's git-topic-diff seems the least > > > > worse, and doesn't suffer from tab-completion problems. > > > > > > Except that this is too limited a view. > > > > Right, I agree with you. Topic branches are the intended use, but that's > > not what it _does_, and obviously it can be applied in other cases. So > > since "branch" is too specific, I think "topic branch" is even more so. > > > > It's really "diff-history" or something, I think. That's not very > > catchy, but I think the best name would imply that it was diffing a set > > of commits (so even "diff-commit" would not be right, because that again > > sounds like endpoints). > > This is exactly what I feel as well, thanks for concise and > to-the-point spelling out. > > From user interface perspective, I would expect something like this > to be possible (and natural): > > (1) git diff topic-v1...topic-v2 No, we cannot. The `git diff topic-v1...topic-v2` invocation has worked for a looooong time, and does something very different. We should not even allow ourselves to think of such a breakage. > (2) git diff --branch topic-v1...topic-v2 From my point of view, `git diff --branch` indicates that I diff *branches*. Which is not really something that makes sense, and definitely not what this command is about. We are not comparing branches. We are comparing versions of the same branch. > (1) is what we are all familiar with, providing a diff between two > revisions with focus on file changes, where (2) shifts focus to > history changes. > > It`s all still a comparison between two revisions (pointed to by > "topic-v1" and "topic-v2" branch heads in this specific example), but > it differs in what we are comparing - (1) set of files contained in > endpoints, or (2) set of revisions contained in (or "leading to") > endpoints. It is very much not about comparing *two* revisions. It is very much about comparing two *ranges of* revisions, and not just any ranges, no. Those ranges need to be so related as to contain mostly identical changes. Otherwise, `git branch --diff` will spend a ton of time, just to come back with a series of `-` lines followed by a series of `+` lines (figuratively, not literally). Which would be stupid, to spend that much time on something that `git rev-list --left-right topic1...topic2` would have computed a lot faster. > Hmm... what about `git diff --history`? :/ It does seem more "true" > to what it does, though I still like `git diff --branch` more > (catchier, indeed). It certainly is catchier. But also a ton more puzzling. I do not want to compare histories, after all. That would be like saying: okay, topic1 and topic2 ended up at the same stage, but *how* did they get there? What I *want* to ask via the command implemented by this patch series is the question: there was a set of patches previously, and now I have a set of revised patches, what changed? Most fellow German software engineers (who seem to have a knack for idiotically long variable/function names) would now probably suggest: git compare-patch-series-with-revised-patch-series I hope you agree that that is better *and* worse than your suggestions, depending from what angle you look at it: it is better because it describes what the command is *actually* doing. But it is much worse at the same time because it is too long. Ciao, Dscho
next prev parent reply index Thread overview: 387+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-05-03 15:30 [PATCH 00/18] Add `branch-diff`, a `tbdiff` lookalike Johannes Schindelin 2018-05-03 15:30 ` [PATCH 01/18] Add a function to solve least-cost assignment problems Johannes Schindelin 2018-05-13 18:19 ` Duy Nguyen 2018-05-21 9:52 ` Johannes Schindelin 2018-05-03 15:30 ` [PATCH 02/18] Add a new builtin: branch-diff Johannes Schindelin 2018-05-03 16:10 ` Ramsay Jones 2018-05-03 20:25 ` Johannes Schindelin 2018-05-03 23:20 ` Ramsay Jones 2018-05-04 6:40 ` Johannes Schindelin 2018-05-04 15:37 ` Ramsay Jones 2018-05-05 19:41 ` Johannes Schindelin 2018-05-09 16:24 ` Ramsay Jones 2018-06-01 8:23 ` Johannes Schindelin 2018-05-04 16:34 ` Elijah Newren 2018-05-05 20:24 ` Johannes Schindelin 2018-05-03 16:41 ` Duy Nguyen 2018-05-03 20:30 ` Johannes Schindelin 2018-05-03 20:32 ` Johannes Schindelin 2018-05-04 5:15 ` Duy Nguyen 2018-05-04 7:23 ` Johannes Schindelin 2018-05-04 14:44 ` Duy Nguyen 2018-05-04 15:17 ` Duy Nguyen 2018-05-04 15:23 ` Johannes Schindelin 2018-05-04 15:29 ` Duy Nguyen 2018-05-03 16:43 ` Stefan Beller 2018-05-03 20:42 ` Johannes Schindelin 2018-05-03 21:12 ` Stefan Beller 2018-05-03 21:49 ` Johannes Schindelin 2018-05-04 3:23 ` Junio C Hamano 2018-05-04 2:35 ` Eric Sunshine 2018-05-04 6:52 ` Johannes Schindelin 2018-05-04 7:27 ` Eric Sunshine 2018-05-03 15:30 ` [PATCH 03/18] branch-diff: first rudimentary implementation Johannes Schindelin 2018-05-03 16:30 ` Ramsay Jones 2018-05-03 20:44 ` Johannes Schindelin 2018-05-03 17:06 ` Stefan Beller 2018-05-03 21:01 ` Johannes Schindelin 2018-05-03 21:19 ` Stefan Beller 2018-05-03 22:00 ` Johannes Schindelin 2018-05-04 2:35 ` Eric Sunshine 2018-05-04 7:03 ` Johannes Schindelin 2018-05-04 4:56 ` Junio C Hamano 2018-05-04 7:18 ` Johannes Schindelin 2018-05-03 15:30 ` [PATCH 04/18] branch-diff: improve the order of the shown commits Johannes Schindelin 2018-05-03 15:30 ` [PATCH 05/18] branch-diff: also show the diff between patches Johannes Schindelin 2018-05-04 2:51 ` Eric Sunshine 2018-05-04 3:15 ` Eric Sunshine 2018-05-04 7:15 ` Johannes Schindelin 2018-05-03 15:30 ` [PATCH 06/18] branch-diff: right-trim commit messages Johannes Schindelin 2018-05-03 15:30 ` [PATCH 07/18] branch-diff: indent the diffs just like tbdiff Johannes Schindelin 2018-05-03 15:30 ` [PATCH 08/18] branch-diff: suppress the diff headers Johannes Schindelin 2018-05-03 15:30 ` [PATCH 09/18] branch-diff: adjust the output of the commit pairs Johannes Schindelin 2018-05-03 15:30 ` [PATCH 10/18] branch-diff: do not show "function names" in hunk headers Johannes Schindelin 2018-05-03 15:30 ` [PATCH 11/18] branch-diff: add tests Johannes Schindelin 2018-05-03 16:56 ` Ævar Arnfjörð Bjarmason 2018-05-03 21:03 ` Johannes Schindelin 2018-05-03 17:11 ` Stefan Beller 2018-05-03 21:05 ` Johannes Schindelin 2018-05-03 23:27 ` Philip Oakley 2018-05-04 6:42 ` Johannes Schindelin 2018-05-03 15:30 ` [PATCH 12/18] branch-diff: use color for the commit pairs Johannes Schindelin 2018-05-03 15:30 ` [PATCH 13/18] color: provide inverted colors, too Johannes Schindelin 2018-05-03 15:30 ` [PATCH 14/18] diff: add an internal option to dual-color diffs of diffs Johannes Schindelin 2018-05-03 15:31 ` [PATCH 15/18] branch-diff: offer to dual-color the diffs Johannes Schindelin 2018-05-03 15:31 ` [PATCH 16/18] branch-diff --dual-color: work around bogus white-space warning Johannes Schindelin 2018-05-03 15:31 ` [PATCH 17/18] branch-diff: add a man page Johannes Schindelin 2018-05-04 3:27 ` Eric Sunshine 2018-05-04 7:17 ` Johannes Schindelin 2018-05-03 15:31 ` [PATCH 18/18] completion: support branch-diff Johannes Schindelin 2018-05-03 18:05 ` [PATCH 00/18] Add `branch-diff`, a `tbdiff` lookalike Ævar Arnfjörð Bjarmason 2018-05-03 21:07 ` Johannes Schindelin 2018-05-03 21:50 ` Jacob Keller 2018-05-04 5:24 ` Junio C Hamano 2018-05-04 7:24 ` Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 " Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 01/18] Add a function to solve least-cost assignment problems Johannes Schindelin 2018-05-05 18:24 ` Jeff King 2018-05-05 21:55 ` Johannes Schindelin 2018-05-30 13:55 ` SZEDER Gábor 2018-05-30 16:14 ` Stefan Beller 2018-05-30 23:28 ` brian m. carlson 2018-05-31 12:19 ` Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 02/18] Add a new builtin: branch-diff Johannes Schindelin 2018-05-05 18:26 ` Jeff King 2018-05-05 21:57 ` Johannes Schindelin 2018-05-06 0:25 ` Todd Zullinger 2018-05-06 0:38 ` Todd Zullinger 2018-05-06 12:04 ` Johannes Schindelin 2018-05-06 1:05 ` Igor Djordjevic 2018-05-06 4:53 ` Jacob Keller 2018-05-06 8:32 ` Duy Nguyen 2018-05-06 12:08 ` Johannes Schindelin 2018-05-06 12:10 ` Johannes Schindelin 2018-05-06 13:37 ` Igor Djordjevic 2018-05-07 1:34 ` Johannes Schindelin 2018-05-07 22:05 ` Igor Djordjevic 2018-05-07 22:24 ` Stefan Beller 2018-05-07 23:39 ` Igor Djordjevic 2018-05-08 3:44 ` Jeff King 2018-05-08 3:48 ` Jeff King 2018-05-22 11:38 ` Ævar Arnfjörð Bjarmason 2018-05-25 22:06 ` Stefan Beller [not found] ` <CAA8fPEkNjy+ETz4Mx+C2kUfLjLzR9uuOmO3GfN48ZH1SwyfE1A@mail.gmail.com> 2018-05-26 6:15 ` Fwd: " Øyvind Rønningstad 2018-06-01 8:15 ` Johannes Schindelin 2018-05-06 2:33 ` Junio C Hamano 2018-05-06 12:21 ` Johannes Schindelin 2018-05-06 20:51 ` Eric Sunshine 2018-05-07 2:04 ` Johannes Schindelin 2018-05-07 7:48 ` Jeff King 2018-05-07 21:33 ` Igor Djordjevic 2018-05-21 10:33 ` Johannes Schindelin [this message] 2018-05-21 17:56 ` Stefan Beller 2018-05-21 20:24 ` Jeff King 2018-05-21 21:40 ` Brandon Williams 2018-05-21 21:48 ` Stefan Beller 2018-05-21 21:52 ` Jeff King 2018-05-22 2:08 ` Junio C Hamano 2018-05-08 0:30 ` Junio C Hamano 2018-05-07 1:45 ` Junio C Hamano 2018-05-07 5:39 ` Johannes Schindelin 2018-05-07 15:12 ` Junio C Hamano 2018-05-21 10:41 ` Johannes Schindelin 2018-05-07 7:50 ` Jeff King 2018-05-07 15:28 ` Duy Nguyen 2018-05-07 19:58 ` Stefan Beller 2018-05-04 15:34 ` [PATCH v2 03/18] branch-diff: first rudimentary implementation Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 04/18] branch-diff: improve the order of the shown commits Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 05/18] branch-diff: also show the diff between patches Johannes Schindelin 2018-05-06 1:14 ` Igor Djordjevic 2018-05-06 12:18 ` Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 06/18] branch-diff: right-trim commit messages Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 07/18] branch-diff: indent the diffs just like tbdiff Johannes Schindelin 2018-05-06 14:15 ` Martin Ågren 2018-05-07 1:54 ` Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 08/18] branch-diff: suppress the diff headers Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 09/18] branch-diff: adjust the output of the commit pairs Johannes Schindelin 2018-05-04 16:25 ` Elijah Newren 2018-05-04 15:34 ` [PATCH v2 10/18] branch-diff: do not show "function names" in hunk headers Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 11/18] branch-diff: add tests Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 12/18] branch-diff: use color for the commit pairs Johannes Schindelin 2018-05-05 23:48 ` Todd Zullinger 2018-05-07 1:52 ` Johannes Schindelin 2018-05-08 2:10 ` Todd Zullinger 2018-06-01 8:17 ` Johannes Schindelin 2018-05-04 15:34 ` [PATCH v2 13/18] color: provide inverted colors, too Johannes Schindelin 2018-05-05 18:29 ` Jeff King 2018-05-05 22:03 ` Johannes Schindelin 2018-05-06 6:35 ` Jeff King 2018-05-06 6:41 ` Jeff King 2018-05-07 1:20 ` Johannes Schindelin 2018-05-07 7:37 ` Jeff King 2018-05-07 1:35 ` Junio C Hamano 2018-05-07 5:38 ` Johannes Schindelin 2018-05-07 7:40 ` Jeff King 2018-05-04 15:34 ` [PATCH v2 14/18] diff: add an internal option to dual-color diffs of diffs Johannes Schindelin 2018-05-04 15:35 ` [PATCH v2 15/18] branch-diff: offer to dual-color the diffs Johannes Schindelin 2018-05-04 15:35 ` [PATCH v2 16/18] branch-diff --dual-color: work around bogus white-space warning Johannes Schindelin 2018-05-04 15:35 ` [PATCH v2 17/18] branch-diff: add a man page Johannes Schindelin 2018-05-04 15:35 ` [PATCH v2 18/18] completion: support branch-diff Johannes Schindelin 2018-05-06 8:24 ` Duy Nguyen 2018-05-07 1:23 ` Johannes Schindelin 2018-05-04 16:21 ` [PATCH v2 00/18] Add `branch-diff`, a `tbdiff` lookalike Elijah Newren 2018-05-04 16:30 ` Elijah Newren 2018-05-05 20:03 ` Johannes Schindelin 2018-05-07 17:07 ` Elijah Newren 2018-05-07 17:50 ` SZEDER Gábor 2018-05-07 18:38 ` Elijah Newren 2018-05-06 5:22 ` Junio C Hamano 2018-05-06 12:23 ` Johannes Schindelin 2018-05-06 22:56 ` brian m. carlson 2018-05-07 2:05 ` Johannes Schindelin 2018-07-03 11:26 ` [PATCH v3 00/20] Add `range-diff`, " Johannes Schindelin via GitGitGadget 2018-04-30 21:54 ` [PATCH v3 01/20] linear-assignment: a function to solve least-cost assignment problems Johannes Schindelin via GitGitGadget 2018-07-06 22:43 ` Junio C Hamano 2018-07-07 11:34 ` Johannes Schindelin 2018-07-07 16:34 ` Junio C Hamano 2018-07-07 19:27 ` Johannes Schindelin 2018-07-07 22:23 ` Johannes Schindelin 2018-07-09 22:08 ` refs/notes/amlog problems, was " Johannes Schindelin 2018-07-11 16:12 ` Junio C Hamano 2018-07-12 15:23 ` Johannes Schindelin 2018-07-12 16:59 ` Junio C Hamano 2018-07-19 17:06 ` Junio C Hamano 2018-07-20 18:51 ` Johannes Schindelin 2018-07-20 19:34 ` Junio C Hamano 2018-07-20 21:20 ` Stefan Beller 2018-07-20 21:24 ` Junio C Hamano [not found] ` <CAPc5daW-KoyUX3i7M5YbdQC2mFKAmVBS42-XT84hpm30VFcZ1g@mail.gmail.com> 2018-07-20 21:30 ` Stefan Beller 2018-07-21 22:02 ` Johannes Schindelin 2018-07-21 21:56 ` Johannes Schindelin 2018-07-23 1:25 ` Jeff King 2018-07-24 1:50 ` Junio C Hamano 2018-07-24 9:45 ` Jeff King 2018-07-09 22:23 ` Junio C Hamano 2018-07-10 10:47 ` refs/notes/amlog woes, was " Johannes Schindelin 2018-07-11 10:07 ` SZEDER Gábor 2018-07-12 15:11 ` Johannes Schindelin 2018-05-01 19:42 ` [PATCH v3 02/20] Introduce `range-diff` to compare iterations of a topic branch Johannes Schindelin via GitGitGadget 2018-05-02 0:34 ` [PATCH v3 03/20] range-diff: first rudimentary implementation Johannes Schindelin via GitGitGadget 2018-07-16 6:55 ` Eric Sunshine 2018-07-17 9:53 ` Johannes Schindelin 2018-05-02 10:22 ` [PATCH v3 04/20] range-diff: improve the order of the shown commits Johannes Schindelin via GitGitGadget 2018-05-02 14:49 ` [PATCH v3 06/20] range-diff: right-trim commit messages Johannes Schindelin via GitGitGadget 2018-05-02 14:52 ` [PATCH v3 07/20] range-diff: indent the diffs just like tbdiff Johannes Schindelin via GitGitGadget 2018-05-02 14:53 ` [PATCH v3 08/20] range-diff: suppress the diff headers Johannes Schindelin via GitGitGadget 2018-05-02 15:19 ` [PATCH v3 11/20] range-diff: add tests Thomas Rast via GitGitGadget 2018-07-16 7:28 ` Eric Sunshine 2018-07-17 16:28 ` Johannes Schindelin 2018-05-02 21:35 ` [PATCH v3 09/20] range-diff: adjust the output of the commit pairs Johannes Schindelin via GitGitGadget 2018-07-16 7:21 ` Eric Sunshine 2018-07-17 16:24 ` Johannes Schindelin 2018-07-17 17:47 ` Stefan Beller 2018-07-20 18:57 ` Johannes Schindelin 2018-07-20 19:16 ` Stefan Beller 2018-07-21 22:07 ` Johannes Schindelin 2018-05-02 23:32 ` [PATCH v3 12/20] range-diff: use color for " Johannes Schindelin via GitGitGadget 2018-05-03 0:14 ` [PATCH v3 13/20] color: add the meta color GIT_COLOR_REVERSE Johannes Schindelin via GitGitGadget 2018-05-03 0:17 ` [PATCH v3 14/20] diff: add an internal option to dual-color diffs of diffs Johannes Schindelin via GitGitGadget 2018-07-09 19:29 ` Stefan Beller 2018-07-10 17:45 ` [PATCH 0/2] " Stefan Beller 2018-07-10 17:45 ` [PATCH 1/2] diff.c: convert emit_line_ws_markup to take string for sign Stefan Beller 2018-07-10 17:45 ` [PATCH 2/2] WIP diff.c: clarify emit_line_0 Stefan Beller 2018-07-10 19:58 ` [PATCH 1/2] diff.c: convert emit_line_ws_markup to take string for sign Stefan Beller 2018-07-10 19:59 ` [PATCH] diff.c: clarify emit_line_0 Stefan Beller 2018-07-10 21:54 ` [PATCH] ws: do not reset and set color twice Stefan Beller 2018-07-21 21:13 ` [PATCH 0/2] Re: [PATCH v3 14/20] diff: add an internal option to dual-color diffs of diffs Johannes Schindelin 2018-05-03 1:01 ` [PATCH v3 15/20] range-diff: offer to dual-color the diffs Johannes Schindelin via GitGitGadget 2018-05-03 1:11 ` [PATCH v3 16/20] range-diff --dual-color: work around bogus white-space warning Johannes Schindelin via GitGitGadget 2018-07-09 19:34 ` Stefan Beller 2018-07-09 21:02 ` Junio C Hamano 2018-07-10 10:08 ` Johannes Schindelin 2018-07-10 15:50 ` Junio C Hamano 2018-07-10 16:32 ` Stefan Beller 2018-07-21 21:44 ` Johannes Schindelin 2018-05-03 13:50 ` [PATCH v3 17/20] range-diff: add a man page Johannes Schindelin via GitGitGadget 2018-07-09 18:20 ` Stefan Beller 2018-07-09 20:00 ` Johannes Schindelin 2018-07-09 20:25 ` Stefan Beller 2018-07-09 20:38 ` Johannes Schindelin 2018-07-16 8:01 ` Eric Sunshine 2018-07-17 16:39 ` Johannes Schindelin 2018-05-03 14:44 ` [PATCH v3 18/20] completion: support `git range-diff` Johannes Schindelin via GitGitGadget 2018-07-06 22:46 ` Junio C Hamano 2018-07-07 11:38 ` Johannes Schindelin 2018-05-05 19:52 ` [PATCH v3 19/20] range-diff: left-pad patch numbers Johannes Schindelin via GitGitGadget 2018-05-06 15:26 ` [PATCH v3 05/20] range-diff: also show the diff between patches Johannes Schindelin via GitGitGadget 2018-05-06 15:35 ` [PATCH v3 10/20] range-diff: do not show "function names" in hunk headers Johannes Schindelin via GitGitGadget 2018-06-30 20:41 ` [PATCH v3 20/20] range-diff: make --dual-color the default mode Johannes Schindelin via GitGitGadget 2018-07-16 8:06 ` Eric Sunshine 2018-07-17 16:40 ` Johannes Schindelin 2018-07-21 22:04 ` [PATCH v4 00/21] Add `range-diff`, a `tbdiff` lookalike Johannes Schindelin via GitGitGadget 2018-07-21 22:04 ` [PATCH v4 01/21] linear-assignment: a function to solve least-cost assignment problems Johannes Schindelin via GitGitGadget 2018-07-28 8:46 ` Thomas Gummerer 2018-07-30 15:59 ` Johannes Schindelin 2018-07-21 22:04 ` [PATCH v4 02/21] Introduce `range-diff` to compare iterations of a topic branch Johannes Schindelin via GitGitGadget 2018-07-21 22:04 ` [PATCH v4 03/21] range-diff: first rudimentary implementation Johannes Schindelin via GitGitGadget 2018-07-29 18:36 ` Thomas Gummerer 2018-07-30 16:21 ` Johannes Schindelin 2018-07-30 21:16 ` Thomas Gummerer 2018-08-10 20:50 ` Johannes Schindelin 2018-07-21 22:04 ` [PATCH v4 04/21] range-diff: improve the order of the shown commits Johannes Schindelin via GitGitGadget 2018-07-21 22:04 ` [PATCH v4 05/21] range-diff: also show the diff between patches Johannes Schindelin via GitGitGadget 2018-07-29 19:03 ` Thomas Gummerer 2018-07-29 19:22 ` Eric Sunshine 2018-07-29 21:45 ` Thomas Gummerer 2018-07-30 16:28 ` Johannes Schindelin 2018-07-30 21:26 ` Thomas Gummerer 2018-07-30 21:51 ` Eric Sunshine 2018-08-10 21:12 ` Johannes Schindelin 2018-08-10 21:31 ` Eric Sunshine 2018-08-10 22:02 ` Johannes Schindelin 2018-08-10 20:36 ` Johannes Schindelin 2018-07-21 22:04 ` [PATCH v4 06/21] range-diff: right-trim commit messages Johannes Schindelin via GitGitGadget 2018-07-21 22:04 ` [PATCH v4 07/21] range-diff: indent the diffs just like tbdiff Johannes Schindelin via GitGitGadget 2018-07-21 22:04 ` [PATCH v4 08/21] range-diff: suppress the diff headers Johannes Schindelin via GitGitGadget 2018-07-21 22:04 ` [PATCH v4 09/21] range-diff: adjust the output of the commit pairs Johannes Schindelin via GitGitGadget 2018-07-29 19:38 ` Thomas Gummerer 2018-08-10 21:01 ` Johannes Schindelin 2018-07-29 21:28 ` Thomas Gummerer 2018-07-21 22:04 ` [PATCH v4 10/21] range-diff: do not show "function names" in hunk headers Johannes Schindelin via GitGitGadget 2018-07-29 20:52 ` Thomas Gummerer 2018-08-10 21:03 ` Johannes Schindelin 2018-07-21 22:05 ` [PATCH v4 11/21] range-diff: add tests Thomas Rast via GitGitGadget 2018-07-22 5:04 ` Eric Sunshine 2018-07-30 16:30 ` Johannes Schindelin 2018-07-30 20:18 ` Junio C Hamano 2018-07-30 23:40 ` Stefan Beller 2018-07-31 15:19 ` Junio C Hamano 2018-07-23 21:25 ` Stefan Beller 2018-07-21 22:05 ` [PATCH v4 12/21] range-diff: use color for the commit pairs Johannes Schindelin via GitGitGadget 2018-07-21 22:05 ` [PATCH v4 13/21] color: add the meta color GIT_COLOR_REVERSE Johannes Schindelin via GitGitGadget 2018-07-21 22:05 ` [PATCH v4 14/21] diff: add an internal option to dual-color diffs of diffs Johannes Schindelin via GitGitGadget 2018-07-23 22:27 ` Junio C Hamano 2018-07-23 22:48 ` Stefan Beller 2018-07-21 22:05 ` [PATCH v4 15/21] range-diff: offer to dual-color the diffs Johannes Schindelin via GitGitGadget 2018-07-21 22:05 ` [PATCH v4 16/21] range-diff --dual-color: fix bogus white-space warning Johannes Schindelin via GitGitGadget 2018-07-23 22:20 ` Stefan Beller 2018-08-10 21:05 ` Johannes Schindelin 2018-07-23 22:39 ` Junio C Hamano 2018-07-24 1:27 ` Junio C Hamano 2018-07-21 22:05 ` [PATCH v4 17/21] range-diff: populate the man page Johannes Schindelin via GitGitGadget 2018-07-29 21:23 ` Thomas Gummerer 2018-08-10 21:06 ` Johannes Schindelin 2018-07-21 22:05 ` [PATCH v4 18/21] completion: support `git range-diff` Johannes Schindelin via GitGitGadget 2018-07-22 5:49 ` Eric Sunshine 2018-08-10 20:24 ` Johannes Schindelin 2018-07-21 22:05 ` [PATCH v4 19/21] range-diff: left-pad patch numbers Johannes Schindelin via GitGitGadget 2018-07-21 22:05 ` [PATCH v4 20/21] range-diff: make --dual-color the default mode Johannes Schindelin via GitGitGadget 2018-07-29 21:33 ` Thomas Gummerer 2018-08-10 21:07 ` Johannes Schindelin 2018-07-21 22:05 ` [PATCH v4 21/21] range-diff: use dim/bold cues to improve dual color mode Johannes Schindelin via GitGitGadget 2018-07-23 21:03 ` [PATCH v4 00/21] Add `range-diff`, a `tbdiff` lookalike Stefan Beller 2018-07-23 21:49 ` Junio C Hamano 2018-07-25 17:44 ` Stefan Beller 2018-07-26 9:47 ` Johannes Schindelin 2018-08-08 13:05 ` Johannes Schindelin 2018-08-08 17:33 ` Stefan Beller 2018-08-10 21:18 ` Johannes Schindelin 2018-08-10 21:31 ` Junio C Hamano 2018-08-10 22:00 ` Johannes Schindelin 2018-07-29 21:50 ` Thomas Gummerer 2018-08-10 22:14 ` [PATCH v5 00/21] Add range-diff, a tbdiff lookalike Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 01/21] linear-assignment: a function to solve least-cost assignment problems Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 02/21] Introduce `range-diff` to compare iterations of a topic branch Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 03/21] range-diff: first rudimentary implementation Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 04/21] range-diff: improve the order of the shown commits Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 05/21] range-diff: also show the diff between patches Johannes Schindelin via GitGitGadget 2018-08-12 21:47 ` Thomas Gummerer 2018-08-13 9:46 ` Johannes Schindelin 2018-08-13 18:01 ` Thomas Gummerer 2018-08-10 22:14 ` [PATCH v5 06/21] range-diff: right-trim commit messages Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 07/21] range-diff: indent the diffs just like tbdiff Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 08/21] range-diff: suppress the diff headers Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 09/21] range-diff: adjust the output of the commit pairs Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 10/21] range-diff: do not show "function names" in hunk headers Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 11/21] range-diff: add tests Thomas Rast via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 12/21] range-diff: use color for the commit pairs Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 13/21] color: add the meta color GIT_COLOR_REVERSE Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 14/21] diff: add an internal option to dual-color diffs of diffs Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 15/21] range-diff: offer to dual-color the diffs Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 16/21] range-diff --dual-color: skip white-space warnings Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 17/21] range-diff: populate the man page Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 18/21] completion: support `git range-diff` Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 19/21] range-diff: left-pad patch numbers Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 20/21] range-diff: make --dual-color the default mode Johannes Schindelin via GitGitGadget 2018-08-10 22:14 ` [PATCH v5 21/21] range-diff: use dim/bold cues to improve dual color mode Johannes Schindelin via GitGitGadget 2018-08-13 11:32 ` [PATCH v6 00/21] Add range-diff, a tbdiff lookalike Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 01/21] linear-assignment: a function to solve least-cost assignment problems Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 02/21] Introduce `range-diff` to compare iterations of a topic branch Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 03/21] range-diff: first rudimentary implementation Johannes Schindelin via GitGitGadget 2019-03-05 6:29 ` Junio C Hamano 2018-08-13 11:33 ` [PATCH v6 04/21] range-diff: improve the order of the shown commits Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 05/21] range-diff: also show the diff between patches Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 06/21] range-diff: right-trim commit messages Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 07/21] range-diff: indent the diffs just like tbdiff Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 08/21] range-diff: suppress the diff headers Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 09/21] range-diff: adjust the output of the commit pairs Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 10/21] range-diff: do not show "function names" in hunk headers Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 11/21] range-diff: add tests Thomas Rast via GitGitGadget 2018-08-13 18:35 ` Thomas Gummerer 2018-08-14 14:53 ` Johannes Schindelin 2018-08-14 15:03 ` Jeff King 2018-08-14 15:06 ` Jeff King 2018-08-14 15:18 ` Junio C Hamano 2018-08-13 11:33 ` [PATCH v6 12/21] range-diff: use color for the commit pairs Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 13/21] color: add the meta color GIT_COLOR_REVERSE Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 14/21] diff: add an internal option to dual-color diffs of diffs Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 15/21] range-diff: offer to dual-color the diffs Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 16/21] range-diff --dual-color: skip white-space warnings Johannes Schindelin via GitGitGadget 2018-08-13 17:48 ` Junio C Hamano 2018-08-13 11:33 ` [PATCH v6 17/21] range-diff: populate the man page Johannes Schindelin via GitGitGadget 2018-09-09 11:14 ` Ævar Arnfjörð Bjarmason 2018-09-09 16:54 ` SZEDER Gábor 2018-09-09 17:19 ` Ævar Arnfjörð Bjarmason 2018-09-10 13:37 ` Jeff King 2018-10-02 15:06 ` Johannes Schindelin 2018-09-10 17:17 ` Junio C Hamano 2018-08-13 11:33 ` [PATCH v6 18/21] completion: support `git range-diff` Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 19/21] range-diff: left-pad patch numbers Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 20/21] range-diff: make --dual-color the default mode Johannes Schindelin via GitGitGadget 2018-08-13 11:33 ` [PATCH v6 21/21] range-diff: use dim/bold cues to improve dual color mode Johannes Schindelin via GitGitGadget 2018-08-13 11:38 ` [PATCH v6 00/21] Add range-diff, a tbdiff lookalike Johannes Schindelin 2018-08-13 20:47 ` Thomas Gummerer 2018-05-21 4:48 ` [PATCH 00/18] Add `branch-diff`, a `tbdiff` lookalike Junio C Hamano 2018-05-21 9:51 ` Johannes Schindelin 2018-05-22 1:42 ` Junio C Hamano 2018-06-01 8:28 ` Johannes Schindelin
Reply instructions: You may reply publically to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style List information: http://vger.kernel.org/majordomo-info.html * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=nycvar.QRO.7.76.6.1805211153370.77@tvgsbejvaqbjf.bet \ --to=johannes.schindelin@gmx.de \ --cc=avarab@gmail.com \ --cc=git@vger.kernel.org \ --cc=gitster@pobox.com \ --cc=igor.d.djordjevic@gmail.com \ --cc=jacob.keller@gmail.com \ --cc=peff@peff.net \ --cc=ramsay@ramsayjones.plus.com \ --cc=sbeller@google.com \ --cc=sunshine@sunshineco.com \ --cc=t.gummerer@gmail.com \ --cc=tr@thomasrast.ch \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
git@vger.kernel.org list mirror (unofficial, one of many) Archives are clonable: git clone --mirror https://public-inbox.org/git git clone --mirror http://ou63pmih66umazou.onion/git git clone --mirror http://czquwvybam4bgbro.onion/git git clone --mirror http://hjrcffqmbrq6wope.onion/git Example config snippet for mirrors Newsgroups are available over NNTP: nntp://news.public-inbox.org/inbox.comp.version-control.git nntp://ou63pmih66umazou.onion/inbox.comp.version-control.git nntp://czquwvybam4bgbro.onion/inbox.comp.version-control.git nntp://hjrcffqmbrq6wope.onion/inbox.comp.version-control.git nntp://news.gmane.org/gmane.comp.version-control.git note: .onion URLs require Tor: https://www.torproject.org/ AGPL code for this site: git clone https://public-inbox.org/public-inbox.git