| commit | 216ebe76f2d61b31fb8a46c2c1764b11ef82e579 | |
| author | Adam Roben <adam@roben.org> | |
| Tue, 5 Jun 2012 14:24:11 +0000 (5 10:24 -0400) | ||
| committer | Johannes Schindelin <johannes.schindelin@gmx.de> | |
| Thu, 10 Apr 2014 18:53:35 +0000 (10 13:53 -0500) | ||
| tree | eeb6eb9f38996fda15ffab97f2b09f12b81a54ae | treesnapshot (tar.gz zip) |
| parent | 37a22b4c02d520de7c769d0502973ec155d9f8f3 | 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 |