Skip to content

Commit 12580ca

Browse files
hvoigtdscho
authored andcommitted
git-gui: provide question helper for retry fallback on Windows
Make use of the new environment variable GIT_ASK_YESNO to support the recently implemented fallback in case unlink, rename or rmdir fail for files in use on Windows. The added dialog will present a yes/no question to the the user which will currently be used by the windows compat layer to let the user retry a failed file operation. Signed-off-by: Heiko Voigt <hvoigt@hvoigt.net>
1 parent 9539c8e commit 12580ca

File tree

3 files changed

+56
-0
lines changed

3 files changed

+56
-0
lines changed

Diff for: git-gui/Makefile

+2
Original file line numberDiff line numberDiff line change
@@ -280,6 +280,7 @@ install: all
280280
$(QUIET)$(INSTALL_D0)'$(DESTDIR_SQ)$(gitexecdir_SQ)' $(INSTALL_D1)
281281
$(QUIET)$(INSTALL_X0)git-gui $(INSTALL_X1) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
282282
$(QUIET)$(INSTALL_X0)git-gui--askpass $(INSTALL_X1) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
283+
$(QUIET)$(INSTALL_X0)git-gui--askyesno $(INSTALL_X1) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
283284
$(QUIET)$(foreach p,$(GITGUI_BUILT_INS), $(INSTALL_L0)'$(DESTDIR_SQ)$(gitexecdir_SQ)/$p' $(INSTALL_L1)'$(DESTDIR_SQ)$(gitexecdir_SQ)/git-gui' $(INSTALL_L2)'$(DESTDIR_SQ)$(gitexecdir_SQ)/$p' $(INSTALL_L3) &&) true
284285
ifdef GITGUI_WINDOWS_WRAPPER
285286
$(QUIET)$(INSTALL_R0)git-gui.tcl $(INSTALL_R1) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
@@ -298,6 +299,7 @@ uninstall:
298299
$(QUIET)$(CLEAN_DST) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
299300
$(QUIET)$(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/git-gui $(REMOVE_F1)
300301
$(QUIET)$(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/git-gui--askpass $(REMOVE_F1)
302+
$(QUIET)$(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/git-gui--askyesno $(REMOVE_F1)
301303
$(QUIET)$(foreach p,$(GITGUI_BUILT_INS), $(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/$p $(REMOVE_F1) &&) true
302304
ifdef GITGUI_WINDOWS_WRAPPER
303305
$(QUIET)$(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/git-gui.tcl $(REMOVE_F1)

Diff for: git-gui/git-gui--askyesno

+51
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
#!/bin/sh
2+
# Tcl ignores the next line -*- tcl -*- \
3+
exec wish "$0" -- "$@"
4+
5+
# This is an implementation of a simple yes no dialog
6+
# which is injected into the git commandline by git gui
7+
# in case a yesno question needs to be answered.
8+
9+
set NS {}
10+
set use_ttk [package vsatisfies [package provide Tk] 8.5]
11+
if {$use_ttk} {
12+
set NS ttk
13+
}
14+
15+
if {$argc < 1} {
16+
puts stderr "Usage: $argv0 <question>"
17+
exit 1
18+
} else {
19+
set prompt [join $argv " "]
20+
}
21+
22+
${NS}::frame .t
23+
${NS}::label .t.m -text $prompt -justify center -width 40
24+
.t.m configure -wraplength 400
25+
pack .t.m -side top -fill x -padx 20 -pady 20 -expand 1
26+
pack .t -side top -fill x -ipadx 20 -ipady 20 -expand 1
27+
28+
${NS}::frame .b
29+
${NS}::frame .b.left -width 200
30+
${NS}::button .b.yes -text Yes -command yes
31+
${NS}::button .b.no -text No -command no
32+
33+
34+
pack .b.left -side left -expand 1 -fill x
35+
pack .b.yes -side left -expand 1
36+
pack .b.no -side right -expand 1 -ipadx 5
37+
pack .b -side bottom -fill x -ipadx 20 -ipady 15
38+
39+
bind . <Key-Return> {exit 0}
40+
bind . <Key-Escape> {exit 1}
41+
42+
proc no {} {
43+
exit 1
44+
}
45+
46+
proc yes {} {
47+
exit 0
48+
}
49+
50+
wm title . "Question?"
51+
tk::PlaceWindow .

Diff for: git-gui/git-gui.sh

+3
Original file line numberDiff line numberDiff line change
@@ -1248,6 +1248,9 @@ set have_tk85 [expr {[package vcompare $tk_version "8.5"] >= 0}]
12481248
if {![info exists env(SSH_ASKPASS)]} {
12491249
set env(SSH_ASKPASS) [gitexec git-gui--askpass]
12501250
}
1251+
if {![info exists env(GIT_ASK_YESNO)]} {
1252+
set env(GIT_ASK_YESNO) [gitexec git-gui--askyesno]
1253+
}
12511254

12521255
######################################################################
12531256
##

0 commit comments

Comments
 (0)