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

Re: [PATCH] gitk - restore operation of git-reset on Cygwin

From
Mark Levedahl <mlevedahl@gmail.com>
Date
May 23, 2008, 23:13 UTC
Message-ID
<48374F82.7050005@gmail.com>
In-Reply-To
<483744AB.8070509@gmail.com>
Mark Levedahl wrote:
Show 18 quoted lines
> Paul Mackerras wrote:
>> Are you using Tk 8.4 or 8.5?  The man pages for 8.5 imply that the
>> 2>@1 form should work for all platforms, but maybe that wasn't true in
>> 8.4.  If that's the case it's worth mentioning in the patch
>> description.
>>
>> Paul.
>>   
> Cygwin's Tcl/Tk is essentially frozen at 8.4.1. However, I really 
> believe the problem is that the Tcl/Tk is not a full Cygwin 
> application: it is mostly Windows, with only a few pieces converted to 
> Posix through the cygwin.dll, so many things don't quite work 
> correctly. For instance, the current gitk works fine under a pure 
> Cygwin build of Tcl/Tk (8.4.13) I have locally. So, I believe the 
> commit message I wrote is correct.
>
> Mark
>

I just tried installation of ActiveState Tcl 8.4.16, put that on the Cygwin path, and gitk operates correctly without the patch. So, the issue seems to be entirely one of Cygwin's particular Tcl/Tk build, though whether that is due to the version or the odd attributes of its compilation I cannot say.

Mark
Previous: Mark LevedahlNext: Shawn O. Pearce
Message 4 of 5 in “gitk - restore operation of git-reset on Cygwin”
  1. gitk - restore operation of git-reset on CygwinMark Levedahl, May 23, 2008
  2. Paul MackerrasMay 23, 2008
  3. Mark LevedahlMay 23, 2008
  4. Mark LevedahlMay 23, 2008
  5. Shawn O. PearceMay 23, 2008

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.