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

Disable file/dispatch-related code in Foundation for WASI #3050

Merged
merged 1 commit into from
Sep 24, 2021

Conversation

MaxDesiatov
Copy link
Member

Functionality related to Dispatch, run loops, and full filesystem access (other than stdio/stderr file handles) is not available on WASI/Wasm. As a follow-up to previous Core Foundation PRs, this one updates parts of Foundation to add support for that platform.

@MaxDesiatov
Copy link
Member Author

@swift-ci please test

@MaxDesiatov
Copy link
Member Author

@swift-ci please test

@millenomi
Copy link
Contributor

Same comments here re: NSCoding.

@MaxDesiatov MaxDesiatov force-pushed the maxd/wasi-foundation branch 4 times, most recently from c7ae122 to 7a98383 Compare August 25, 2021 09:44
@MaxDesiatov
Copy link
Member Author

@swift-ci please test

@MaxDesiatov
Copy link
Member Author

@millenomi I've re-enabled NSCoding-related code here, it's ready for review

@MaxDesiatov
Copy link
Member Author

@swift-ci please test

@MaxDesiatov MaxDesiatov force-pushed the maxd/wasi-foundation branch 4 times, most recently from c54b739 to ed90316 Compare September 7, 2021 15:29
@MaxDesiatov
Copy link
Member Author

@swift-ci please test

@MaxDesiatov
Copy link
Member Author

@swift-ci please test macOS platform

@millenomi
Copy link
Contributor

@MaxDesiatov I'm looking tomorrow.

@RSilicon
Copy link
Contributor

@millenomi any updates?

@millenomi
Copy link
Contributor

Things slipped a little but I'm finally looking right now >.<'

@millenomi
Copy link
Contributor

I'm going to take it for right now, but I hope we can have follow-ups to re-expose write(to…) methods etc. to throw at runtime rather than be a hard compile error on use and force #if os(WASI) on clients. cc @MaxDesiatov

@millenomi millenomi merged commit ee856f1 into main Sep 24, 2021
@MaxDesiatov
Copy link
Member Author

That makes sense, thanks!

@MaxDesiatov MaxDesiatov deleted the maxd/wasi-foundation branch September 24, 2021 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants