threads / patch / 43553

patchgitk: pass --no-left-right to rev-list

Subject: [PATCH] gitk: pass --no-left-right to rev-list

## tl;dr

3 messages between Dec 19, 2006 and Dec 19, 2006. Diffs are folded; open one to read it.

replies: 2people: 3as markdown or json

Jeff King· Dec 19, 2006, 10:05 UTC · lore
Now that rev-list does left-right marking by default on symmetric
differences, command lines like this:
  gitk master...origin
produce an error.
Signed-off-by: Jeff King <peff@peff.net>
---
I think there are three fixes:
  1. this patch (avoid using left-right)
  2. gitk can detect and parse the new format
  3. revert automatic use of left-right
The third makes the most sense to me.
People do use symmetric difference with various porcelains. For example,
I typically carry a few local patches in 'master'. My daily
sit-down-with-git workflow is:
  git-fetch
  gitk master...origin
  git-rebase origin
where the second step gives me a nice graph of what's new and which
patches I'm still carrying.

That means that porcelains which use git-rev-list are _all_ going to break.

 gitk |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Show changes to gitk +1 −1
diff --git a/gitk b/gitk
index 3dabc69..ec61a37 100755
--- a/gitk
+++ b/gitk
@@ -33,7 +33,7 @@ proc start_rev_list {view} {
 	set order "--date-order"
     }
     if {[catch {
-	set fd [open [concat | git rev-list --header $order \
+	set fd [open [concat | git rev-list --no-left-right --header $order \
 			  --parents --boundary --default HEAD $args] r]
     } err]} {
 	puts stderr "Error executing git rev-list: $err"
Junio C Hamano· Dec 19, 2006, 10:28 UTC · re: Jeff King · lore

Re: [PATCH] gitk: pass --no-left-right to rev-list

Jeff King <peff@peff.net> writes:
Show 5 quoted lines
> I think there are three fixes:
>   1. this patch (avoid using left-right)
>   2. gitk can detect and parse the new format
>   3. revert automatic use of left-right
> The third makes the most sense to me.
1. has the problem that it requires the version of Porcelains
and the core to be tightly linked.  2. is nice but unnecessary
burden in the short term.  3. does not break things for anybody,
but if left-right output is useful on the terminal, it burdens
the end-user typing from the command line.

So in the longer term, 2 is probably slightly more preferrable, but my philosophy is "do not change anything unless absolutely necessary", so 3 is also sensible, at least in the short term.

Thanks for catching this quickly, before it hit the master. Let's revert 57612319.

Johannes Schindelin· Dec 19, 2006, 10:47 UTC · re: Jeff King · lore

Re: [PATCH] gitk: pass --no-left-right to rev-list

Hi,
On Tue, 19 Dec 2006, Jeff King wrote:
Show 5 quoted lines
> I think there are three fixes:
>   1. this patch (avoid using left-right)
>   2. gitk can detect and parse the new format
>   3. revert automatic use of left-right
> The third makes the most sense to me.
IMHO 2. is the way to go.
Ciao,

← back to recent threads