On 3/18/07, Junio C Hamano <junkio@xxxxxxx> wrote:
The problem description looks correct, but I think the original
meant to reject configuration value for merge_tool that is not
supported with the version of the script (and screwed up).
There's a bit later on in mergetool that errors out if you have
provided an unknown merge program (either via the command line or
through your config). The command line and the config ways should
probably behave the same, eh? If so, the case block should be brought
up one level like so:
IOW, wouldn't this be a better way?
if test -z "$merge_tool"
then
merge_tool=`git-config merge.tool`
fi
case "$merge_tool" in
kdiff3 | tkdiff | xxdiff | meld | emerge)
;; # happy
*)
echo >&2 "We do not know how to drive $merge_tool"
echo >&2 "Resetting to default..."
unset merge_tool
;;
esac
And then remove the 'Unknown mergetool' bit.
I think either way is fine since they both let you know that you've
entered gobbledeegook or forgot to install something, so I'll defer to
you all for the choice on which way to go.
-James
-
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