Skip to content

[wasm] Merge all wasi-sysroot into a single sysroot directory #76544

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

Conversation

kateinoigakukun
Copy link
Member

This change merges all wasi-sysroot directories into a single sysroot directory to reuse header files across wasm32-wasi and wasm32-wasip1-threads. The resulted sysroot layout aligns with the layout in wasi-sdk distribution.

This change merges all wasi-sysroot directories into a single sysroot
directory to reuse header files across `wasm32-wasi` and `wasm32-wasip1-threads`.
@kateinoigakukun
Copy link
Member Author

@swift-ci smoke test

@MaxDesiatov MaxDesiatov added WebAssembly Platform: WebAssembly WASI WebAssembly System Interface support build-script Area → utils: The build script labels Sep 18, 2024
@MaxDesiatov
Copy link
Contributor

@swift-ci test linux

@MaxDesiatov
Copy link
Contributor

@swift-ci build toolchain

@MaxDesiatov
Copy link
Contributor

@swift-ci test

@MaxDesiatov
Copy link
Contributor

swiftlang/sourcekit-lsp#1706

@swift-ci build toolchain

@MaxDesiatov
Copy link
Contributor

@swift-ci test linux

@kateinoigakukun
Copy link
Member Author

It's not something flaky but the change includes something wrong. I need to debug it

@MaxDesiatov
Copy link
Contributor

@swift-ci test linux

@kateinoigakukun
Copy link
Member Author

kateinoigakukun commented Oct 28, 2024

Closing for now as this cannot be practically accomplished without fixing swiftlang/swift-package-manager#7973
I'll re-open this once I make a plan to fix the SDK design

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build-script Area → utils: The build script WASI WebAssembly System Interface support WebAssembly Platform: WebAssembly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants