Skip to content

Fix .wasm product paths for wasm32-unknown-none triple #7355

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

Merged
merged 5 commits into from
Feb 21, 2024

Conversation

MaxDesiatov
Copy link
Contributor

Absence of this extension on produced binaries makes it inconvenient to work with triple, as you have to add that extension manually after the fact. For examples, HTTP servers frequently don't infer correct mime-type for these binaries, which confuses browsers that try to load these binaries.

Absence of this extension on produced binaries makes it inconvenient to work with triple, as you have to add that extension manually after the fact. For examples, HTTP servers frequently don't infer correct mime-type for these binaries, which confuses browsers that try to load these binaries.
@MaxDesiatov MaxDesiatov added swift test Changes impacting `swift test` tool swift build Changes impacting `swift build` embedded Support for Embedded Swift labels Feb 20, 2024
@MaxDesiatov MaxDesiatov self-assigned this Feb 20, 2024
@@ -3431,111 +3431,6 @@ final class BuildPlanTests: XCTestCase {
XCTAssertMatch(executablePathExtension, "exe")
}

func testWASITarget() throws {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved to Tests/BuildTests/CrossCompilationBuildPlanTests.swift to make Tests/BuildTests/BuildPlanTests.swift smaller and increase parallelization during testing, as separate XCTestCase types can be parallelized, while multiple functions in XCTestCase can't.

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov MaxDesiatov enabled auto-merge (squash) February 21, 2024 10:57
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test windows

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test windows

@MaxDesiatov MaxDesiatov merged commit 98747d5 into main Feb 21, 2024
@MaxDesiatov MaxDesiatov deleted the maxd/wasm-unknown-none-ext branch February 21, 2024 13:31
furby-tm pushed a commit to wabiverse/swift-package-manager that referenced this pull request May 15, 2024
…#7355)

Absence of this extension on produced binaries makes it inconvenient to
work with triple, as you have to add that extension manually after the
fact. For examples, HTTP servers frequently don't infer correct
mime-type for these binaries, which confuses browsers that try to load
these binaries.
furby-tm pushed a commit to wabiverse/swift-package-manager that referenced this pull request May 15, 2024
…#7355)

Absence of this extension on produced binaries makes it inconvenient to
work with triple, as you have to add that extension manually after the
fact. For examples, HTTP servers frequently don't infer correct
mime-type for these binaries, which confuses browsers that try to load
these binaries.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
embedded Support for Embedded Swift swift build Changes impacting `swift build` swift test Changes impacting `swift test` tool WebAssembly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants