-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[WASI][draft] UnsupportedOSPlatformAttribute on the API surface #107185
[WASI][draft] UnsupportedOSPlatformAttribute on the API surface #107185
Conversation
Note regarding the
|
1 similar comment
Note regarding the
|
@lewing @terrajobst does this need proper API review process ? This is quickly evolving landscape but having those attributes to give us compiler errors is useful in meantime. |
Great work! My note is that there should be an issue detailing missing WASI functionality like in your point with the terminal settings. We could (and arguably should) use it to influence future WASI APIs. |
Shipping another set of ref assemblies is ok as a temporary solution for previews. It won't work well as a permanent solution since it is not compatible package ecosystem. The system is not setup to allow packages to ship multiple set of ref assemblies per TFM. |
We already do that for browser, but we are not happy about it. What would be alternate solution ? |
Add a new RIDs like Shipping second set of ref assemblies as a temporary measure makes sense only if we believe that multi-threaded wasm is going to be available everywhere relatively soon and we will be able to abandon support for single-threaded wasm. |
okay, thanks.
I think that improved determinism and easier ability to snapshot/hydrate memory makes single-threaded wasm long term attractive for WASI. I think that some wasm vendors don't even consider supporting MT at all. The question remains how many server side dotnet workloads will be possible to convert to ST+async. Many existing server side codebases use blocking MT code and make network calls from synchronous methods. We don't have to solve MT today. |
a5348dc
to
db53310
Compare
Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it. |
This is adding
UnsupportedOSPlatform
attributes on API where WASI platform can't support the functionality.It follows
Browser
OS example becauseTODO
wasi-sockets
in progressSocketsHttpHandler
will be probably possible soon after. With limitations.System.IO.Compression.Brotli
System.Net.Security
Slowly Evolving
Console
control characters are probably possible, becausewasi-cli
stdio is byte-stream not character stream. But the VM would not tell us what type terminal is connected (if any).SmtpClient
could be based on sockets, but it's low priority.