Re: [PATCH 2/3] Add -n/--no-prompt option to mergetool
- From
William Pursell <bill.pursell@gmail.com>
- Date
- Oct 24, 2008, 22:32 UTC
- Message-ID
- <49024CF1.5040406@gmail.com>
- In-Reply-To
- <20081023064455.GA10675@hashpling.org>
Charles Bailey wrote:
Show 9 quoted lines
> > After more thoughts, I'm somewhat in favour of dropping the short > switch altogether. As it just saves a single keypress per merge I > imagine that most mergetool users, once they discover this new > feature and decide that they want to use it, will prefer to use a user > config option to switch it on. The command line option then becomes > something that you would only need to use to override your normal > default and something on which to hang the option description in the > man page.
If the short option is dropped, the config option should probably associated with mergetool.<tool>.interactive rather than mergetool.interactive. (s/interactive/whatever)
-- William Pursell