From: Mark Levedahl <mlevedahl@gmail.com>
Date: Tue, 1 Apr 2025 03:01:01 +0000 (-0400)
Subject: gitk: _search_exe is no longer needed
X-Git-Tag: v2.50.0-rc0~55^2~2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=74571bff17986359170f7dcf135c9a21bbdce06e;p=thirdparty%2Fgit.git

gitk: _search_exe is no longer needed

The _search_exe variable allows specifying the suffix used for executables,
typically {} on unix, .exe on Windows. But, the override code is now
used only on Windows, so _search_exe is no longer needed. Eliminate it.

Signed-off-by: Mark Levedahl <mlevedahl@gmail.com>
Signed-off-by: Johannes Sixt <j6t@kdbg.org>
---

diff --git a/gitk b/gitk
index a22657bc8e..899b51ca55 100755
--- a/gitk
+++ b/gitk
@@ -27,7 +27,7 @@ proc is_Windows {} {
 if {[is_Windows]} {
     set _search_path {}
     proc _which {what args} {
-        global env _search_exe _search_path
+        global env _search_path
 
         if {$_search_path eq {}} {
             set gitguidir [file dirname [info script]]
@@ -37,13 +37,12 @@ if {[is_Windows]} {
             # Skip empty `PATH` elements
             set _search_path [lsearch -all -inline -not -exact \
                 $_search_path ""]
-            set _search_exe .exe
         }
 
         if {[lsearch -exact $args -script] >= 0} {
             set suffix {}
         } else {
-            set suffix $_search_exe
+            set suffix .exe
         }
 
         foreach p $_search_path {