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

Remove emsdk version update from 1.84.0 relnotes #135266

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

Conversation

kadiwa4
Copy link
Contributor

@kadiwa4 kadiwa4 commented Jan 8, 2025

See this comment. The reproducer in that comment does indeed show that rustup's rust-std component is still compiled with the old emscripten ABI because libc's config flag emscripten_new_stat_abi is not set.

#131533 presumably had no effect because the wrong CI file was modified. So nothing has changed since 1.83.0. The PR author (workingjubilee) is currently on vacation.
Also the issue #131467 should be reopened.

@rustbot
Copy link
Collaborator

rustbot commented Jan 8, 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 S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-release Relevant to the release subteam, which will review and decide on the PR/issue. labels Jan 8, 2025
@workingjubilee
Copy link
Member

I suppose we needed a libc 0.2.170 release which hasn't happened yet.

@workingjubilee
Copy link
Member

and a backport. sigh.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. 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.

4 participants