-
Notifications
You must be signed in to change notification settings - Fork 5
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
Lower deployment target to macOS 10.13 #91
Lower deployment target to macOS 10.13 #91
Conversation
Sources/WasmKit/ModuleParser.swift
Outdated
@@ -126,7 +125,8 @@ func parseModule<Stream: ByteStream>(stream: Stream, features: WasmFeatureSet = | |||
memoryTypes: module.memories.map { $0.type }, | |||
tables: module.tables | |||
) | |||
let enableAssertDefault = _slowPath(getenv("WASMKIT_ENABLE_ASSERT") != nil) | |||
#warning("Get env without foundation") | |||
let enableAssertDefault = false // _slowPath(getenv("WASMKIT_ENABLE_ASSERT") != nil) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kateinoigakukun any suggestions for working around this? This is the only spot where I haven't been able to find a solution yet that doesn't involve Foundation — afaict SwiftSystem doesn't have support for reading the environment.
For Foundation dependency, I cleaned up a bit and picked a part of your changes here #92 |
# Conflicts: # Package.swift # Sources/CLI/Run/Run.swift # Sources/WASI/Platform/File.swift # Sources/WASI/WASI.swift # Sources/WasmKit/Execution/Runtime/Profiler.swift # Sources/WasmKit/ModuleParser.swift # Sources/WasmParser/CMakeLists.txt # Sources/WasmParser/Stream/FileHandleStream.swift
@kateinoigakukun I updated this PR to just lower the deployment target, should be ready for review! Aside: can't seem to cancel stale CI jobs, feel free to do that for the intermediate commits. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Haven't tested everything yet but so far it builds. See:
https://github.com/apple/swift/pull/73031/files#r1581621839
https://github.com/apple/swift/pull/73031/files#r1581597854