| commit | 44d03692587106452873a77f0d194babb8f2d84b | |
| author | Adam Roben <adam@roben.org> | |
| Tue, 5 Jun 2012 14:24:11 +0000 (5 10:24 -0400) | ||
| committer | Pat Thoyts <patthoyts@users.sourceforge.net> | |
| Mon, 22 Oct 2012 12:30:24 +0000 (22 13:30 +0100) | ||
| tree | 82a0ce46f3d8ac52d8dadf88b71c58cde815182b | treesnapshot (tar.gz zip) |
| parent | 65d4a2c537e932c5f7d7f7fc95bb3223828576bd | commitdiff |
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>
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 | diffblobblamehistory |