Skip to content

[Backport release-24.11] windows.mingw_w64_headers: fix building against msvcrt#365167

Merged
misuzu merged 1 commit intorelease-24.11from
backport-365060-to-release-24.11
Dec 14, 2024
Merged

[Backport release-24.11] windows.mingw_w64_headers: fix building against msvcrt#365167
misuzu merged 1 commit intorelease-24.11from
backport-365060-to-release-24.11

Conversation

@nixpkgs-ci
Copy link
Contributor

@nixpkgs-ci nixpkgs-ci bot commented Dec 14, 2024

Bot-based backport to release-24.11, triggered by a label in #365060.

  • Before merging, ensure that this backport is acceptable for the release.
    • Even as a non-commiter, if you find that it is not acceptable, leave a comment.

Previously we relied on the default CRT selected by upstream. This
broke builds against MSVCRT starting with 12.0, as the default was
changed to UCRT

We can instead manually configure the CRT based on our `stdenv` to
ensure there is no discrepancy between the CRT we want to use and that
being targeted by the build

(cherry picked from commit 6f73c03)
@github-actions github-actions bot added 6.topic: windows Running, or buiding, packages on Windows 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux. labels Dec 14, 2024
@misuzu misuzu merged commit a81555f into release-24.11 Dec 14, 2024
@misuzu misuzu deleted the backport-365060-to-release-24.11 branch December 14, 2024 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: windows Running, or buiding, packages on Windows 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 11-100 This PR causes between 11 and 100 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants