Skip to content

Conversation

@mati865
Copy link
Member

@mati865 mati865 commented Dec 1, 2025

Rust no longer requires MSYS2 provided tools like make.

Rust no longer requires MSYS2 tools like make.
@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 Dec 1, 2025
@mati865
Copy link
Member Author

mati865 commented Dec 1, 2025

@bors try jobs=dist-i686-mingw,dist-x86_64-llvm-mingw,dist-x86_64-mingw,x86_64-mingw-1,x86_64-mingw-2

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request Dec 1, 2025
Stop adding MSYS2 to PATH

try-job: dist-i686-mingw
try-job: dist-x86_64-llvm-mingw
try-job: dist-x86_64-mingw
try-job: x86_64-mingw-1
try-job: x86_64-mingw-2
@rust-bors
Copy link

rust-bors bot commented Dec 1, 2025

☀️ Try build successful (CI)
Build commit: 985933c (985933c5806c18741cc07431a6c7eb14bbf67677, parent: 2fb805367dd3012ce5c50865d03c86e70bf10f0f)

@mati865 mati865 marked this pull request as ready for review December 1, 2025 21:02
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 1, 2025
@rustbot rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Dec 1, 2025
@rustbot
Copy link
Collaborator

rustbot commented Dec 1, 2025

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jieyouxu
Copy link
Member

jieyouxu commented Dec 2, 2025

r? me @bors r+

@bors
Copy link
Collaborator

bors commented Dec 2, 2025

📌 Commit 662192a has been approved by jieyouxu

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Dec 2, 2025
bors added a commit that referenced this pull request Dec 2, 2025
Rollup of 8 pull requests

Successful merges:

 - #145628 ([std][BTree] Fix behavior of `::append` to match documentation, `::insert`, and `::extend`)
 - #149241 (Fix armv4t- and armv5te- bare metal targets)
 - #149470 (compiletest: Prepare ignore/only conditions once in advance, without a macro)
 - #149507 (Mark windows-gnu* as lacking build with assertions)
 - #149508 (Prefer helper functions to identify MinGW targets)
 - #149516 (Stop adding MSYS2 to PATH)
 - #149525 (debuginfo/macro-stepping test: extend comments)
 - #149526 (Add myself (mati865) to the review rotation)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 5afd8ff into rust-lang:main Dec 2, 2025
12 checks passed
@rustbot rustbot added this to the 1.93.0 milestone Dec 2, 2025
rust-timer added a commit that referenced this pull request Dec 2, 2025
Rollup merge of #149516 - mati865:no-msys2, r=jieyouxu

Stop adding MSYS2 to PATH

Rust no longer requires MSYS2 provided tools like make.
@mati865 mati865 deleted the no-msys2 branch December 2, 2025 08:30
mati865 added a commit to mati865/rust that referenced this pull request Dec 3, 2025
Revert rust-lang#149516 because it caused dist to pick up x86_64
`libwinpthreads.dll` dependency, resulting in broken toolchain.
mati865 added a commit to mati865/rust that referenced this pull request Dec 3, 2025
Revert rust-lang#149516 because it caused dist to pick up x86_64
`libwinpthreads.dll` dependency, resulting in broken toolchain.

This reverts commit 5afd8ff, reversing
changes made to 7cb02f9.
bors added a commit that referenced this pull request Dec 4, 2025
Revert #149516 to fix i686-pc-windows-gnu host

Revert #149516 because it caused dist to pick up x86_64 `libwinpthreads.dll` dependency, resulting in broken toolchain.

Note, I didn't use the procedure explained at: https://forge.rust-lang.org/compiler/reviews.html#reverts because the revert button isn't there.

This reverts commit 5afd8ff, reversing
changes made to 7cb02f9.
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-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

5 participants