git-gui: Automatically update-index all included files before commit.
[gitweb.git] / git-gui
diff --git a/git-gui b/git-gui
index b8ebe31777ddfd2eff64c2fb5e44050321fc8a86..9a6953e970bd5a499f13404559122a903546cf30 100755 (executable)
--- a/git-gui
+++ b/git-gui
@@ -181,8 +181,9 @@ if {$appname eq {git-citool}} {
 ##
 ## task management
 
-set status_active 0
+set rescan_active 0
 set diff_active 0
+set last_clicked {}
 
 set disable_on_lock [list]
 set index_lock_type none
@@ -233,13 +234,13 @@ proc repository_state {hdvar ctvar} {
        }
 }
 
-proc update_status {{final Ready.}} {
+proc rescan {after} {
        global HEAD PARENT commit_type
        global ui_index ui_other ui_status_value ui_comm
-       global status_active file_states
+       global rescan_active file_states
        global repo_config
 
-       if {$status_active || ![lock_index read]} return
+       if {$rescan_active > 0 || ![lock_index read]} return
 
        repository_state new_HEAD new_type
        if {$commit_type eq {amend}
@@ -264,9 +265,9 @@ proc update_status {{final Ready.}} {
        }
 
        if {$repo_config(gui.trustmtime) eq {true}} {
-               update_status_stage2 {} $final
+               rescan_stage2 {} $after
        } else {
-               set status_active 1
+               set rescan_active 1
                set ui_status_value {Refreshing file status...}
                set cmd [list git update-index]
                lappend cmd -q
@@ -276,14 +277,14 @@ proc update_status {{final Ready.}} {
                set fd_rf [open "| $cmd" r]
                fconfigure $fd_rf -blocking 0 -translation binary
                fileevent $fd_rf readable \
-                       [list update_status_stage2 $fd_rf $final]
+                       [list rescan_stage2 $fd_rf $after]
        }
 }
 
-proc update_status_stage2 {fd final} {
+proc rescan_stage2 {fd after} {
        global gitdir PARENT commit_type
        global ui_index ui_other ui_status_value ui_comm
-       global status_active
+       global rescan_active
        global buf_rdi buf_rdf buf_rlo
 
        if {$fd ne {}} {
@@ -303,7 +304,7 @@ proc update_status_stage2 {fd final} {
        set buf_rdf {}
        set buf_rlo {}
 
-       set status_active 3
+       set rescan_active 3
        set ui_status_value {Scanning for modified files ...}
        set fd_di [open "| git diff-index --cached -z $PARENT" r]
        set fd_df [open "| git diff-files -z" r]
@@ -312,9 +313,9 @@ proc update_status_stage2 {fd final} {
        fconfigure $fd_di -blocking 0 -translation binary
        fconfigure $fd_df -blocking 0 -translation binary
        fconfigure $fd_lo -blocking 0 -translation binary
-       fileevent $fd_di readable [list read_diff_index $fd_di $final]
-       fileevent $fd_df readable [list read_diff_files $fd_df $final]
-       fileevent $fd_lo readable [list read_ls_others $fd_lo $final]
+       fileevent $fd_di readable [list read_diff_index $fd_di $after]
+       fileevent $fd_df readable [list read_diff_files $fd_df $after]
+       fileevent $fd_lo readable [list read_ls_others $fd_lo $after]
 }
 
 proc load_message {file} {
@@ -334,7 +335,7 @@ proc load_message {file} {
        return 0
 }
 
-proc read_diff_index {fd final} {
+proc read_diff_index {fd after} {
        global buf_rdi
 
        append buf_rdi [read $fd]
@@ -351,7 +352,7 @@ proc read_diff_index {fd final} {
                incr z2 -1
                display_file \
                        [string range $buf_rdi $z1 $z2] \
-                       [string index $buf_rdi [expr $z1 - 2]]_
+                       [string index $buf_rdi [expr {$z1 - 2}]]_
                incr c
        }
        if {$c < $n} {
@@ -360,10 +361,10 @@ proc read_diff_index {fd final} {
                set buf_rdi {}
        }
 
-       status_eof $fd buf_rdi $final
+       rescan_done $fd buf_rdi $after
 }
 
-proc read_diff_files {fd final} {
+proc read_diff_files {fd after} {
        global buf_rdf
 
        append buf_rdf [read $fd]
@@ -380,7 +381,7 @@ proc read_diff_files {fd final} {
                incr z2 -1
                display_file \
                        [string range $buf_rdf $z1 $z2] \
-                       _[string index $buf_rdf [expr $z1 - 2]]
+                       _[string index $buf_rdf [expr {$z1 - 2}]]
                incr c
        }
        if {$c < $n} {
@@ -389,10 +390,10 @@ proc read_diff_files {fd final} {
                set buf_rdf {}
        }
 
-       status_eof $fd buf_rdf $final
+       rescan_done $fd buf_rdf $after
 }
 
-proc read_ls_others {fd final} {
+proc read_ls_others {fd after} {
        global buf_rlo
 
        append buf_rlo [read $fd]
@@ -401,24 +402,25 @@ proc read_ls_others {fd final} {
        foreach p [lrange $pck 0 end-1] {
                display_file $p _O
        }
-       status_eof $fd buf_rlo $final
+       rescan_done $fd buf_rlo $after
 }
 
-proc status_eof {fd buf final} {
-       global status_active ui_status_value
+proc rescan_done {fd buf after} {
+       global rescan_active
        global file_states repo_config
        upvar $buf to_clear
 
        if {![eof $fd]} return
        set to_clear {}
        close $fd
-       if {[incr status_active -1] > 0} return
+       if {[incr rescan_active -1] > 0} return
 
+       prune_selection
        unlock_index
        display_all_files
 
        if {$repo_config(gui.partialinclude) ne {true}} {
-               set pathList  [list]
+               set pathList [list]
                foreach path [array names file_states] {
                        switch -- [lindex $file_states($path) 0] {
                        AM -
@@ -426,13 +428,26 @@ proc status_eof {fd buf final} {
                        }
                }
                if {$pathList ne {}} {
-                       update_index $pathList
+                       update_index \
+                               "Updating included files" \
+                               $pathList \
+                               [concat {reshow_diff;} $after]
                        return
                }
        }
 
        reshow_diff
-       set ui_status_value $final
+       uplevel #0 $after
+}
+
+proc prune_selection {} {
+       global file_states selected_paths
+
+       foreach path [array names selected_paths] {
+               if {[catch {set still_here $file_states($path)}]} {
+                       unset selected_paths($path)
+               }
+       }
 }
 
 ######################################################################
@@ -475,13 +490,15 @@ proc handle_empty_diff {} {
 
 [short_path $path] has no changes.
 
-The modification date of this file was updated by another
-application and you currently have the Trust File Modification
-Timestamps option enabled, so Git did not automatically detect
-that there are no content differences in this file.
+The modification date of this file was updated
+by another application and you currently have
+the Trust File Modification Timestamps option
+enabled, so Git did not automatically detect
+that there are no content differences in this
+file.
 
-This file will now be removed from the modified files list, to
-prevent possible confusion.
+This file will now be removed from the modified
+files list, to prevent possible confusion.
 "
        if {[catch {exec git update-index -- $path} err]} {
                error_popup "Failed to refresh index:\n\n$err"
@@ -495,7 +512,7 @@ prevent possible confusion.
                        [lreplace $file_lists($old_w) $lno $lno]
                incr lno
                $old_w conf -state normal
-               $old_w delete $lno.0 [expr $lno + 1].0
+               $old_w delete $lno.0 [expr {$lno + 1}].0
                $old_w conf -state disabled
        }
 }
@@ -518,7 +535,7 @@ proc show_diff {path {w {}} {lno {}}} {
                }
        }
        if {$w ne {} && $lno >= 1} {
-               $w tag add in_diff $lno.0 [expr $lno + 1].0
+               $w tag add in_diff $lno.0 [expr {$lno + 1}].0
        }
 
        set s $file_states($path)
@@ -670,7 +687,7 @@ proc load_last_commit {} {
                set commit_type amend
                set HEAD {}
                set PARENT {}
-               update_status
+               rescan {set ui_status_value {Ready.}}
        } elseif {$parent_count == 1} {
                set commit_type amend
                set PARENT $parent
@@ -678,7 +695,7 @@ proc load_last_commit {} {
                $ui_comm insert end $msg
                $ui_comm edit modified false
                $ui_comm edit reset
-               update_status
+               rescan {set ui_status_value {Ready.}}
        } else {
                error_popup {You can't amend a merge commit.}
                return
@@ -686,9 +703,7 @@ proc load_last_commit {} {
 }
 
 proc commit_tree {} {
-       global tcl_platform HEAD gitdir commit_type file_states
-       global pch_error
-       global ui_status_value ui_comm
+       global HEAD commit_type file_states ui_comm repo_config
 
        if {![lock_index update]} return
 
@@ -702,11 +717,13 @@ proc commit_tree {} {
                error_popup {Last scanned state does not match repository state.
 
 Its highly likely that another Git program modified the
-repository since our last scan.  A rescan is required
+repository since the last scan.  A rescan is required
 before committing.
+
+A rescan will be automatically started now.
 }
                unlock_index
-               update_status
+               rescan {set ui_status_value {Ready.}}
                return
        }
 
@@ -714,8 +731,7 @@ before committing.
        #
        set files_ready 0
        foreach path [array names file_states] {
-               set s $file_states($path)
-               switch -glob -- [lindex $s 0] {
+               switch -glob -- [lindex $file_states($path) 0] {
                _? {continue}
                A? -
                D? -
@@ -762,10 +778,39 @@ A good commit message has the following format:
                return
        }
 
-       # -- Ask the pre-commit hook for the go-ahead.
+       # -- Update included files if partialincludes are off.
        #
+       if {$repo_config(gui.partialinclude) ne {true}} {
+               set pathList [list]
+               foreach path [array names file_states] {
+                       switch -glob -- [lindex $file_states($path) 0] {
+                       A? -
+                       M? {lappend pathList $path}
+                       }
+               }
+               if {$pathList ne {}} {
+                       unlock_index
+                       update_index \
+                               "Updating included files" \
+                               $pathList \
+                               [concat {lock_index update;} \
+                                       [list commit_prehook $curHEAD $msg]]
+                       return
+               }
+       }
+
+       commit_prehook $curHEAD $msg
+}
+
+proc commit_prehook {curHEAD msg} {
+       global tcl_platform gitdir ui_status_value pch_error
+
+       # On Cygwin [file executable] might lie so we need to ask
+       # the shell if the hook is executable.  Yes that's annoying.
+
        set pchook [file join $gitdir hooks pre-commit]
-       if {$tcl_platform(platform) eq {windows} && [file isfile $pchook]} {
+       if {$tcl_platform(platform) eq {windows}
+               && [file isfile $pchook]} {
                set pchook [list sh -c [concat \
                        "if test -x \"$pchook\";" \
                        "then exec \"$pchook\" 2>&1;" \
@@ -773,21 +818,19 @@ A good commit message has the following format:
        } elseif {[file executable $pchook]} {
                set pchook [list $pchook |& cat]
        } else {
-               set pchook {}
-       }
-       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
+               commit_writetree $curHEAD $msg
+               return
        }
+
+       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_prehook_wait $fd_ph $curHEAD $msg]
 }
 
-proc commit_stage1 {fd_ph curHEAD msg} {
+proc commit_prehook_wait {fd_ph curHEAD msg} {
        global pch_error ui_status_value
 
        append pch_error [read $fd_ph]
@@ -798,28 +841,27 @@ proc commit_stage1 {fd_ph curHEAD msg} {
                        hook_failed_popup pre-commit $pch_error
                        unlock_index
                } else {
-                       commit_stage2 $curHEAD $msg
+                       commit_writetree $curHEAD $msg
                }
                set pch_error {}
-       } else {
-               fconfigure $fd_ph -blocking 0
+               return
        }
+       fconfigure $fd_ph -blocking 0
 }
 
-proc commit_stage2 {curHEAD msg} {
+proc commit_writetree {curHEAD msg} {
        global ui_status_value
 
-       # -- Write the tree in the background.
-       #
        set ui_status_value {Committing changes...}
        set fd_wt [open "| git write-tree" r]
-       fileevent $fd_wt readable [list commit_stage3 $fd_wt $curHEAD $msg]
+       fileevent $fd_wt readable \
+               [list commit_committree $fd_wt $curHEAD $msg]
 }
 
-proc commit_stage3 {fd_wt curHEAD msg} {
+proc commit_committree {fd_wt curHEAD msg} {
        global single_commit gitdir HEAD PARENT commit_type tcl_platform
        global ui_status_value ui_comm
-       global file_states
+       global file_states selected_paths
 
        gets $fd_wt tree_id
        if {$tree_id eq {} || [catch {close $fd_wt} err]} {
@@ -869,7 +911,7 @@ proc commit_stage3 {fd_wt curHEAD msg} {
        }
        set i [string first "\n" $msg]
        if {$i >= 0} {
-               append reflogm {: } [string range $msg 0 [expr $i - 1]]
+               append reflogm {: } [string range $msg 0 [expr {$i - 1}]]
        } else {
                append reflogm {: } $msg
        }
@@ -932,6 +974,7 @@ proc commit_stage3 {fd_wt curHEAD msg} {
 
                if {$m eq {__}} {
                        unset file_states($path)
+                       catch {unset selected_paths($path)}
                } else {
                        lset file_states($path) 0 $m
                }
@@ -972,7 +1015,7 @@ repository since our last scan.  A rescan is required
 before a pull can be started.
 }
                unlock_index
-               update_status
+               rescan {set ui_status_value {Ready.}}
                return
        }
 
@@ -1009,10 +1052,10 @@ proc post_pull_remote {remote branch success} {
        if {$success} {
                repository_state HEAD commit_type
                set PARENT $HEAD
-               set $ui_status_value {Ready.}
+               set $ui_status_value "Pulling $branch from $remote complete."
        } else {
-               update_status \
-                       "Conflicts detected while pulling $branch from $remote."
+               set m "Conflicts detected while pulling $branch from $remote."
+               rescan "set ui_status_value {$m}"
        }
 }
 
@@ -1100,10 +1143,10 @@ proc merge_state {path new_state} {
 }
 
 proc display_file {path state} {
-       global file_states file_lists status_active
+       global file_states file_lists selected_paths rescan_active
 
        set old_m [merge_state $path $state]
-       if {$status_active} return
+       if {$rescan_active > 0} return
 
        set s $file_states($path)
        set new_m [lindex $s 0]
@@ -1116,7 +1159,7 @@ proc display_file {path state} {
                if {$lno >= 0} {
                        incr lno
                        $old_w conf -state normal
-                       $old_w delete $lno.0 [expr $lno + 1].0
+                       $old_w delete $lno.0 [expr {$lno + 1}].0
                        $old_w conf -state disabled
                }
 
@@ -1130,6 +1173,12 @@ proc display_file {path state} {
                        -name [lindex $s 1] \
                        -image $new_icon
                $new_w insert $lno.1 "[escape_path $path]\n"
+               if {[catch {set in_sel $selected_paths($path)}]} {
+                       set in_sel 0
+               }
+               if {$in_sel} {
+                       $new_w tag add in_sel $lno.0 [expr {$lno + 1}].0
+               }
                $new_w conf -state disabled
        } elseif {$new_icon ne [mapicon $old_m $path]} {
                $new_w conf -state normal
@@ -1139,13 +1188,16 @@ proc display_file {path state} {
 }
 
 proc display_all_files {} {
-       global ui_index ui_other file_states file_lists
+       global ui_index ui_other
+       global file_states file_lists
+       global last_clicked selected_paths
 
        $ui_index conf -state normal
        $ui_other conf -state normal
 
        $ui_index delete 0.0 end
        $ui_other delete 0.0 end
+       set last_clicked {}
 
        set file_lists($ui_index) [list]
        set file_lists($ui_other) [list]
@@ -1155,18 +1207,25 @@ proc display_all_files {} {
                set m [lindex $s 0]
                set w [mapcol $m $path]
                lappend file_lists($w) $path
+               set lno [expr {[lindex [split [$w index end] .] 0] - 1}]
                $w image create end \
                        -align center -padx 5 -pady 1 \
                        -name [lindex $s 1] \
                        -image [mapicon $m $path]
                $w insert end "[escape_path $path]\n"
+               if {[catch {set in_sel $selected_paths($path)}]} {
+                       set in_sel 0
+               }
+               if {$in_sel} {
+                       $w tag add in_sel $lno.0 [expr {$lno + 1}].0
+               }
        }
 
        $ui_index conf -state disabled
        $ui_other conf -state disabled
 }
 
-proc update_index {pathList} {
+proc update_index {msg pathList after} {
        global update_index_cp update_index_rsd ui_status_value
 
        if {![lock_index update]} return
@@ -1179,7 +1238,7 @@ proc update_index {pathList} {
        if {$batch > 25} {set batch 25}
 
        set ui_status_value [format \
-               "Including files ... %i/%i files (%.2f%%)" \
+               "$msg... %i/%i files (%.2f%%)" \
                $update_index_cp \
                $totalCnt \
                0.0]
@@ -1195,20 +1254,20 @@ proc update_index {pathList} {
                $pathList \
                $totalCnt \
                $batch \
+               $msg \
+               $after \
                ]
 }
 
-proc write_update_index {fd pathList totalCnt batch} {
+proc write_update_index {fd pathList totalCnt batch msg after} {
        global update_index_cp update_index_rsd ui_status_value
        global file_states ui_fname_value
 
        if {$update_index_cp >= $totalCnt} {
                close $fd
                unlock_index
-               set ui_status_value {Ready.}
-               if {$update_index_rsd} {
-                       reshow_diff
-               }
+               if {$update_index_rsd} reshow_diff
+               uplevel #0 $after
                return
        }
 
@@ -1218,14 +1277,20 @@ proc write_update_index {fd pathList totalCnt batch} {
                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*}
+               switch -glob -- [lindex $file_states($path) 0] {
                AD -
+               MD -
                _D {set new D*}
-               default {continue}
+
+               _M -
+               MM -
+               M_ {set new M*}
+
+               _O -
+               AM -
+               A_ {set new A*}
+
+               ?? {continue}
                }
 
                puts -nonewline $fd $path
@@ -1237,7 +1302,7 @@ proc write_update_index {fd pathList totalCnt batch} {
        }
 
        set ui_status_value [format \
-               "Including files ... %i/%i files (%.2f%%)" \
+               "$msg... %i/%i files (%.2f%%)" \
                $update_index_cp \
                $totalCnt \
                [expr {100.0 * $update_index_cp / $totalCnt}]]
@@ -1601,8 +1666,8 @@ proc console_read {w fd after} {
                while {$c < $n} {
                        set cr [string first "\r" $buf $c]
                        set lf [string first "\n" $buf $c]
-                       if {$cr < 0} {set cr [expr $n + 1]}
-                       if {$lf < 0} {set lf [expr $n + 1]}
+                       if {$cr < 0} {set cr [expr {$n + 1}]}
+                       if {$lf < 0} {set lf [expr {$n + 1}]}
 
                        if {$lf < $cr} {
                                $w.m.t insert end [string range $buf $c $lf]
@@ -1713,7 +1778,7 @@ proc do_quit {} {
 }
 
 proc do_rescan {} {
-       update_status
+       rescan {set ui_status_value {Ready.}}
 }
 
 proc do_include_all {} {
@@ -1735,7 +1800,10 @@ proc do_include_all {} {
        if {$pathList eq {}} {
                unlock_index
        } else {
-               update_index $pathList
+               update_index \
+                       "Including all modified files" \
+                       $pathList \
+                       {set ui_status_value {Ready to commit.}}
        }
 }
 
@@ -1935,34 +2003,88 @@ proc do_save_config {w} {
        destroy $w
 }
 
-proc file_left_click {w x y} {
-       global file_lists
+proc toggle_or_diff {w x y} {
+       global file_lists ui_index ui_other
+       global last_clicked selected_paths
 
        set pos [split [$w index @$x,$y] .]
        set lno [lindex $pos 0]
        set col [lindex $pos 1]
-       set path [lindex $file_lists($w) [expr $lno - 1]]
-       if {$path eq {}} return
+       set path [lindex $file_lists($w) [expr {$lno - 1}]]
+       if {$path eq {}} {
+               set last_clicked {}
+               return
+       }
+
+       set last_clicked [list $w $lno]
+       array unset selected_paths
+       $ui_index tag remove in_sel 0.0 end
+       $ui_other tag remove in_sel 0.0 end
 
-       if {$col > 0} {
+       if {$col == 0} {
+               update_index \
+                       "Including [short_path $path]" \
+                       [list $path] \
+                       {set ui_status_value {Ready.}}
+       } else {
                show_diff $path $w $lno
        }
 }
 
-proc file_left_unclick {w x y} {
+proc add_one_to_selection {w x y} {
        global file_lists
+       global last_clicked selected_paths
 
        set pos [split [$w index @$x,$y] .]
        set lno [lindex $pos 0]
        set col [lindex $pos 1]
-       set path [lindex $file_lists($w) [expr $lno - 1]]
-       if {$path eq {}} return
+       set path [lindex $file_lists($w) [expr {$lno - 1}]]
+       if {$path eq {}} {
+               set last_clicked {}
+               return
+       }
 
-       if {$col == 0} {
-               update_index [list $path]
+       set last_clicked [list $w $lno]
+       if {[catch {set in_sel $selected_paths($path)}]} {
+               set in_sel 0
+       }
+       if {$in_sel} {
+               unset selected_paths($path)
+               $w tag remove in_sel $lno.0 [expr {$lno + 1}].0
+       } else {
+               set selected_paths($path) 1
+               $w tag add in_sel $lno.0 [expr {$lno + 1}].0
        }
 }
 
+proc add_range_to_selection {w x y} {
+       global file_lists
+       global last_clicked selected_paths
+
+       if {[lindex $last_clicked 0] ne $w} {
+               toggle_or_diff $w $x $y
+               return
+       }
+
+       set pos [split [$w index @$x,$y] .]
+       set lno [lindex $pos 0]
+       set lc [lindex $last_clicked 1]
+       if {$lc < $lno} {
+               set begin $lc
+               set end $lno
+       } else {
+               set begin $lno
+               set end $lc
+       }
+
+       foreach path [lrange $file_lists($w) \
+               [expr {$begin - 1}] \
+               [expr {$end - 1}]] {
+               set selected_paths($path) 1
+       }
+       $w tag add in_sel $begin.0 [expr {$end + 1}].0
+}
+
 ######################################################################
 ##
 ## config defaults
@@ -2172,8 +2294,13 @@ pack .vpane.files.other.sb -side right -fill y
 pack $ui_other -side left -fill both -expand 1
 .vpane.files add .vpane.files.other -sticky nsew
 
-$ui_index tag conf in_diff -font font_uibold
-$ui_other tag conf in_diff -font font_uibold
+foreach i [list $ui_index $ui_other] {
+       $i tag conf in_diff -font font_uibold
+       $i tag conf in_sel \
+               -background [$i cget -foreground] \
+               -foreground [$i cget -background]
+}
+unset i
 
 # -- Diff and Commit Area
 frame .vpane.lower -height 300 -width 400
@@ -2455,8 +2582,9 @@ bind all <$M1B-Key-Q> do_quit
 bind all <$M1B-Key-w> {destroy [winfo toplevel %W]}
 bind all <$M1B-Key-W> {destroy [winfo toplevel %W]}
 foreach i [list $ui_index $ui_other] {
-       bind $i <Button-1>        {file_left_click %W %x %y; break}
-       bind $i <ButtonRelease-1> {file_left_unclick %W %x %y; break}
+       bind $i <Button-1>       "toggle_or_diff         $i %x %y; break"
+       bind $i <$M1B-Button-1>  "add_one_to_selection   $i %x %y; break"
+       bind $i <Shift-Button-1> "add_range_to_selection $i %x %y; break"
 }
 unset i
 
@@ -2471,4 +2599,4 @@ if {!$single_commit} {
        populate_remote_menu .mbar.push To push_to
        populate_pull_menu .mbar.pull
 }
-after 1 update_status
+after 1 do_rescan