git-gui: Always use eq/ne for string comparsions.
[gitweb.git] / git-gui
diff --git a/git-gui b/git-gui
index 931a959307929de4b8a78c037ebdb6f3f5b78e04..05ef8e2e336d8f0d522bc7fcd615817f2c643406 100755 (executable)
--- a/git-gui
+++ b/git-gui
@@ -24,24 +24,27 @@ proc is_many_config {name} {
        }
 }
 
-proc load_config {} {
+proc load_config {include_global} {
        global repo_config global_config default_config
 
        array unset global_config
-       array unset repo_config
-       catch {
-               set fd_rc [open "| git repo-config --global --list" r]
-               while {[gets $fd_rc line] >= 0} {
-                       if {[regexp {^([^=]+)=(.*)$} $line line name value]} {
-                               if {[is_many_config $name]} {
-                                       lappend global_config($name) $value
-                               } else {
-                                       set global_config($name) $value
+       if {$include_global} {
+               catch {
+                       set fd_rc [open "| git repo-config --global --list" r]
+                       while {[gets $fd_rc line] >= 0} {
+                               if {[regexp {^([^=]+)=(.*)$} $line line name value]} {
+                                       if {[is_many_config $name]} {
+                                               lappend global_config($name) $value
+                                       } else {
+                                               set global_config($name) $value
+                                       }
                                }
                        }
+                       close $fd_rc
                }
-               close $fd_rc
        }
+
+       array unset repo_config
        catch {
                set fd_rc [open "| git repo-config --list" r]
                while {[gets $fd_rc line] >= 0} {
@@ -87,14 +90,15 @@ proc save_config {} {
 
        foreach name [array names default_config] {
                set value $global_config_new($name)
-               if {$value != $global_config($name)} {
-                       if {$value == $default_config($name)} {
+               if {$value ne $global_config($name)} {
+                       if {$value eq $default_config($name)} {
                                catch {exec git repo-config --global --unset $name}
                        } else {
-                               catch {exec git repo-config --global $name $value}
+                               regsub -all "\[{}\]" $value {"} value
+                               exec git repo-config --global $name $value
                        }
                        set global_config($name) $value
-                       if {$value == $repo_config($name)} {
+                       if {$value eq $repo_config($name)} {
                                catch {exec git repo-config --unset $name}
                                set repo_config($name) $value
                        }
@@ -103,11 +107,12 @@ proc save_config {} {
 
        foreach name [array names default_config] {
                set value $repo_config_new($name)
-               if {$value != $repo_config($name)} {
-                       if {$value == $global_config($name)} {
+               if {$value ne $repo_config($name)} {
+                       if {$value eq $global_config($name)} {
                                catch {exec git repo-config --unset $name}
                        } else {
-                               catch {exec git repo-config $name $value}
+                               regsub -all "\[{}\]" $value {"} value
+                               exec git repo-config $name $value
                        }
                        set repo_config($name) $value
                }
@@ -118,7 +123,7 @@ proc error_popup {msg} {
        global gitdir appname
 
        set title $appname
-       if {$gitdir != {}} {
+       if {$gitdir ne {}} {
                append title { (}
                append title [lindex \
                        [file split [file normalize [file dirname $gitdir]]] \
@@ -137,7 +142,7 @@ proc info_popup {msg} {
        global gitdir appname
 
        set title $appname
-       if {$gitdir != {}} {
+       if {$gitdir ne {}} {
                append title { (}
                append title [lindex \
                        [file split [file normalize [file dirname $gitdir]]] \
@@ -162,12 +167,13 @@ if {   [catch {set cdup [exec git rev-parse --show-cdup]} err]
        error_popup "Cannot find the git directory:\n\n$err"
        exit 1
 }
-if {$cdup != ""} {
+if {$cdup ne ""} {
        cd $cdup
 }
 unset cdup
 
-if {$appname == {git-citool}} {
+set single_commit 0
+if {$appname eq {git-citool}} {
        set single_commit 1
 }
 
@@ -175,12 +181,8 @@ if {$appname == {git-citool}} {
 ##
 ## task management
 
-set single_commit 0
 set status_active 0
 set diff_active 0
-set update_active 0
-set commit_active 0
-set update_index_fd {}
 
 set disable_on_lock [list]
 set index_lock_type none
@@ -192,13 +194,13 @@ set commit_type {}
 proc lock_index {type} {
        global index_lock_type disable_on_lock
 
-       if {$index_lock_type == {none}} {
+       if {$index_lock_type eq {none}} {
                set index_lock_type $type
                foreach w $disable_on_lock {
                        uplevel #0 $w disabled
                }
                return 1
-       } elseif {$index_lock_type == {begin-update} && $type == {update}} {
+       } elseif {$index_lock_type eq {begin-update} && $type eq {update}} {
                set index_lock_type $type
                return 1
        }
@@ -240,9 +242,9 @@ proc update_status {{final Ready.}} {
        if {$status_active || ![lock_index read]} return
 
        repository_state new_HEAD new_type
-       if {$commit_type == {amend} 
-               && $new_type == {normal}
-               && $new_HEAD == $HEAD} {
+       if {$commit_type eq {amend}
+               && $new_type eq {normal}
+               && $new_HEAD eq $HEAD} {
        } else {
                set HEAD $new_HEAD
                set PARENT $new_HEAD
@@ -252,7 +254,7 @@ proc update_status {{final Ready.}} {
        array unset file_states
 
        if {![$ui_comm edit modified]
-               || [string trim [$ui_comm get 0.0 end]] == {}} {
+               || [string trim [$ui_comm get 0.0 end]] eq {}} {
                if {[load_message GITGUI_MSG]} {
                } elseif {[load_message MERGE_MSG]} {
                } elseif {[load_message SQUASH_MSG]} {
@@ -261,7 +263,7 @@ proc update_status {{final Ready.}} {
                $ui_comm edit reset
        }
 
-       if {$repo_config(gui.trustmtime) == {true}} {
+       if {$repo_config(gui.trustmtime) eq {true}} {
                update_status_stage2 {} $final
        } else {
                set status_active 1
@@ -284,7 +286,7 @@ proc update_status_stage2 {fd final} {
        global status_active
        global buf_rdi buf_rdf buf_rlo
 
-       if {$fd != {}} {
+       if {$fd ne {}} {
                read $fd
                if {![eof $fd]} return
                close $fd
@@ -440,7 +442,7 @@ proc clear_diff {} {
 proc reshow_diff {} {
        global ui_fname_value ui_status_value file_states
 
-       if {$ui_fname_value == {}
+       if {$ui_fname_value eq {}
                || [catch {set s $file_states($ui_fname_value)}]} {
                clear_diff
        } else {
@@ -453,7 +455,7 @@ proc handle_empty_diff {} {
 
        set path $ui_fname_value
        set s $file_states($path)
-       if {[lindex $s 0] != {_M}} return
+       if {[lindex $s 0] ne {_M}} return
 
        info_popup "No differences detected.
 
@@ -492,7 +494,7 @@ proc show_diff {path {w {}} {lno {}}} {
        if {$diff_active || ![lock_index read]} return
 
        clear_diff
-       if {$w == {} || $lno == {}} {
+       if {$w eq {} || $lno == {}} {
                foreach w [array names file_lists] {
                        set lno [lsearch -sorted $file_lists($w) $path]
                        if {$lno >= 0} {
@@ -501,7 +503,7 @@ proc show_diff {path {w {}} {lno {}}} {
                        }
                }
        }
-       if {$w != {} && $lno >= 1} {
+       if {$w ne {} && $lno >= 1} {
                $w tag add in_diff $lno.0 [expr $lno + 1].0
        }
 
@@ -602,8 +604,8 @@ proc read_diff {fd} {
                unlock_index
                set ui_status_value {Ready.}
 
-               if {$repo_config(gui.trustmtime) == {true}
-                       && [$ui_diff index end] == {2.0}} {
+               if {$repo_config(gui.trustmtime) eq {true}
+                       && [$ui_diff index end] eq {2.0}} {
                        handle_empty_diff
                }
        }
@@ -616,8 +618,8 @@ proc read_diff {fd} {
 proc load_last_commit {} {
        global HEAD PARENT commit_type ui_comm
 
-       if {$commit_type == {amend}} return
-       if {$commit_type != {normal}} {
+       if {$commit_type eq {amend}} return
+       if {$commit_type ne {normal}} {
                error_popup "Can't amend a $commit_type commit."
                return
        }
@@ -661,18 +663,18 @@ proc load_last_commit {} {
 
 proc commit_tree {} {
        global tcl_platform HEAD gitdir commit_type file_states
-       global commit_active ui_status_value
-       global ui_comm
+       global pch_error
+       global ui_status_value ui_comm
 
-       if {$commit_active || ![lock_index update]} return
+       if {![lock_index update]} return
 
        # -- Our in memory state should match the repository.
        #
        repository_state curHEAD cur_type
-       if {$commit_type == {amend} 
-               && $cur_type == {normal}
-               && $curHEAD == $HEAD} {
-       } elseif {$commit_type != $cur_type || $HEAD != $curHEAD} {
+       if {$commit_type eq {amend}
+               && $cur_type eq {normal}
+               && $curHEAD eq $HEAD} {
+       } elseif {$commit_type ne $cur_type || $HEAD ne $curHEAD} {
                error_popup {Last scanned state does not match repository state.
 
 Its highly likely that another Git program modified the
@@ -723,7 +725,7 @@ You must include at least 1 file before you can commit.
        # -- A message is required.
        #
        set msg [string trim [$ui_comm get 1.0 end]]
-       if {$msg == {}} {
+       if {$msg eq {}} {
                error_popup {Please supply a commit message.
 
 A good commit message has the following format:
@@ -739,38 +741,65 @@ A good commit message has the following format:
        # -- Ask the pre-commit hook for the go-ahead.
        #
        set pchook [file join $gitdir hooks pre-commit]
-       if {$tcl_platform(platform) == {windows} && [file isfile $pchook]} {
-               set pchook [list sh -c \
-                       "if test -x \"$pchook\"; then exec \"$pchook\"; fi"]
+       if {$tcl_platform(platform) eq {windows} && [file isfile $pchook]} {
+               set pchook [list sh -c [concat \
+                       "if test -x \"$pchook\";" \
+                       "then exec \"$pchook\" 2>&1;" \
+                       "fi"]]
        } elseif {[file executable $pchook]} {
-               set pchook [list $pchook]
+               set pchook [list $pchook |& cat]
        } else {
                set pchook {}
        }
-       if {$pchook != {} && [catch {eval exec $pchook} err]} {
-               hook_failed_popup pre-commit $err
-               unlock_index
-               return
+       if {$pchook ne {}} {
+               set ui_status_value {Calling pre-commit hook...}
+               set pch_error {}
+               set fd_ph [open "| $pchook" r]
+               fconfigure $fd_ph -blocking 0 -translation binary
+               fileevent $fd_ph readable \
+                       [list commit_stage1 $fd_ph $curHEAD $msg]
+       } else {
+               commit_stage2 $curHEAD $msg
+       }
+}
+
+proc commit_stage1 {fd_ph curHEAD msg} {
+       global pch_error ui_status_value
+
+       append pch_error [read $fd_ph]
+       fconfigure $fd_ph -blocking 1
+       if {[eof $fd_ph]} {
+               if {[catch {close $fd_ph}]} {
+                       set ui_status_value {Commit declined by pre-commit hook.}
+                       hook_failed_popup pre-commit $pch_error
+                       unlock_index
+               } else {
+                       commit_stage2 $curHEAD $msg
+               }
+               set pch_error {}
+       } else {
+               fconfigure $fd_ph -blocking 0
        }
+}
+
+proc commit_stage2 {curHEAD msg} {
+       global ui_status_value
 
        # -- Write the tree in the background.
        #
-       set commit_active 1
        set ui_status_value {Committing changes...}
-
        set fd_wt [open "| git write-tree" r]
-       fileevent $fd_wt readable [list commit_stage2 $fd_wt $curHEAD $msg]
+       fileevent $fd_wt readable [list commit_stage3 $fd_wt $curHEAD $msg]
 }
 
-proc commit_stage2 {fd_wt curHEAD msg} {
-       global single_commit gitdir HEAD PARENT commit_type
-       global commit_active ui_status_value ui_comm
+proc commit_stage3 {fd_wt curHEAD msg} {
+       global single_commit gitdir HEAD PARENT commit_type tcl_platform
+       global ui_status_value ui_comm
        global file_states
 
        gets $fd_wt tree_id
-       if {$tree_id == {} || [catch {close $fd_wt} err]} {
+       if {$tree_id eq {} || [catch {close $fd_wt} err]} {
                error_popup "write-tree failed:\n\n$err"
-               set commit_active 0
                set ui_status_value {Commit failed.}
                unlock_index
                return
@@ -779,10 +808,10 @@ proc commit_stage2 {fd_wt curHEAD msg} {
        # -- Create the commit.
        #
        set cmd [list git commit-tree $tree_id]
-       if {$PARENT != {}} {
+       if {$PARENT ne {}} {
                lappend cmd -p $PARENT
        }
-       if {$commit_type == {merge}} {
+       if {$commit_type eq {merge}} {
                if {[catch {
                                set fd_mh [open [file join $gitdir MERGE_HEAD] r]
                                while {[gets $fd_mh merge_head] >= 0} {
@@ -791,20 +820,18 @@ proc commit_stage2 {fd_wt curHEAD msg} {
                                close $fd_mh
                        } err]} {
                        error_popup "Loading MERGE_HEAD failed:\n\n$err"
-                       set commit_active 0
                        set ui_status_value {Commit failed.}
                        unlock_index
                        return
                }
        }
-       if {$PARENT == {}} {
+       if {$PARENT eq {}} {
                # git commit-tree writes to stderr during initial commit.
                lappend cmd 2>/dev/null
        }
        lappend cmd << $msg
        if {[catch {set cmt_id [eval exec $cmd]} err]} {
                error_popup "commit-tree failed:\n\n$err"
-               set commit_active 0
                set ui_status_value {Commit failed.}
                unlock_index
                return
@@ -813,7 +840,7 @@ proc commit_stage2 {fd_wt curHEAD msg} {
        # -- Update the HEAD ref.
        #
        set reflogm commit
-       if {$commit_type != {normal}} {
+       if {$commit_type ne {normal}} {
                append reflogm " ($commit_type)"
        }
        set i [string first "\n" $msg]
@@ -825,7 +852,6 @@ proc commit_stage2 {fd_wt curHEAD msg} {
        set cmd [list git update-ref -m $reflogm HEAD $cmt_id $curHEAD]
        if {[catch {eval exec $cmd} err]} {
                error_popup "update-ref failed:\n\n$err"
-               set commit_active 0
                set ui_status_value {Commit failed.}
                unlock_index
                return
@@ -844,6 +870,21 @@ proc commit_stage2 {fd_wt curHEAD msg} {
                catch {exec git rerere}
        }
 
+       # -- Run the post-commit hook.
+       #
+       set pchook [file join $gitdir hooks post-commit]
+       if {$tcl_platform(platform) eq {windows} && [file isfile $pchook]} {
+               set pchook [list sh -c [concat \
+                       "if test -x \"$pchook\";" \
+                       "then exec \"$pchook\";" \
+                       "fi"]]
+       } elseif {![file executable $pchook]} {
+               set pchook {}
+       }
+       if {$pchook ne {}} {
+               catch {exec $pchook &}
+       }
+
        $ui_comm delete 0.0 end
        $ui_comm edit modified false
        $ui_comm edit reset
@@ -852,7 +893,6 @@ proc commit_stage2 {fd_wt curHEAD msg} {
 
        # -- Update status without invoking any git commands.
        #
-       set commit_active 0
        set commit_type normal
        set HEAD $cmt_id
        set PARENT $cmt_id
@@ -866,7 +906,7 @@ proc commit_stage2 {fd_wt curHEAD msg} {
                D? {set m _[string index $m 1]}
                }
 
-               if {$m == {__}} {
+               if {$m eq {__}} {
                        unset file_states($path)
                } else {
                        lset file_states($path) 0 $m
@@ -893,15 +933,14 @@ proc fetch_from {remote} {
 }
 
 proc pull_remote {remote branch} {
-       global HEAD commit_type
-       global file_states
+       global HEAD commit_type file_states repo_config
 
        if {![lock_index update]} return
 
        # -- Our in memory state should match the repository.
        #
        repository_state curHEAD cur_type
-       if {$commit_type != $cur_type || $HEAD != $curHEAD} {
+       if {$commit_type ne $cur_type || $HEAD ne $curHEAD} {
                error_popup {Last scanned state does not match repository state.
 
 Its highly likely that another Git program modified the
@@ -930,6 +969,9 @@ Commit or throw away all changes before starting a pull operation.
        set w [new_console "pull $remote $branch" \
                "Pulling new changes from branch $branch in $remote"]
        set cmd [list git pull]
+       if {$repo_config(gui.pullsummary) eq {false}} {
+               lappend cmd --no-summary
+       }
        lappend cmd $remote
        lappend cmd $branch
        console_exec $w $cmd [list post_pull_remote $remote $branch]
@@ -1017,15 +1059,15 @@ proc merge_state {path new_state} {
                set icon [lindex $info 1]
        }
 
-       if {$s0 == {_}} {
+       if {$s0 eq {_}} {
                set s0 [string index $state 0]
-       } elseif {$s0 == {*}} {
+       } elseif {$s0 eq {*}} {
                set s0 _
        }
 
-       if {$s1 == {_}} {
+       if {$s1 eq {_}} {
                set s1 [string index $state 1]
-       } elseif {$s1 == {*}} {
+       } elseif {$s1 eq {*}} {
                set s1 _
        }
 
@@ -1045,7 +1087,7 @@ proc display_file {path state} {
        set old_w [mapcol $old_m $path]
        set new_icon [mapicon $new_m $path]
 
-       if {$new_w != $old_w} {
+       if {$new_w ne $old_w} {
                set lno [lsearch -sorted $file_lists($old_w) $path]
                if {$lno >= 0} {
                        incr lno
@@ -1065,7 +1107,7 @@ proc display_file {path state} {
                        -image $new_icon
                $new_w insert $lno.1 "[escape_path $path]\n"
                $new_w conf -state disabled
-       } elseif {$new_icon != [mapicon $old_m $path]} {
+       } elseif {$new_icon ne [mapicon $old_m $path]} {
                $new_w conf -state normal
                $new_w image conf [lindex $s 1] -image $new_icon
                $new_w conf -state disabled
@@ -1100,55 +1142,73 @@ proc display_all_files {} {
        $ui_other conf -state disabled
 }
 
-proc with_update_index {body} {
-       global update_index_fd
+proc update_index {pathList} {
+       global update_index_cp ui_status_value
 
-       if {$update_index_fd == {}} {
-               if {![lock_index update]} return
-               set update_index_fd [open \
-                       "| git update-index --add --remove -z --stdin" \
-                       w]
-               fconfigure $update_index_fd -translation binary
-               uplevel 1 $body
-               close $update_index_fd
-               set update_index_fd {}
-               unlock_index
-       } else {
-               uplevel 1 $body
-       }
-}
+       if {![lock_index update]} return
 
-proc update_index {path} {
-       global update_index_fd
+       set update_index_cp 0
+       set totalCnt [llength $pathList]
+       set batch [expr {int($totalCnt * .01) + 1}]
+       if {$batch > 25} {set batch 25}
+
+       set ui_status_value "Including files ... 0/$totalCnt 0%"
+       set ui_status_value [format \
+               "Including files ... %i/%i files (%.2f%%)" \
+               $update_index_cp \
+               $totalCnt \
+               0.0]
+       set fd [open "| git update-index --add --remove -z --stdin" w]
+       fconfigure $fd -blocking 0 -translation binary
+       fileevent $fd writable [list \
+               write_update_index \
+               $fd \
+               $pathList \
+               $totalCnt \
+               $batch \
+               ]
+}
+
+proc write_update_index {fd pathList totalCnt batch} {
+       global update_index_cp ui_status_value
+       global file_states ui_fname_value
 
-       if {$update_index_fd == {}} {
-               error {not in with_update_index}
-       } else {
-               puts -nonewline $update_index_fd "$path\0"
+       if {$update_index_cp >= $totalCnt} {
+               close $fd
+               unlock_index
+               set ui_status_value {Ready.}
+               return
        }
-}
 
-proc toggle_mode {path} {
-       global file_states ui_fname_value
-
-       set s $file_states($path)
-       set m [lindex $s 0]
+       for {set i $batch} \
+               {$update_index_cp < $totalCnt && $i > 0} \
+               {incr i -1} {
+               set path [lindex $pathList $update_index_cp]
+               incr update_index_cp
+
+               switch -- [lindex $file_states($path) 0] {
+               AM -
+               _O {set new A*}
+               _M -
+               MM {set new M*}
+               AD -
+               _D {set new D*}
+               default {continue}
+               }
 
-       switch -- $m {
-       AM -
-       _O {set new A*}
-       _M -
-       MM {set new M*}
-       AD -
-       _D {set new D*}
-       default {return}
+               puts -nonewline $fd $path
+               puts -nonewline $fd "\0"
+               display_file $path $new
+               if {$ui_fname_value eq $path} {
+                       show_diff $path
+               }
        }
 
-       with_update_index {update_index $path}
-       display_file $path $new
-       if {$ui_fname_value == $path} {
-               show_diff $path
-       }
+       set ui_status_value [format \
+               "Including files ... %i/%i files (%.2f%%)" \
+               $update_index_cp \
+               $totalCnt \
+               [expr {100.0 * $update_index_cp / $totalCnt}]]
 }
 
 ######################################################################
@@ -1192,8 +1252,8 @@ proc populate_pull_menu {m} {
 
        foreach remote $all_remotes {
                set rb {}
-               if {[array get repo_config remote.$remote.url] != {}} {
-                       if {[array get repo_config remote.$remote.fetch] != {}} {
+               if {[array get repo_config remote.$remote.url] ne {}} {
+                       if {[array get repo_config remote.$remote.fetch] ne {}} {
                                regexp {^([^:]+):} \
                                        [lindex $repo_config(remote.$remote.fetch) 0] \
                                        line rb
@@ -1212,7 +1272,7 @@ proc populate_pull_menu {m} {
 
                set rb_short $rb
                regsub ^refs/heads/ $rb {} rb_short
-               if {$rb_short != {}} {
+               if {$rb_short ne {}} {
                        $m add command \
                                -label "Branch $rb_short from $remote..." \
                                -command [list pull_remote $remote $rb] \
@@ -1324,7 +1384,7 @@ foreach i {
        if {$max_status_desc < [string length [lindex $i 3]]} {
                set max_status_desc [string length [lindex $i 3]]
        }
-       if {[lindex $i 1] == {i}} {
+       if {[lindex $i 1] eq {i}} {
                set all_cols([lindex $i 0]) $ui_index
        } else {
                set all_cols([lindex $i 0]) $ui_other
@@ -1340,8 +1400,8 @@ unset filemask i
 
 proc is_MacOSX {} {
        global tcl_platform tk_library
-       if {$tcl_platform(platform) == {unix}
-               && $tcl_platform(os) == {Darwin}
+       if {$tcl_platform(platform) eq {unix}
+               && $tcl_platform(os) eq {Darwin}
                && [string match /Library/Frameworks/* $tk_library]} {
                return 1
        }
@@ -1483,7 +1543,7 @@ proc console_exec {w cmd {after {}}} {
        # -- Windows tosses the enviroment when we exec our child.
        #    But most users need that so we have to relogin. :-(
        #
-       if {$tcl_platform(platform) == {windows}} {
+       if {$tcl_platform(platform) eq {windows}} {
                set cmd [list sh --login -c "cd \"[pwd]\" && [join $cmd { }]"]
        }
 
@@ -1501,7 +1561,7 @@ proc console_read {w fd after} {
        global console_cr console_data
 
        set buf [read $fd]
-       if {$buf != {}} {
+       if {$buf ne {}} {
                if {![winfo exists $w]} {console_init $w}
                $w.m.t conf -state normal
                set c 0
@@ -1545,7 +1605,7 @@ proc console_read {w fd after} {
                }
                array unset console_cr $w
                array unset console_data $w
-               if {$after != {}} {
+               if {$after ne {}} {
                        uplevel #0 $after $ok
                }
                return
@@ -1564,12 +1624,12 @@ proc do_gitk {} {
 
        set ui_status_value $starting_gitk_msg
        after 10000 {
-               if {$ui_status_value == $starting_gitk_msg} {
+               if {$ui_status_value eq $starting_gitk_msg} {
                        set ui_status_value {Ready.}
                }
        }
 
-       if {$tcl_platform(platform) == {windows}} {
+       if {$tcl_platform(platform) eq {windows}} {
                exec sh -c gitk &
        } else {
                exec gitk &
@@ -1596,13 +1656,13 @@ proc do_quit {} {
        #
        set save [file join $gitdir GITGUI_MSG]
        set msg [string trim [$ui_comm get 0.0 end]]
-       if {[$ui_comm edit modified] && $msg != {}} {
+       if {[$ui_comm edit modified] && $msg ne {}} {
                catch {
                        set fd [open $save w]
                        puts $fd [string trim [$ui_comm get 0.0 end]]
                        close $fd
                }
-       } elseif {$msg == {} && [file exists $save]} {
+       } elseif {$msg eq {} && [file exists $save]} {
                file delete $save
        }
 
@@ -1615,7 +1675,7 @@ proc do_quit {} {
        if {[catch {set rc_geometry $repo_config(gui.geometry)}]} {
                set rc_geometry {}
        }
-       if {$cfg_geometry != $rc_geometry} {
+       if {$cfg_geometry ne $rc_geometry} {
                catch {exec git repo-config gui.geometry $cfg_geometry}
        }
 
@@ -1627,27 +1687,25 @@ proc do_rescan {} {
 }
 
 proc do_include_all {} {
-       global update_active ui_status_value
-
-       if {$update_active || ![lock_index begin-update]} return
-
-       set update_active 1
-       set ui_status_value {Including all modified files...}
-       after 1 {
-               with_update_index {
-                       foreach path [array names file_states] {
-                               set s $file_states($path)
-                               set m [lindex $s 0]
-                               switch -- $m {
-                               AM -
-                               MM -
-                               _M -
-                               _D {toggle_mode $path}
-                               }
-                       }
+       global file_states
+
+       if {![lock_index begin-update]} return
+
+       set pathList [list]
+       foreach path [array names file_states] {
+               set s $file_states($path)
+               set m [lindex $s 0]
+               switch -- $m {
+               AM -
+               MM -
+               _M -
+               _D {lappend pathList $path}
                }
-               set update_active 0
-               set ui_status_value {Ready.}
+       }
+       if {$pathList eq {}} {
+               unlock_index
+       } else {
+               update_index $pathList
        }
 }
 
@@ -1656,7 +1714,7 @@ set GIT_COMMITTER_IDENT {}
 proc do_signoff {} {
        global ui_comm GIT_COMMITTER_IDENT
 
-       if {$GIT_COMMITTER_IDENT == {}} {
+       if {$GIT_COMMITTER_IDENT eq {}} {
                if {[catch {set me [exec git var GIT_COMMITTER_IDENT]} err]} {
                        error_popup "Unable to obtain your identity:\n\n$err"
                        return
@@ -1670,9 +1728,9 @@ proc do_signoff {} {
 
        set sob "Signed-off-by: $GIT_COMMITTER_IDENT"
        set last [$ui_comm get {end -1c linestart} {end -1c}]
-       if {$last != $sob} {
+       if {$last ne $sob} {
                $ui_comm edit separator
-               if {$last != {}
+               if {$last ne {}
                        && ![regexp {^[A-Z][A-Za-z]*-[A-Za-z-]+: *} $last]} {
                        $ui_comm insert end "\n"
                }
@@ -1695,7 +1753,7 @@ proc do_options {} {
        global repo_config global_config
        global repo_config_new global_config_new
 
-       load_config
+       load_config 1
        array unset repo_config_new
        array unset global_config_new
        foreach name [array names repo_config] {
@@ -1704,9 +1762,13 @@ proc do_options {} {
        foreach name [array names global_config] {
                set global_config_new($name) $global_config($name)
        }
+       set reponame [lindex [file split \
+               [file normalize [file dirname $gitdir]]] \
+               end]
 
        set w .options_editor
        toplevel $w
+       wm geometry $w "+[winfo rootx .]+[winfo rooty .]"
 
        label $w.header -text "$appname Options" \
                -font font_uibold
@@ -1727,7 +1789,7 @@ proc do_options {} {
        pack $w.buttons.cancel -side right
        pack $w.buttons -side bottom -fill x -pady 10 -padx 10
 
-       labelframe $w.repo -text {This Repository} \
+       labelframe $w.repo -text "$reponame Repository" \
                -font font_ui \
                -relief raised -borderwidth 2
        labelframe $w.global -text {Global (All Repositories)} \
@@ -1737,6 +1799,7 @@ proc do_options {} {
        pack $w.global -side right -fill both -expand 1 -pady 5 -padx 5
 
        foreach option {
+               {pullsummary {Show Pull Summary}}
                {trustmtime {Trust File Modification Timestamps}}
                } {
                set name [lindex $option 0]
@@ -1780,14 +1843,12 @@ proc do_options {} {
 
        bind $w <Visibility> "grab $w; focus $w"
        bind $w <Key-Escape> "destroy $w"
-       wm title $w "$appname ([lindex [file split \
-               [file normalize [file dirname $gitdir]]] \
-               end]): Options"
+       wm title $w "$appname ($reponame): Options"
        tkwait window $w
 }
 
 proc do_restore_defaults {} {
-       global font_descs default_config
+       global font_descs default_config repo_config
        global repo_config_new global_config_new
 
        foreach name [array names default_config] {
@@ -1797,7 +1858,7 @@ proc do_restore_defaults {} {
 
        foreach option $font_descs {
                set name [lindex $option 0]
-               set repo_config($name) $default_config(gui.$name)
+               set repo_config(gui.$name) $default_config(gui.$name)
        }
        apply_config
 
@@ -1827,7 +1888,7 @@ proc click {w x y shift wx wy} {
        set lno [lindex $pos 0]
        set col [lindex $pos 1]
        set path [lindex $file_lists($w) [expr $lno - 1]]
-       if {$path == {}} return
+       if {$path eq {}} return
 
        if {$col > 0 && $shift == 1} {
                show_diff $path $w $lno
@@ -1841,10 +1902,10 @@ proc unclick {w x y} {
        set lno [lindex $pos 0]
        set col [lindex $pos 1]
        set path [lindex $file_lists($w) [expr $lno - 1]]
-       if {$path == {}} return
+       if {$path eq {}} return
 
        if {$col == 0} {
-               toggle_mode $path
+               update_index [list $path]
        }
 }
 
@@ -1866,7 +1927,7 @@ font create font_diffbold
 
 set M1B M1
 set M1T M1
-if {$tcl_platform(platform) == {windows}} {
+if {$tcl_platform(platform) eq {windows}} {
        set M1B Control
        set M1T Ctrl
 } elseif {[is_MacOSX]} {
@@ -1895,13 +1956,14 @@ proc apply_config {} {
 }
 
 set default_config(gui.trustmtime) false
+set default_config(gui.pullsummary) true
 set default_config(gui.fontui) [font configure font_ui]
 set default_config(gui.fontdiff) [font configure font_diff]
 set font_descs {
        {fontui   font_ui   {Main Font}}
        {fontdiff font_diff {Diff/Console Font}}
 }
-load_config
+load_config 0
 apply_config
 
 ######################################################################
@@ -1913,9 +1975,11 @@ menu .mbar -tearoff 0
 .mbar add cascade -label Project -menu .mbar.project
 .mbar add cascade -label Edit -menu .mbar.edit
 .mbar add cascade -label Commit -menu .mbar.commit
-.mbar add cascade -label Fetch -menu .mbar.fetch
-.mbar add cascade -label Pull -menu .mbar.pull
-.mbar add cascade -label Push -menu .mbar.push
+if {!$single_commit} {
+       .mbar add cascade -label Fetch -menu .mbar.fetch
+       .mbar add cascade -label Pull -menu .mbar.pull
+       .mbar add cascade -label Push -menu .mbar.push
+}
 . configure -menu .mbar
 
 # -- Project Menu
@@ -1923,9 +1987,11 @@ menu .mbar.project
 .mbar.project add command -label Visualize \
        -command do_gitk \
        -font font_ui
-.mbar.project add command -label {Repack Database} \
-       -command do_repack \
-       -font font_ui
+if {!$single_commit} {
+       .mbar.project add command -label {Repack Database} \
+               -command do_repack \
+               -font font_ui
+}
 .mbar.project add command -label Quit \
        -command do_quit \
        -accelerator $M1T-Q \
@@ -1999,14 +2065,16 @@ lappend disable_on_lock \
 lappend disable_on_lock \
        [list .mbar.commit entryconf [.mbar.commit index last] -state]
 
-# -- Fetch Menu
-menu .mbar.fetch
+if {!$single_commit} {
+       # -- Fetch Menu
+       menu .mbar.fetch
 
-# -- Pull Menu
-menu .mbar.pull
+       # -- Pull Menu
+       menu .mbar.pull
 
-# -- Push Menu
-menu .mbar.push
+       # -- Push Menu
+       menu .mbar.push
+}
 
 # -- Main Window Layout
 panedwindow .vpane -orient vertical
@@ -2052,7 +2120,7 @@ $ui_index tag conf in_diff -font font_uibold
 $ui_other tag conf in_diff -font font_uibold
 
 # -- Diff and Commit Area
-frame .vpane.lower -height 400 -width 400
+frame .vpane.lower -height 300 -width 400
 frame .vpane.lower.commarea
 frame .vpane.lower.diff -relief sunken -borderwidth 1
 pack .vpane.lower.commarea -side top -fill x
@@ -2238,6 +2306,9 @@ $ui_diff.ctxm add command -label "Decrease Font Size" \
 $ui_diff.ctxm add command -label "Increase Font Size" \
        -font font_ui \
        -command {incr_font_size font_diff 1}
+$ui_diff.ctxm add command -label {Options...} \
+       -font font_ui \
+       -command do_options
 bind_button3 $ui_diff "tk_popup $ui_diff.ctxm %X %Y"
 
 # -- Status Bar
@@ -2314,8 +2385,10 @@ set file_lists($ui_other) [list]
 
 wm title . "$appname ([file normalize [file dirname $gitdir]])"
 focus -force $ui_comm
-load_all_remotes
-populate_remote_menu .mbar.fetch From fetch_from
-populate_remote_menu .mbar.push To push_to
-populate_pull_menu .mbar.pull
-update_status
+if {!$single_commit} {
+       load_all_remotes
+       populate_remote_menu .mbar.fetch From fetch_from
+       populate_remote_menu .mbar.push To push_to
+       populate_pull_menu .mbar.pull
+}
+after 1 update_status