-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Rename wasm32-wasi
to wasm32-wasi-preview1
#110596
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
Changes from all commits
23b66f2
6afae81
7bb19c4
24347f2
e2cef50
7080065
fa04cc1
07d3b4b
fae26c9
f066a51
a04c8a0
e9f5aaa
453bc91
734c8c9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -306,7 +306,7 @@ fn copy_self_contained_objects( | |
.unwrap_or_else(|| { | ||
panic!("Target {:?} does not have a \"wasi-root\" key", target.triple) | ||
}) | ||
.join("lib/wasm32-wasi"); | ||
.join("lib/wasm32-wasi-preview1"); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is it able to find wasi-libc sysroot? My understanding wasi-libc targets doesn't mention There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I tried to compile a simple file with
And it failed with
Also I ran the tests the same way I did here #112922 (comment) Without this change I'm able to compile the same file to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also I might be doing something wrong though, but could you try to run it yourself please @yoshuawuyts There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. tried to fix it by removing There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
You most likely have to revert the other change in the same file (line 411) as well. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah I did, stiil the same There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Okay, several lines above it should check for containing wasi instead of ending with it. I will send a patch There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think I don't have permissions to update your branch @yoshuawuyts , here's the fix + merge master branch and resolving of conflicts https://github.com/rust-lang/rust/compare/master...g0djan:rust:rename-wasm32-wasi?expand=1 I also reran ui tests and now your pr doesn't break or fix any tests compared to running tests without your pr. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I've spoken to the maintainers of wasi-libc, and they indeed haven't updated the target yet to include the So thank you for catching and fixing this; we indeed should land this change! |
||
for &obj in &["libc.a", "crt1-command.o", "crt1-reactor.o"] { | ||
copy_and_stamp( | ||
builder, | ||
|
@@ -408,7 +408,7 @@ pub fn std_cargo(builder: &Builder<'_>, target: TargetSelection, stage: u32, car | |
|
||
if target.ends_with("-wasi") { | ||
if let Some(p) = builder.wasi_root(target) { | ||
let root = format!("native={}/lib/wasm32-wasi", p.to_str().unwrap()); | ||
let root = format!("native={}/lib/wasm32-wasi-preview1", p.to_str().unwrap()); | ||
cargo.rustflag("-L").rustflag(&root); | ||
} | ||
} | ||
|
This file was deleted.
Uh oh!
There was an error while loading. Please reload this page.