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

Re: git gui create desktop icon currently does not work

From
Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Date
Feb 23, 2010, 22:54 UTC
Message-ID
<cb7bb73a1002231454j4720b28ei20c10de37e929fde@mail.gmail.com>
In-Reply-To
<20100223224955.GB11271@book.hvoigt.net>
On Tue, Feb 23, 2010 at 11:49 PM, Heiko Voigt <hvoigt@hvoigt.net> wrote:
Show 38 quoted lines
> Hi,
>
> I have a patch (further down) to this issue but I only tested this on
> Windows 7 and my virtual machine (XP) did not like it for some reason (it
> created the shortcut but the shortcut did not work). If anyone else can
> confirm that this patch actually works we can include it.
>
> Commit 3748b0 git-gui: update shortcut tools to use _gitworktree
> from Guiseppe does break the creation of Desktop shortcuts on Windows.
>
> I do not know whether this has worked with any tcl interpreter as it
> attempts to execute $_gitworktree as a command.
>
> Here is my fix:
>
> From 5c22c39d530ffb308826629d974048d30cd32e53 Mon Sep 17 00:00:00 2001
> From: Heiko Voigt <hvoigt@hvoigt.net>
> Date: Tue, 23 Feb 2010 10:40:14 +0100
> Subject: [PATCH 1/2] git-gui: fix usage of _gitworktree when creating shortcut for windows
>
> Signed-off-by: Heiko Voigt <heiko.voigt@mahr.de>
> ---
>  git-gui/lib/shortcut.tcl |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/git-gui/lib/shortcut.tcl b/git-gui/lib/shortcut.tcl
> index 79c1888..8cad0e2 100644
> --- a/git-gui/lib/shortcut.tcl
> +++ b/git-gui/lib/shortcut.tcl
> @@ -16,7 +16,7 @@ proc do_windows_shortcut {} {
>                                        [info nameofexecutable] \
>                                        [file normalize $::argv0] \
>                                        ] \
> -                                       [file normalize [$_gitworktree]]
> +                                       [file normalize $_gitworktree]
>                        } err]} {
>                        error_popup [strcat [mc "Cannot write shortcut:"] "\n\n$err"]
>                }
Ouch. Yes, I think your patch is obviously correct.
(I'll go sit in a corner with my 'TCL Dunce' cap on for a while ...)
-- 
Giuseppe "Oblomov" Bilotta
Previous: Heiko VoigtNext: Heiko Voigt
Message 2 of 12 in “git gui create desktop icon currently does not work”
  1. Heiko VoigtFeb 23, 2010
  2. Giuseppe BilottaFeb 23, 2010
  3. Heiko VoigtFeb 23, 2010
  4. Giuseppe BilottaFeb 25, 2010
  5. Heiko VoigtFeb 27, 2010
  6. 1/3 git-gui: fix usage of _gitworktree when creating shortcut for windowsHeiko Voigt, Feb 27, 2010
  7. 2/3 git-gui: fix PATH environment for mingw development environmentHeiko Voigt, Feb 27, 2010
  8. Pat ThoytsFeb 28, 2010
  9. Heiko VoigtFeb 28, 2010
  10. 3/3 git-gui: fix shortcut creation on cygwinHeiko Voigt, Feb 27, 2010
  11. Johannes SchindelinMar 8, 2010
  12. Giuseppe BilottaFeb 27, 2010

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.