Skip to content

Commit e9a1206

Browse files
madsodgaardktoso
andauthored
misc: use upstream Subprocess (#490)
Co-authored-by: Konrad `ktoso` Malawski <konrad_malawski@apple.com>
1 parent 0eb7c22 commit e9a1206

27 files changed

Lines changed: 14 additions & 6366 deletions

‎Package.swift‎

Lines changed: 3 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -259,10 +259,7 @@ let package = Package(
259259
.package(url: "https://github.com/apple/swift-system", from: "1.4.0"),
260260
.package(url: "https://github.com/apple/swift-log", from: "1.2.0"),
261261
.package(url: "https://github.com/apple/swift-collections", .upToNextMinor(from: "1.3.0")), // primarily for ordered collections
262-
263-
// // FIXME: swift-subprocess stopped supporting 6.0 when it moved into a package;
264-
// // we'll need to drop 6.0 as well, but currently blocked on doing so by swiftpm plugin pending design questions
265-
// .package(url: "https://github.com/swiftlang/swift-subprocess.git", revision: "de15b67f7871c8a039ef7f4813eb39a8878f61a6"),
262+
.package(url: "https://github.com/swiftlang/swift-subprocess.git", from: "0.2.1", traits: ["SubprocessFoundation"]),
266263

267264
// Benchmarking
268265
.package(url: "https://github.com/ordo-one/package-benchmark", .upToNextMajor(from: "1.4.0")),
@@ -464,8 +461,7 @@ let package = Package(
464461
"JavaTypes",
465462
"SwiftJavaShared",
466463
"SwiftJavaConfigurationShared",
467-
// .product(name: "Subprocess", package: "swift-subprocess")
468-
"_Subprocess",
464+
.product(name: "Subprocess", package: "swift-subprocess")
469465
],
470466
swiftSettings: [
471467
.swiftLanguageMode(.v5),
@@ -590,28 +586,6 @@ let package = Package(
590586
.swiftLanguageMode(.v5),
591587
.unsafeFlags(["-I\(javaIncludePath)", "-I\(javaPlatformIncludePath)"])
592588
]
593-
),
594-
595-
// Experimental Foundation Subprocess Copy
596-
.target(
597-
name: "_SubprocessCShims",
598-
swiftSettings: [
599-
.swiftLanguageMode(.v5)
600-
]
601-
),
602-
.target(
603-
name: "_Subprocess",
604-
dependencies: [
605-
"_SubprocessCShims",
606-
.product(name: "SystemPackage", package: "swift-system"),
607-
],
608-
swiftSettings: [
609-
.swiftLanguageMode(.v5),
610-
.define(
611-
"SYSTEM_PACKAGE_DARWIN",
612-
.when(platforms: [.macOS, .macCatalyst, .iOS, .watchOS, .tvOS, .visionOS])),
613-
.define("SYSTEM_PACKAGE"),
614-
]
615-
),
589+
)
616590
]
617591
)

‎Sources/ExampleSwiftLibrary/MySwiftLibrary.swift‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ import Glibc
2323
import CRT
2424
#elseif canImport(Darwin)
2525
import Darwin.C
26+
#elseif canImport(Android)
27+
import Android
2628
#endif
2729

2830
public func helloWorld() {

‎Sources/SwiftJavaTool/Commands/ResolveCommand.swift‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import JavaUtilJar
2121
import SwiftJavaToolLib
2222
import SwiftJavaConfigurationShared
2323
import SwiftJavaShared
24-
import _Subprocess
24+
import Subprocess
2525
#if canImport(System)
2626
import System
2727
#else
@@ -146,7 +146,7 @@ extension SwiftJava.ResolveCommand {
146146
try! printGradleProject(directory: resolverDir, dependencies: dependencies)
147147

148148
if #available(macOS 15, *) {
149-
let process = try! await _Subprocess.run(
149+
let process = try! await Subprocess.run(
150150
.path(FilePath(resolverDir.appendingPathComponent("gradlew").path)),
151151
arguments: [
152152
"--no-daemon",

0 commit comments

Comments
 (0)