Fix launching of externals from Unicode paths
commitacd4a9baff5b02e6efceddc25ef68b054829a856
authorAdam Roben <adam@roben.org>
Tue, 5 Jun 2012 14:24:11 +0000 (5 10:24 -0400)
committerKarsten Blees <blees@dcon.de>
Sat, 16 Aug 2014 01:02:36 +0000 (16 03:02 +0200)
treed73d4ce3f96af4ee024bf43c0b558d0939070923
parent19583437bf73c07308c6c84f7dbb63abe3b4cd27
Fix launching of externals from Unicode paths

If Git were installed in a path containing non-ASCII characters,
commands such as git-am and git-submodule, which are implemented as
externals, would fail to launch with the following error:

> fatal: 'am' appears to be a git command, but we were not
> able to execute it. Maybe git-am is broken?

This was due to lookup_prog not being Unicode-aware. It was somehow
missed in 2ee5a1a14ad17ff35f0ad52390a27fbbc41258f3.

Note that the only problem in this function was calling
GetFileAttributes instead of GetFileAttributesW. The calls to access()
were fine because access() is a macro which resolves to mingw_access,
which already handles Unicode correctly. But I changed lookup_prog to
use _waccess directly so that we only convert the path to UTF-16 once.

Signed-off-by: Adam Roben <adam@roben.org>
compat/mingw.c