From: Sylvain Rabot Date: Sat, 05 Jun 2010 02:31:51 GMT Subject: [PATCH] git-mergetool--lib.sh: fix mergetool..* configurations ignored for known tools Message-ID: <1275705112-8088-1-git-send-email-sylvain@abstraction.fr> Hi, Here a patch I made after Junio's remarks in this thread : http://thread.gmane.org/gmane.comp.version-control.git/148267 I think git-mergetool--lib.sh needs some refactoring. I'm no sh expert, far from it, but I know a bit about scripting (I have been developing in PHP for more than 6 years now :P) and I had quite some difficulties to understand the behavior of some functions. For example, get_merge_tool_path returns the name of the tool if no mergetool..path have been set, which, from my point of view, makes no sense. I know there is maybe no time or no need to rewrite something which works rather well but the +5 hours I spent to write this poor patch make me wonder if I am overrating myself. So if someone could just agree with me that would be a huge step to help me to regain some self respect :) Cheers.