Hi, what the series tries to achieve looks good to me. Just one comment. On Fri, Oct 14, 2011 at 09:25:21PM +0200, Bert Wesarg wrote: [...] > diff --git a/lib/index.tcl b/lib/index.tcl > index 014acf9..45094c2 100644 > --- a/lib/index.tcl > +++ b/lib/index.tcl > @@ -367,7 +367,19 @@ proc do_add_all {} { > } > } > if {[llength $untracked_paths]} { > - set reply [ask_popup [mc "Stage also untracked files?"]] > + set reply 0 > + switch -- [get_config gui.stageuntracked] { > + no { > + set reply 0 > + } [...] Here I am wondering whether we have a similar mechanism in git gui like in core git that makes yes,true,1 equivalents (and similar with other values) ? If we don't I think the series is fine as it is otherwise it probably makes sense to use that mechanism. Cheers Heiko -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html