Skip to content

Commit c0698df

Browse files
committed
Revert "git-gui: remove lines starting with the comment character"
This reverts commit b9a4386. This commit causes breakage on macOS (10.13). It causes errors on startup and completely breaks the commit functionality. There are two main problems. First, it uses `string cat` which is not supported on older Tcl versions. Second, it does a half close of the bidirectional pipe to git-stripspace which is also not supported on older Tcl versions. Reported-by: Eric Sunshine <[email protected]> Signed-off-by: Pratyush Yadav <[email protected]>
1 parent b9a4386 commit c0698df

File tree

2 files changed

+2
-21
lines changed

2 files changed

+2
-21
lines changed

git-gui.sh

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -875,7 +875,6 @@ set default_config(merge.summary) false
875875
set default_config(merge.verbosity) 2
876876
set default_config(user.name) {}
877877
set default_config(user.email) {}
878-
set default_config(core.commentchar) "#"
879878
880879
set default_config(gui.encoding) [encoding system]
881880
set default_config(gui.matchtrackingbranch) false
@@ -3437,10 +3436,6 @@ proc trace_commit_type {varname args} {
34373436
merge {set txt [mc "Merge Commit Message:"]}
34383437
* {set txt [mc "Commit Message:"]}
34393438
}
3440-
3441-
set comment_char [get_config core.commentchar]
3442-
set txt [string cat $txt \
3443-
[mc " (Lines starting with '$comment_char' will be ignored)"]]
34443439
$ui_coml conf -text $txt
34453440
}
34463441
trace add variable commit_type write trace_commit_type

lib/commit.tcl

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -141,20 +141,6 @@ proc setup_commit_encoding {msg_wt {quiet 0}} {
141141
}
142142
}
143143

144-
proc strip_msg {msg} {
145-
set cmd [concat [list | ] [_git_cmd stripspace] --strip-comments]
146-
_trace_exec $cmd
147-
set fd [open $cmd r+]
148-
fconfigure $fd -translation binary -encoding utf-8
149-
150-
puts -nonewline $fd $msg
151-
close $fd w
152-
set result [read $fd]
153-
close $fd
154-
155-
return $result
156-
}
157-
158144
proc commit_tree {} {
159145
global HEAD commit_type file_states ui_comm repo_config
160146
global pch_error
@@ -221,8 +207,8 @@ You must stage at least 1 file before you can commit.
221207

222208
# -- A message is required.
223209
#
224-
set msg [strip_msg [$ui_comm get 1.0 end]]
225-
210+
set msg [string trim [$ui_comm get 1.0 end]]
211+
regsub -all -line {[ \t\r]+$} $msg {} msg
226212
if {$msg eq {}} {
227213
error_popup [mc "Please supply a commit message.
228214

0 commit comments

Comments
 (0)