Skip to content

Remove old MinGW workaround#158021

Draft
mati865 wants to merge 2 commits into
rust-lang:mainfrom
mati865:mingw-remove-workaround
Draft

Remove old MinGW workaround#158021
mati865 wants to merge 2 commits into
rust-lang:mainfrom
mati865:mingw-remove-workaround

Conversation

@mati865

@mati865 mati865 commented Jun 17, 2026

Copy link
Copy Markdown
Member

Before https://github.com/mingw-w64/mingw-w64/commit/ mkstemp used by Binutils was not sufficiently collision resilient and that led to CI errors for us.

Our new baseline mingw-w64 toolchain's Binutils are built using mingw-w64 with the fix: niXman/mingw-builds#682
So, we should be able to get rid of it.

mati865 added 2 commits June 17, 2026 13:07
Before mingw-w64/mingw-w64@dbfdf80
mkstemp used by Binutils was not sufficiently
collision resilient and that led to CI errors for
us.

Our new baseline mingw-w64 toolchain's Binutils
are built using mingw-w64 with the fix:
niXman/mingw-builds#682
So, we should be able to get rid of it.
@rustbot rustbot added A-CI Area: Our Github Actions CI A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Jun 17, 2026
@mati865

This comment was marked as resolved.

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 17, 2026
Remove old MinGW workaround


try-job: *i686-mingw
try-job: x86_64-mingw
@rust-log-analyzer

This comment was marked as resolved.

@rust-bors

This comment was marked as resolved.

@mati865

This comment was marked as resolved.

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 17, 2026
Remove old MinGW workaround


try-job: *i686-mingw
try-job: *x86_64-mingw*
@rust-bors

This comment was marked as resolved.

@mati865

This comment was marked as resolved.

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 17, 2026
Remove old MinGW workaround


try-job: *i686-mingw
try-job: x86_64-mingw
@rust-bors

This comment was marked as resolved.

@rust-log-analyzer

This comment was marked as resolved.

@mati865

This comment was marked as resolved.

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 17, 2026
Remove old MinGW workaround


try-job: *i686-mingw
try-job: *x86_64-mingw*
@rust-bors

This comment was marked as resolved.

@mati865

mati865 commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

@bors try jobs=*i686-mingw,x86_64-mingw

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Jun 17, 2026
Remove old MinGW workaround


try-job: *i686-mingw
try-job: *x86_64-mingw*
@rust-bors

rust-bors Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: b4890a5 (b4890a588e40d6e21aae5da529ff6ea0b38b9083)
Base parent: 693b3e4 (693b3e4c6e4e686cb9878c1722ad26858b5f1d2a)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CI Area: Our Github Actions CI A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants