On 24.07.2012 00:41, Junio C Hamano wrote:
> + if test -f $(dirname "$(type --path compare)")/AraxisMerge
We would need additional quotes around the whole path here as the Windows installation path is usually something like "C:\Program Files\Araxis\Araxis Merge" and contains spaces.
Moreover, "test -f" requires the ".exe" extension to be explicitly present for the file to test. But I'd rather not do that because the test would be specific to Windows then and e.g. not work on Mac OS X. That's why I'd still like to use ls like in my first patch:
mergetools/araxis | 15 ++++++++++++++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/mergetools/araxis b/mergetools/araxis
index 64f97c5..c406ead 100644
--- a/mergetools/araxis
+++ b/mergetools/araxis
@@ -16,5 +16,18 @@ merge_cmd () {
}
translate_merge_tool_path() {
- echo compare
+ case "$BASH_VERSION" in
+ ??*)
+ # we can safely use "type --path"
+ if ls "$(dirname "$(type --path compare)")"/Araxis* >/dev/null 2>&1
+ then
+ echo compare
+ else
+ echo "$1"
+ fi
+ ;;
+ *)
+ echo compare
+ ;;
+ esac
}
--
Sebastian Schuberth
--
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