git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: The latest commit to add new keybindings

From
Paul Mackerras <paulus@samba.org>
Date
Sep 26, 2005, 00:26 UTC
Message-ID
<17207.16457.551775.470665@cargo.ozlabs.ibm.com>
In-Reply-To
<7v4q8b8d5j.fsf@assigned-by-dhcp.cox.net>
Junio C Hamano writes:
> Do you actually require 8.4, or any reasonably recent wish would
> do?

Checking in the change from "wish" to "wish8.4" was a mistake - that was a change I made for some tests, and I forgot to change it back. However, gitk does need tk 8.4 or later, since it uses the panedwindow widget, which tk 8.3 doesn't have. I have tk8.5a2 installed here, which is nice because it does antialiased fonts, although it is a bit slower (clock format is much slower because they changed it from being implemented in C to Tcl).

> With "${1+$@}", you are passing an empty parameter after '--'
> when gitk itself receives no parameter.  Maybe it is intended,
> maybe not...
No, it isn't intended, thanks for pointing that out.

Regards, Paul.

Previous: Junio C HamanoNext: H. Peter Anvin
Message 2 of 4 in “The latest commit to add new keybindings”
  1. Junio C HamanoSep 24, 2005
  2. Paul MackerrasSep 26, 2005
  3. H. Peter AnvinSep 26, 2005
  4. Junio C HamanoSep 26, 2005

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.