Skip to content

Add windows-gnullvm hosts to the manifest #143031

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mati865
Copy link
Member

@mati865 mati865 commented Jun 25, 2025

I made a mistake testing #140772 only with rustup-toolchain-install-master which doesn't care about the manifests.

This means windows-gnullvm self-hosting will have to wait one more release, unless this change is backported to beta and a new beta release is made, which doesn't seem worth the trouble.

@rustbot
Copy link
Collaborator

rustbot commented Jun 25, 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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 25, 2025
@rustbot

This comment was marked as resolved.

@mati865 mati865 force-pushed the push-mutywntmvomx branch from e7a5863 to 508021a Compare June 25, 2025 23:42
@workingjubilee
Copy link
Member

Well. I am going to nominate this for beta and let people decide it's not worth the trouble.

@workingjubilee workingjubilee added beta-nominated Nominated for backporting to the compiler in the beta channel. T-release Relevant to the release subteam, which will review and decide on the PR/issue. labels Jun 25, 2025
@Mark-Simulacrum Mark-Simulacrum added beta-accepted Accepted for backporting to the compiler in the beta channel. and removed beta-nominated Nominated for backporting to the compiler in the beta channel. beta-accepted Accepted for backporting to the compiler in the beta channel. labels Jun 28, 2025
@Mark-Simulacrum
Copy link
Member

@bors r+

I'm going to go ahead and reject the backport since we don't normally backport features (and this is one). We should really figure out a way to make it much harder to miss updating this file though...

@bors
Copy link
Collaborator

bors commented Jun 28, 2025

📌 Commit 508021a has been approved by Mark-Simulacrum

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 Jun 28, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jun 28, 2025
…Simulacrum

Add windows-gnullvm hosts to the manifest

I made a mistake testing rust-lang#140772 only with `rustup-toolchain-install-master` which doesn't care about the manifests.

This means windows-gnullvm self-hosting will have to wait one more release, unless this change is backported to beta and a new beta release is made, which doesn't seem worth the trouble.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-release Relevant to the release subteam, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants