Skip to content
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

regression: std no longer in toolchain dir #132202

Closed
Mark-Simulacrum opened this issue Oct 26, 2024 · 6 comments
Closed

regression: std no longer in toolchain dir #132202

Mark-Simulacrum opened this issue Oct 26, 2024 · 6 comments
Labels
regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Milestone

Comments

@Mark-Simulacrum
Copy link
Member

https://crater-reports.s3.amazonaws.com/beta-1.83-3/beta-2024-10-19/reg/bevy_dexterous_developer_library-0.4.0-alpha.3/log.txt

root is in prefer-dynamic crate, probably just needs a compat note

@Mark-Simulacrum Mark-Simulacrum added regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Oct 26, 2024
@Mark-Simulacrum Mark-Simulacrum added this to the 1.83.0 milestone Oct 26, 2024
@rustbot rustbot added I-prioritize Issue: Indicates that prioritization has been requested for this issue. needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Oct 26, 2024
@saethlin saethlin removed the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Oct 27, 2024
@apiraino
Copy link
Contributor

I quickly skimmed at the most recent merges with relnotes and didn't find anything from {t-libs,-api} about that. Very likely have missed that, just wanted to check if it's tagged for the changelog

@apiraino apiraino removed the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Oct 30, 2024
@jieyouxu
Copy link
Member

jieyouxu commented Oct 30, 2024

I quickly skimmed at the most recent merges with relnotes and didn't find anything from {t-libs,-api} about that. Very likely have missed that, just wanted to check if it's tagged for the changelog

As discussed in https://rust-lang.zulipchat.com/#narrow/channel/245100-t-compiler.2Fwg-prioritization.2Falerts/topic/.23132202.20regression.3A.20std.20no.20longer.20in.20toolchain.20dir, this is probably T-bootstrap (and maybe weakly T-compiler) change from #131188. But yes, we should probably add a compat note.

cc @Kobzol as you may know more about this, but I suspect the prefer-dynamic crate was relying on undocumented unguaranteed behavior and that this isn't considered a "breakage" in the strict sense.

EDIT: nevermind @bjorn3 already commented on the prefer-dynamic crate WilliamVenner/prefer-dynamic#2, oops.

@jieyouxu jieyouxu added the T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) label Oct 30, 2024
@Kobzol
Copy link
Contributor

Kobzol commented Oct 30, 2024

Yeah I think that if anyone depended on libstd.so within the lib directory, it was an instance of Hyrum's Law.

@cuviper
Copy link
Member

cuviper commented Nov 22, 2024

This will be documented as a compat note via #133352.

@jieyouxu
Copy link
Member

I think this can be closed since 1.83.0 is already released containing said compat notes?

@apiraino
Copy link
Contributor

apiraino commented Dec 2, 2024

I think so, so I will go and close this one.

This thing of closing regressions accepted for one reason or another was mentioned on Zulip but I'd want to document this reasoning in some place to make it more "official" and discoverable.

@apiraino apiraino closed this as completed Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

7 participants