From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Tue, 12 May 2015 15:03:51 GMT
Subject: Re: [PATCH] mergetools: add winmerge as a builtin tool
Message-ID: <f44bed1d712b4be11ee143cc33ef0bb1@www.dscho.org>
In-Reply-To: <1431422977-61243-1-git-send-email-davvid@gmail.com>

Hi David,

On 2015-05-12 11:29, David Aguilar wrote:

> +	# Look for WinMergeU.exe in the typical locations
> +	winmerge_exe="WinMerge/WinMergeU.exe"
> +	if test -n "$PROGRAMFILES" && test -x "$PROGRAMFILES/$winmerge_exe"
> +	then
> +		printf '%s' "$PROGRAMFILES/$winmerge_exe"
> +	elif test -x "/c/Program Files (x86)/$winmerge_exe"

At this stage you probably want to look for "$ProgramW6432" first.

> +	then
> +		printf '%s' "/c/Program Files (x86)/$winmerge_exe"
> +	elif test -x "/c/Program Files/$winmerge_exe"
> +	then
> +		printf '%s' "/c/Program Files/$winmerge_exe"
> +	else
> +		echo WinMergeU.exe
> +	fi
> +}

Ciao,
Johannes