[win][CI] Convert paths to Windows format before adding to PATH #140754
Conversation
|
r? @marcoieni rustbot has assigned @marcoieni. Use |
rustbot
added
A-testsuite
| @@ -42,5 +42,5 @@ if isWindows && isKnownToBeMingwBuild; then | |||
|
|
|||
| curl -o mingw.7z "${MIRRORS_BASE}/${mingw_archive}" | |||
| 7z x -y mingw.7z > /dev/null | |||
| ciCommandAddPath "$(pwd)/${mingw_dir}/bin" | |||
| ciCommandAddPath "$(cygpath -m "$(pwd)/${mingw_dir}/bin")" | |||
There was a problem hiding this comment.
this change is useful for non-mingw builds, but this line runs only in mingw builds (see isKnownToBeMingwBuild in the if).
Is this still useful in your opinion? 🤔
There was a problem hiding this comment.
Yes, it is still useful.
Even though this is a MinGW build, we are not running the MinGW versions of all of the tools. For example, we are still using a native Windows Rust toolchain, and it is using native Windows paths (for example: https://github.com/rust-lang-ci/rust/actions/runs/14918934614/job/41910551227#step:27:214).
|
the rest of the pr looks good to me. Let's just run a bors try to check that the CI keeps working 👍 |
Sure: but I don't have permissions to run try... |
While trying to get the
aarch64-msvcbuild working correctly (#140136), I noticed that thePATHobserved during the Rust steps of the build on Windows builds had some entries still in MinGW format, which means that Windows would not be able to use them correctly:From https://github.com/rust-lang/rust/actions/runs/14606167901/job/40975555680#step:28:192 note the path for
ninjaandsccache.The fix for this is to use
cygpathto convert from the Cygwin format back to native Windows format before adding to thePATH.