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
Update Windows platform support #115141
Update Windows platform support #115141
Conversation
r? @ehuss (rustbot has picked a reviewer for you, use r? to override) |
Similarly to #104385 (comment), should this be a T-libs + T-compiler FCP, rather than T-compiler + T-release? I don't actually think we have a policy around bumping minimum support. |
I have no idea. I'd really love to have some clarity on the procedure. |
The current target policy says:
(I'm not opposed to including T-libs, just quoting what we have.) |
I'm not sure who to assign. |
☔ The latest upstream changes (presumably #104385) made this pull request unmergeable. Please resolve the merge conflicts. |
Sure let's do an FCP. See MCP for details. @rfcbot fcp merge |
Team member @compiler-errors has proposed to merge this. The next step is review by the rest of the tagged team members:
Concerns:
Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up! See this document for info about what commands tagged team members can give me. |
@rfcbot concern needs-blog-post |
is this in part a deprecation of Windows 7 official support? |
Just to note, the intent to update Windows compatibility was announced in the Rust 1.72.0 blog post, which was a light release so there was some discussion. |
I think this PR should be merged with (and backport to) the 1.76 release this week to avoid confusion like the MIPS demotion. |
A blog post is being prepared if anyone has any final comments. I don't know if anyone has any thoughts on post date for the blog. I'm thinking soon. |
e4b75da
to
b711bb2
Compare
As noted by @dpaoliello, we don't officially support XP even at tier 3 so I've also removed that. Of course |
The final comment period, with a disposition to merge, as per the review above, is now complete. As the automated representative of the governance process, I would like to thank the author for their work and everyone else who contributed. This will be merged soon. |
b711bb2
to
d7045ea
Compare
These commits modify compiler targets. |
Rebased. |
☔ The latest upstream changes (presumably #122305) made this pull request unmergeable. Please resolve the merge conflicts. |
d7045ea
to
779ac69
Compare
FYI, I used the Windows targets as examples when filling out the |
I think I sneaked that in before your comment, no? |
Ah, you did, my bad for not checking the files... |
I probably should have made a new commit. But I am a fan of having a clean commit history 😁 |
The FCP has concluded, therefore: @bors r+ rollup |
…leywiser Update Windows platform support This should not be merged until Rust 1.76 but I'm told this may need an fcp in addition to [MCP 651](rust-lang/compiler-team#651). cc `@rust-lang/compiler` `@rust-lang/release`
…leywiser Update Windows platform support This should not be merged until Rust 1.76 but I'm told this may need an fcp in addition to [MCP 651](rust-lang/compiler-team#651). cc ``@rust-lang/compiler`` ``@rust-lang/release``
…iaskrgr Rollup of 8 pull requests Successful merges: - rust-lang#115141 (Update Windows platform support) - rust-lang#121865 (Add FileCheck annotations to MIR-opt unnamed-fields tests) - rust-lang#122000 (Fix 32-bit overflows in LLVM composite constants) - rust-lang#122194 (Enable creating backtraces via -Ztreat-err-as-bug when stashing errors) - rust-lang#122319 (Don't ICE when non-self part of trait goal is constrained in new solver) - rust-lang#122339 (Update books) - rust-lang#122342 (Update /NODEFAUTLIB comment for msvc) - rust-lang#122343 (Remove some unnecessary `allow(incomplete_features)` in the test suite) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#115141 - ChrisDenton:windows-support, r=wesleywiser Update Windows platform support This should not be merged until Rust 1.76 but I'm told this may need an fcp in addition to [MCP 651](rust-lang/compiler-team#651). cc ```@rust-lang/compiler``` ```@rust-lang/release```
This should not be merged until Rust 1.76 but I'm told this may need an fcp in addition to MCP 651.
cc @rust-lang/compiler @rust-lang/release