Skip to content

Commit 6134b42

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 <[email protected]>
1 parent 5501b4a commit 6134b42

File tree

3 files changed

+56
-0
lines changed

3 files changed

+56
-0
lines changed

git-gui/Makefile

+2
Original file line numberDiff line numberDiff line change
@@ -295,6 +295,7 @@ install: all
295295
$(QUIET)$(INSTALL_D0)'$(DESTDIR_SQ)$(gitexecdir_SQ)' $(INSTALL_D1)
296296
$(QUIET)$(INSTALL_X0)git-gui $(INSTALL_X1) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
297297
$(QUIET)$(INSTALL_X0)git-gui--askpass $(INSTALL_X1) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
298+
$(QUIET)$(INSTALL_X0)git-gui--askyesno $(INSTALL_X1) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
298299
$(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
299300
ifdef GITGUI_WINDOWS_WRAPPER
300301
$(QUIET)$(INSTALL_R0)git-gui.tcl $(INSTALL_R1) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
@@ -313,6 +314,7 @@ uninstall:
313314
$(QUIET)$(CLEAN_DST) '$(DESTDIR_SQ)$(gitexecdir_SQ)'
314315
$(QUIET)$(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/git-gui $(REMOVE_F1)
315316
$(QUIET)$(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/git-gui--askpass $(REMOVE_F1)
317+
$(QUIET)$(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/git-gui--askyesno $(REMOVE_F1)
316318
$(QUIET)$(foreach p,$(GITGUI_BUILT_INS), $(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/$p $(REMOVE_F1) &&) true
317319
ifdef GITGUI_WINDOWS_WRAPPER
318320
$(QUIET)$(REMOVE_F0)'$(DESTDIR_SQ)$(gitexecdir_SQ)'/git-gui.tcl $(REMOVE_F1)

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 .

git-gui/git-gui.sh

+3
Original file line numberDiff line numberDiff line change
@@ -1230,6 +1230,9 @@ set have_tk85 [expr {[package vcompare $tk_version "8.5"] >= 0}]
12301230
if {![info exists env(SSH_ASKPASS)]} {
12311231
set env(SSH_ASKPASS) [gitexec git-gui--askpass]
12321232
}
1233+
if {![info exists env(GIT_ASK_YESNO)]} {
1234+
set env(GIT_ASK_YESNO) [gitexec git-gui--askyesno]
1235+
}
12331236
12341237
######################################################################
12351238
##

0 commit comments

Comments
 (0)