2009/10/30 Markus Heidelberg <markus.heidelberg@xxxxxx>: > Jay Soffian, 30.10.2009: >> On Thu, Oct 29, 2009 at 9:02 PM, Markus Heidelberg >> <markus.heidelberg@xxxxxx> wrote: >> > He didn't mean p4merge on other platforms, but other merge tools on Mac >> > OS X. What about all the other merge tools already in mergetool--lib? >> > Should they get special handling, too? >> >> If someone wants to scratch that itch, then yes. The default diff tool >> for OS X has its helper already in /usr/bin (opendiff). p4merge is >> arguably a better merge tool, and it installs as an app bundle in >> /Applications. I'm not sure about the other diff tools, I haven't >> looked. >> >> > And for Windows we could add C:\Program Files\MergeToolX\tool.exe for >> > every merge tool. >> >> If it makes those tools easier to use with git, and if someone on >> Windows wants to scratch that itch, then yes, we should. > > Another possible problem: the user can change the installation > destination on Windows. What's the behaviour of Mac OS here? Is the > instalation path fixed or changeable? For Windows, the program should have an InstallDir or similar registry value in a fixed place in the registry to point to where it is installed (something like HKLM/Software/[Vendor]/[Application]/[Version]). As for Linux, there is no guarantee that things like p4merge are in the path either. It could be placed under /opt/perforce or /home/perforce. What would be sensible (for all platforms) is: 1/ if [difftool|mergetool].toolname.path is set, use that (is this documented?) 2/ try looking for the tool in the system path 3/ try some intelligent guessing 4/ if none of these work, print out an error message -- ideally, this should mention the configuration option in (1) (3) is what is being discussed. It is good that it will work without any user configuration (especially for standard tools installed in standard places), but isn't really a big problem as long as the user is prompted to configure the tool path. Also, I'm not sure how this will work with multiple versions of the tools installed (e.g. vim/gvim and p4merge). - Reece -- 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