Stream: t-compiler/wg-prioritization/alerts

Topic: #81051 Compilation of some crates is broken starting with n…


triagebot (Jan 17 2021 at 15:21, on Zulip):

@WG-prioritization/alerts issue #81051 has been requested for prioritization.

Procedure

Camelid (Jan 17 2021 at 19:15, on Zulip):

Apparently Windows 7 has been EOL for a year, so this is probably P-low if anything.

rylev (Jan 18 2021 at 10:31, on Zulip):

Yes though for the std lib Windows 7 is still supported by the Rust project. Dropping support for XP was only recently accepted Windows 7 is still supported. I don't think there's a solid policy on what supporting a target means. Do we support building on Windows 7? Only cross compiling to it?

rylev (Jan 18 2021 at 10:32, on Zulip):

This seems like something that should be clarified at some point

Camelid (Jan 18 2021 at 19:35, on Zulip):

Not only is Windows 7 supported, apparently it's Tier 1.

apiraino (Jan 20 2021 at 13:29, on Zulip):

For now i686-pc-windows-gnu and x86_64-pc-windows-msvc are tier 1 and the issue at hand is from a user that cannot compile on both, so we should uphold our promise of Tier 1 support.

If Win7 should be demoted because EOL'ed, it's fine but there's should be an agreement. Sending different messages to users is confusing.

apiraino (Jan 20 2021 at 13:30, on Zulip):

All this said a permission denied error doesnt sound like a Rust compiler issue :-) so for the moment I'd say P-low until we understand more

apiraino (Jan 20 2021 at 13:36, on Zulip):

by the way, I wonder if those disk volumes (F: and G:) that reporters are compiling onto are local or remote ... or even USB stuff

apiraino (Jan 21 2021 at 13:04, on Zulip):

self-note: leaving this unprioritized, waiting for more info to clarify

apiraino (Jan 27 2021 at 10:45, on Zulip):

removing the I-prioritize label and adding that we are waiting for a feedback form issue reporter

triagebot (Jan 27 2021 at 10:45, on Zulip):

Issue #81051's prioritization request has been removed.

triagebot (Feb 07 2021 at 09:10, on Zulip):

@WG-prioritization/alerts issue #81051 has been requested for prioritization.

Procedure

Léo Lanteri Thauvin (Feb 07 2021 at 09:19, on Zulip):

This makes rustc effectively unusable on windows 7

Sounds kind of bad :P

Léo Lanteri Thauvin (Feb 07 2021 at 09:22, on Zulip):

Pinged the LLVM group

apiraino (Feb 10 2021 at 12:30, on Zulip):

well done @Léo Lanteri Thauvin . This issue now seems clearer thanks to the debugging done. I'd assign a P-high

apiraino (Feb 10 2021 at 12:32, on Zulip):

hopefully someone in the team can comment on how easily this can be reproducible, as this seems affecting Win7 and the CI fails to catch it

apiraino (Feb 10 2021 at 12:34, on Zulip):

I'd also like a more general question about Win7 support to be also touched (like also Camelid commented)

apiraino (Feb 10 2021 at 12:37, on Zulip):

which to be clear could be also only tangent to this issue, though I don't understand the implication (why only on Win7 x64 and that LLVM commit?)

triagebot (Feb 10 2021 at 12:40, on Zulip):

Issue #81051's prioritization request has been removed.

apiraino (Feb 10 2021 at 18:40, on Zulip):

@WG-prioritization I wonder if this issue should be a regression-from-stable-to-nightly. Since it's a) unassigned so I'm afraid it would pass unnoticed and b) going back to a previous nightly fixes the problem

Léo Lanteri Thauvin (Feb 10 2021 at 18:57, on Zulip):

Which version is nightly-2021-01-12?

Léo Lanteri Thauvin (Feb 10 2021 at 18:57, on Zulip):

(checking...)

Léo Lanteri Thauvin (Feb 10 2021 at 19:00, on Zulip):

Right, it's 1.51.0, so nightly soon-to-be-beta

Léo Lanteri Thauvin (Feb 10 2021 at 19:01, on Zulip):

(Applied the label)

apiraino (Feb 10 2021 at 19:02, on Zulip):

ah thanks! how did you figure that out?

Léo Lanteri Thauvin (Feb 10 2021 at 19:54, on Zulip):

I just lazily executed rustup toolchain install nightly-2021-01-12:

nightly-2021-01-12-x86_64-apple-darwin installed - rustc 1.51.0-nightly (c5eae5629 2021-01-11)
Léo Lanteri Thauvin (Feb 10 2021 at 19:54, on Zulip):

Though there must be a better way :D

Léo Lanteri Thauvin (Feb 10 2021 at 19:56, on Zulip):

Just looking at the date might be enough

Hameer Abbasi (Feb 11 2021 at 06:09, on Zulip):

I agree that if Windows 7 is a tier 1 platform; until that’s changed, we should treat it as so, and therefore treat this as a regression.

Though are we the ones who Tag regressions?

apiraino (Feb 11 2021 at 09:19, on Zulip):

I think we do (when we have enough info to assess that, such as in this case)

Last update: Apr 16 2021 at 23:45UTC