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

build: Repair the build on WASI platform #4934

Closed

Conversation

kateinoigakukun
Copy link
Member

@kateinoigakukun kateinoigakukun commented Mar 30, 2024

Fixes package-nized build for WASI of _CoreFoundation.

$ swift --version
Swift version 6.0-dev (LLVM b7b175a4a1263ea, Swift 07eb52a80dee5a1)
Target: x86_64-unknown-linux-gnu
$ swift experimental-sdk install https://github.com/swiftwasm/swift/releases/download/swift-wasm-DEVELOPMENT-SNAPSHOT-2024-03-29-a/swift-wasm-DEVELOPMENT-SNAPSHOT-2024-03-29-a-ubuntu22.04_x86_64.artifactbundle.zip
$ swift build --experimental-swift-sdk DEVELOPMENT-SNAPSHOT-2024-03-29-a-wasm --target _CoreFoundation

The Foundation product needs macro with cross-compilation support in SwiftPM, so it's still not working yet.

@parkera
Copy link
Contributor

parkera commented Jul 24, 2024

Can we rebase this on main?

@parkera parkera deleted the branch swiftlang:package July 30, 2024 22:10
@parkera parkera closed this Jul 30, 2024
MaxDesiatov pushed a commit that referenced this pull request Aug 5, 2024
MaxDesiatov pushed a commit that referenced this pull request Aug 5, 2024
Cherry picked from #4934

(cherry picked from commit 51ee690)
MaxDesiatov added a commit that referenced this pull request Aug 7, 2024
* build: Repair the build on WASI platform

Cherry picked from #4934

* Add `.windows` to `platformsWithThreads` in `Package.swift`

* Reflect `Package.swift` WASI changes in `CMakeLists.txt`

(cherry picked from commit 8873088)

---------

Co-authored-by: Yuta Saito <[email protected]>
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.

2 participants