On Sat, Oct 22, 2011 at 21:39, Bert Wesarg <bert.wesarg@xxxxxxxxxxxxxx> wrote: > Signed-off-by: Bert Wesarg <bert.wesarg@xxxxxxxxxxxxxx> > --- > lib/commit.tcl | 2 +- > lib/option.tcl | 1 + > 2 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/lib/commit.tcl b/lib/commit.tcl > index 372bed9..e27e148 100644 > --- a/lib/commit.tcl > +++ b/lib/commit.tcl > @@ -263,7 +263,7 @@ proc commit_commitmsg {curHEAD msg_p} { > global is_detached repo_config > global pch_error > > - if {$is_detached && $repo_config(gui.warndetachedcommit)} { > + if {$is_detached && [is_config_true gui.warndetachedcommit]} { > set msg [mc "You are about to commit on a detached head.\ > This is a potentially dangerous thing to do because if you switch\ > to another branch you will loose your changes and it can be difficult\ > diff --git a/lib/option.tcl b/lib/option.tcl > index 719103a..f7f866b 100644 > --- a/lib/option.tcl > +++ b/lib/option.tcl > @@ -156,6 +156,7 @@ proc do_options {} { > {i-0..99 gui.commitmsgwidth {mc "Commit Message Text Width"}} > {t gui.newbranchtemplate {mc "New Branch Name Template"}} > {c gui.encoding {mc "Default File Contents Encoding"}} > + {b gui.warndetachedcommit {mc "Warn before commiting to a detached head"}} > {s gui.stageuntracked {mc "Staging of untracked files"} {list "yes" "no" "ask"}} > } { > set type [lindex $option 0] Pat, if you're interessted in this patch, please fix the typo in the second hunk, mentioning 'commiting'. Also shouldn't this variable be called gui.warndetachedhead? Thanks. Bert > -- > 1.7.7.908.g383b5 > > -- 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