msys2_path_conv: pass PC_NOFULL to path_conv#117
Conversation
In theory this doesn't make a difference because posix_to_win32_path() is only called with rooted/absolute paths, but as pointed out in msys2#103 PC_NOFULL will preserve the trailing slash of unix paths (for some reason). See "cygpath -m /bin/" (preserved) vs "cygpath -am /bin/" (dropped) One use case where we need to trailing slashes to be preserved is the GCC build system: https://github.com/gcc-mirror/gcc/blob/6d82e0fea5f988e829912a/gcc/Makefile.in#L2314 The Makefile appends a slash to the prefixes and the C code doing relocation will treat the path as a directory if there is a trailing slash. See msys2/MINGW-packages#14173 for details. With this change all our MSYS2 path_conv tests pass again.
|
For completeness sake I've also checked that git-for-windows/git#1497 (comment) still works with this. |
|
With msys2/MSYS2-packages#3306 merged, we should rebase and merge this here PR, too, right @lazka?
I guess it would be a good idea to merge the test suite into this here repository, and then make it part of the CI. What do you think? |
Please ignore the fact that it's merged, I just wanted to unbreak the gcc build and do some testing with staging.
Yes, I happened to create an issue related to this just now: msys2/msys2-tests#24 |
|
There hasn't been any more fallout with this backported, so I think it is good to merge. |
The long-delayed upgrade of Git for Windows' fork of the MSYS2 runtime to v3.4.* is finally here. And it comes with quite a few changes. The following two patches made it into the MSYS2 runtime proper: - 589fac6 (Fix incorrect path conversion (trailing slash), 2018-03-14) as msys2/msys2-runtime#103 and msys2/msys2-runtime#117 - 3e9c003 (Pass environment variables with empty values, 2015-02-18) as msys2/msys2-runtime#101 The following two patches made it even further upstream, into Cygwin proper: - fa95548 (Implicitly support the /dev/fd symlink and friends, 2022-02-20), in Cygwin as 4ec0889 (Cygwin: Implicitly support the /dev/fd symlink and friends, 2022-02-21) - 4704d29 (dumper: avoid linker problem when `libbfd` depends on `libsframe`, 2023-01-31) was upstreamed as 9acb704afe (dumper: avoid linker problem when `libbfd` depends on `libsframe`, 2023-01-31) Finally, we backported the commit 4d62197 (Cygwin: console: Fix hangup of less on quit after the window is resized., 2022-12-22), so naturally it is no longer necessary to carry it in Git for Windows' fork. This commit starts the rebase of 353f6b9 to 8dd31e7d1a Signed-off-by: Johannes Schindelin <[email protected]>
2 participants
In theory this doesn't make a difference because posix_to_win32_path() is only called with rooted/absolute paths, but as pointed out in #103 PC_NOFULL will preserve the trailing slash of unix paths (for some reason).
See "cygpath -m /bin/" (preserved) vs "cygpath -am /bin/" (dropped)
One use case where we need the trailing slashes to be preserved is the GCC build system:
https://github.com/gcc-mirror/gcc/blob/6d82e0fea5f988e829912a/gcc/Makefile.in#L2314
The Makefile appends a slash to the prefixes and the C code doing relocation will treat the path as a directory if there is a trailing slash. See msys2/MINGW-packages#14173 for details.
With this change all our MSYS2 path_conv tests pass again.