Code

git-gui: Work around odd cygpath bug on Windows.
authorShawn O. Pearce <spearce@spearce.org>
Sun, 21 Jan 2007 00:03:26 +0000 (19:03 -0500)
committerShawn O. Pearce <spearce@spearce.org>
Sun, 21 Jan 2007 07:54:17 +0000 (02:54 -0500)
There appears to be a bug on one of my test systems where cygpath with
the --long-name option is generating a corrupt string that does not
actually refer to sh.exe.  This breaks any desktop icon created by
git-gui as the executable we are trying to invoke does not exist.
Since Cygwin is typically installed as C:\cygwin long path names is
probably not actually necessary to link to the shell.

I also added a small echo to the start of the icon script, as it can
take one of my test systems several seconds to startup git-gui.  This
way the user knows we're starting git-gui, and was politely asked to
wait for the action to complete.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
git-gui

diff --git a/git-gui b/git-gui
index ade64dcd1243123104164c42f2bd26555993c2b4..c5120cc1ac7ca2f3ae44dc06c403e7e6497bc689 100755 (executable)
--- a/git-gui
+++ b/git-gui
@@ -2742,7 +2742,6 @@ proc do_windows_shortcut {} {
                                set sh [exec cygpath \
                                        --windows \
                                        --absolute \
-                                       --long-name \
                                        /bin/sh]
                                set me [exec cygpath \
                                        --unix \
@@ -2754,6 +2753,7 @@ proc do_windows_shortcut {} {
                                        $gitdir]
                                regsub -all ' $me "'\\''" me
                                regsub -all ' $gd "'\\''" gd
+                               puts $fd "@ECHO Starting git-gui... Please wait..."
                                puts -nonewline $fd "\"$sh\" --login -c \""
                                puts -nonewline $fd "GIT_DIR='$gd'"
                                puts -nonewline $fd " '$me'"