Skip to content

Switch over to ucrt64 - #741

Merged
dscho merged 3 commits into
mainfrom
switch-over-to-ucrt64
Sep 10, 2026
Merged

dscho merged 3 commits into
mainfrom
switch-over-to-ucrt64

Conversation

@dscho

@dscho dscho commented Sep 10, 2026

Copy link
Copy Markdown
Member

This is a follow-up to git-for-windows/git-sdk-64#117.

@dscho dscho self-assigned this Sep 10, 2026
@dscho
dscho force-pushed the switch-over-to-ucrt64 branch 4 times, most recently from d6f7fad to 8670707 Compare September 10, 2026 21:58
When we started migrating Git for Windows from MINGW64 to UCRT64 in
git-for-windows/git-sdk-64#117, we eventually
also started to build the UCRT64 installers and published them.

Now that this PR is merged, the order is reversed: the MINGW64 one is
now on its way out and will only be built for a short while, with the
official version being UCRT64.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Now that we've switched to UCRT64, it is _supposed_ to be missing.
Apparently `dash.exe` bails when this happens, even if the test is in
the conditional of the `if` construct.

The `build-installers` arm already handled this gracefully, but the
`makepkg-git` arm needs to handle this, too.

While at it, log more verbosely what is happening in the invocation of
the `please.sh` script.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
@dscho
dscho force-pushed the switch-over-to-ucrt64 branch from 1b88919 to 32c75da Compare September 10, 2026 22:16
@dscho
dscho merged commit 9917a4d into main Sep 10, 2026
26 checks passed
@dscho
dscho deleted the switch-over-to-ucrt64 branch September 10, 2026 22:21
dscho added a commit that referenced this pull request Sep 10, 2026
I meant to include this in
#741...

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant