Skip to content

Commit e7b61c6

Browse files
artemcmartemcm
committed
Resolve warnings about deprecated 'AbsolutePath' initializer
1 parent 067ae69 commit e7b61c6

File tree

7 files changed

+28
-26
lines changed

7 files changed

+28
-26
lines changed

‎Sources/swift-build-sdk-interfaces/main.swift

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,7 @@ do {
174174
// modules for which a textual interface is discovered, ensuring that modules
175175
// always build from interface when one is available.
176176
if let supportedFlagsTestDriver = try? Driver(args: ["swiftc", "-v"],
177+
envBlock: ProcessEnv.block,
177178
executor: executor,
178179
compilerExecutableDir: swiftcPath.parentDirectory),
179180
supportedFlagsTestDriver.isFrontendArgSupported(.moduleLoadMode) {
@@ -185,6 +186,7 @@ do {
185186

186187
let baselineABIDir = try getArgumentAsPath("-baseline-abi-dir")
187188
var driver = try Driver(args: args,
189+
envBlock: ProcessEnv.block,
188190
diagnosticsOutput: .engine(diagnosticsEngine),
189191
executor: executor,
190192
compilerExecutableDir: swiftcPath.parentDirectory)

‎Sources/swift-driver/main.swift

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,7 @@ do {
144144
fileSystem: localFileSystem,
145145
env: ProcessEnv.block)
146146
var driver = try Driver(args: arguments,
147+
envBlock: ProcessEnv.block,
147148
diagnosticsOutput: .engine(diagnosticsEngine),
148149
executor: executor,
149150
integratedDriver: false)

‎Tests/SwiftDriverTests/AssertDiagnosticsTests.swift

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -186,11 +186,7 @@ class FailableTestCase: XCTestCase {
186186
}
187187

188188
if #available(macOS 10.13, *) {
189-
super.recordFailure(
190-
withDescription: description,
191-
inFile: filePath, atLine: lineNumber,
192-
expected: expected
193-
)
189+
super.record(XCTIssue(type: .assertionFailure, compactDescription: description, sourceCodeContext: XCTSourceCodeContext(location: XCTSourceCodeLocation(filePath: filePath, lineNumber: lineNumber))))
194190
} else {
195191
fatalError(description, line: UInt(lineNumber))
196192
}

‎Tests/SwiftDriverTests/Inputs/ExplicitModuleDependencyBuildInputs.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,9 +163,9 @@ enum ModuleDependenciesInputs {
163163
"-Xcc",
164164
"-fno-implicit-module-maps",
165165
"-candidate-module-file",
166-
"\(AbsolutePath("/dummy/path2/SwiftOnoneSupport.swiftmodule").nativePathString(escaped: true))",
166+
"\(try! AbsolutePath(validating: "/dummy/path2/SwiftOnoneSupport.swiftmodule").nativePathString(escaped: true))",
167167
"-candidate-module-file",
168-
"\(AbsolutePath("/dummy/path1/SwiftOnoneSupport.swiftmodule").nativePathString(escaped: true))",
168+
"\(try! AbsolutePath(validating: "/dummy/path1/SwiftOnoneSupport.swiftmodule").nativePathString(escaped: true))",
169169
"-target",
170170
"x86_64-apple-macosx10.15",
171171
"-sdk",

‎Tests/SwiftDriverTests/JobExecutorTests.swift

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ import TSCBasic
1616
import SwiftDriverExecution
1717
import TestUtilities
1818

19-
extension Job.ArgTemplate: ExpressibleByStringLiteral {
19+
extension Job.ArgTemplate: @retroactive ExpressibleByStringLiteral {
2020
public init(stringLiteral value: String) {
2121
self = .flag(value)
2222
}
@@ -490,6 +490,7 @@ final class JobExecutorTests: XCTestCase {
490490
var driver = try Driver(args: ["swiftc", main.pathString,
491491
"-driver-filelist-threshold", "0",
492492
"-o", outputPath.pathString] + getHostToolchainSdkArg(executor),
493+
envBlock: ProcessEnv.block,
493494
diagnosticsOutput: .engine(diags),
494495
fileSystem: localFileSystem,
495496
executor: executor)
@@ -525,6 +526,7 @@ final class JobExecutorTests: XCTestCase {
525526
"-save-temps",
526527
"-driver-filelist-threshold", "0",
527528
"-o", outputPath.pathString] + getHostToolchainSdkArg(executor),
529+
envBlock: ProcessEnv.block,
528530
diagnosticsOutput: .engine(diags),
529531
fileSystem: localFileSystem,
530532
executor: executor)
@@ -560,6 +562,7 @@ final class JobExecutorTests: XCTestCase {
560562
"-driver-filelist-threshold", "0",
561563
"-Xfrontend", "-debug-crash-immediately",
562564
"-o", outputPath.pathString] + getHostToolchainSdkArg(executor),
565+
envBlock: ProcessEnv.block,
563566
diagnosticsOutput: .engine(diags),
564567
fileSystem: localFileSystem,
565568
executor: executor)

‎Tests/SwiftDriverTests/SwiftDriverTests.swift

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -620,7 +620,7 @@ final class SwiftDriverTests: XCTestCase {
620620
XCTAssertJobInvocationMatches(jobs[0], .flag("-file-compilation-dir"), .path(VirtualPath.lookup(path)))
621621
}
622622

623-
let workingDirectory = AbsolutePath("/tmp")
623+
let workingDirectory = try AbsolutePath(validating: "/tmp")
624624
try assertNoDriverDiagnostics(args: "swiftc", "foo.swift", "-g", "-c", "-working-directory", workingDirectory.nativePathString(escaped: false)) { driver in
625625
let jobs = try driver.planBuild()
626626
let path = try VirtualPath.intern(path: workingDirectory.nativePathString(escaped: false))
@@ -1735,7 +1735,7 @@ final class SwiftDriverTests: XCTestCase {
17351735
let manyArgs = (1...20000).map { "-DTEST_\($0)" }
17361736
// Needs response file
17371737
do {
1738-
let source = AbsolutePath("/foo.swift")
1738+
let source = try AbsolutePath(validating: "/foo.swift")
17391739
var driver = try Driver(args: ["swift"] + manyArgs + [source.nativePathString(escaped: false)])
17401740
let jobs = try driver.planBuild()
17411741
XCTAssertEqual(jobs.count, 1)
@@ -1767,7 +1767,7 @@ final class SwiftDriverTests: XCTestCase {
17671767

17681768
// Forced response file
17691769
do {
1770-
let source = AbsolutePath("/foo.swift")
1770+
let source = try AbsolutePath(validating: "/foo.swift")
17711771
var driver = try Driver(args: ["swift"] + [source.nativePathString(escaped: false)])
17721772
let jobs = try driver.planBuild()
17731773
XCTAssertEqual(jobs.count, 1)
@@ -3143,7 +3143,7 @@ final class SwiftDriverTests: XCTestCase {
31433143

31443144

31453145
func testIndexFileEntryInSupplementaryFileOutputMap() throws {
3146-
let workingDirectory = AbsolutePath("/tmp")
3146+
let workingDirectory = try AbsolutePath(validating: "/tmp")
31473147
var driver1 = try Driver(args: [
31483148
"swiftc", "foo1.swift", "foo2.swift", "foo3.swift", "foo4.swift", "foo5.swift",
31493149
"-index-file", "-index-file-path", "foo5.swift", "-o", "/tmp/t.o",
@@ -3894,7 +3894,7 @@ final class SwiftDriverTests: XCTestCase {
38943894
XCTAssertEqual(plannedJobs.count, 2)
38953895
XCTAssertEqual(plannedJobs[0].kind, .compile)
38963896
XCTAssertEqual(plannedJobs[1].kind, .link)
3897-
try XCTAssertJobInvocationMatches(plannedJobs[0], .flag("-default-isolation"), "MainActor")
3897+
XCTAssertJobInvocationMatches(plannedJobs[0], .flag("-default-isolation"), "MainActor")
38983898
}
38993899

39003900
func testImmediateMode() throws {
@@ -6835,9 +6835,9 @@ final class SwiftDriverTests: XCTestCase {
68356835

68366836
// 32-bit iOS jobs under Embedded should be allowed regardless of OS version
68376837
do {
6838-
try Driver(args: ["swiftc", "-c", "-target", "armv7-apple-ios8", "-enable-experimental-feature", "Embedded", "foo.swift"])
6839-
try Driver(args: ["swiftc", "-c", "-target", "armv7-apple-ios12.1", "-enable-experimental-feature", "Embedded", "foo.swift"])
6840-
try Driver(args: ["swiftc", "-c", "-target", "armv7-apple-ios16", "-enable-experimental-feature", "Embedded", "foo.swift"])
6838+
let _ = try Driver(args: ["swiftc", "-c", "-target", "armv7-apple-ios8", "-enable-experimental-feature", "Embedded", "foo.swift"])
6839+
let _ = try Driver(args: ["swiftc", "-c", "-target", "armv7-apple-ios12.1", "-enable-experimental-feature", "Embedded", "foo.swift"])
6840+
let _ = try Driver(args: ["swiftc", "-c", "-target", "armv7-apple-ios16", "-enable-experimental-feature", "Embedded", "foo.swift"])
68416841
}
68426842

68436843
do {
@@ -7136,7 +7136,7 @@ final class SwiftDriverTests: XCTestCase {
71367136
}
71377137

71387138
do {
7139-
let workingDirectory = AbsolutePath("/foo/bar")
7139+
let workingDirectory = try AbsolutePath(validating: "/foo/bar")
71407140

71417141
// Inputs with relative paths with -working-directory flag should prefix all inputs
71427142
var driver = try Driver(args: ["swiftc",
@@ -7148,9 +7148,9 @@ final class SwiftDriverTests: XCTestCase {
71487148
let plannedJobs = try driver.planBuild()
71497149
let compileJob = plannedJobs[0]
71507150
XCTAssertEqual(compileJob.kind, .compile)
7151-
try XCTAssertJobInvocationMatches(compileJob, .flag("-primary-file"), .path(.absolute(workingDirectory.appending(component: "foo.swift"))))
7152-
try XCTAssertJobInvocationMatches(compileJob, .flag("-resource-dir"), .path(.absolute(workingDirectory.appending(component: "relresourcepath"))))
7153-
try XCTAssertJobInvocationMatches(compileJob, .flag("-sdk"), .path(.absolute(workingDirectory.appending(component: "relsdkpath"))))
7151+
XCTAssertJobInvocationMatches(compileJob, .flag("-primary-file"), .path(.absolute(workingDirectory.appending(component: "foo.swift"))))
7152+
XCTAssertJobInvocationMatches(compileJob, .flag("-resource-dir"), .path(.absolute(workingDirectory.appending(component: "relresourcepath"))))
7153+
XCTAssertJobInvocationMatches(compileJob, .flag("-sdk"), .path(.absolute(workingDirectory.appending(component: "relsdkpath"))))
71547154
}
71557155

71567156
try withTemporaryFile { fileMapFile in
@@ -7175,7 +7175,7 @@ final class SwiftDriverTests: XCTestCase {
71757175
let plannedJobs = try driver.planBuild()
71767176
let compileJob = plannedJobs[0]
71777177
XCTAssertEqual(compileJob.kind, .compile)
7178-
try XCTAssertJobInvocationMatches(compileJob, .flag("-o"), .path(.absolute(.init("/tmp/foo/.build/x86_64-apple-macosx/debug/foo.build/foo.o"))))
7178+
try XCTAssertJobInvocationMatches(compileJob, .flag("-o"), .path(.absolute(.init(validating: "/tmp/foo/.build/x86_64-apple-macosx/debug/foo.build/foo.o"))))
71797179
}
71807180

71817181
try withTemporaryFile { fileMapFile in
@@ -7185,7 +7185,7 @@ final class SwiftDriverTests: XCTestCase {
71857185
"diagnostics": "/tmp/foo/.build/x86_64-apple-macosx/debug/foo.build/master.dia",
71867186
"emit-module-diagnostics": "/tmp/foo/.build/x86_64-apple-macosx/debug/foo.build/master.emit-module.dia"
71877187
},
7188-
"\(AbsolutePath("/some/workingdir/foo.swift").nativePathString(escaped: true))": {
7188+
"\(try AbsolutePath(validating: "/some/workingdir/foo.swift").nativePathString(escaped: true))": {
71897189
"object": "/tmp/foo/.build/x86_64-apple-macosx/debug/foo.build/foo.o"
71907190
}
71917191
}
@@ -7196,12 +7196,12 @@ final class SwiftDriverTests: XCTestCase {
71967196
var driver = try Driver(args: ["swiftc",
71977197
"-target", "arm64-apple-ios13.1",
71987198
"foo.swift",
7199-
"-working-directory", AbsolutePath("/some/workingdir").nativePathString(escaped: false),
7199+
"-working-directory", try AbsolutePath(validating: "/some/workingdir").nativePathString(escaped: false),
72007200
"-output-file-map", fileMapFile.path.description])
72017201
let plannedJobs = try driver.planBuild()
72027202
let compileJob = plannedJobs[0]
72037203
XCTAssertEqual(compileJob.kind, .compile)
7204-
try XCTAssertJobInvocationMatches(compileJob, .flag("-o"), .path(.absolute(.init("/tmp/foo/.build/x86_64-apple-macosx/debug/foo.build/foo.o"))))
7204+
try XCTAssertJobInvocationMatches(compileJob, .flag("-o"), .path(.absolute(.init(validating: "/tmp/foo/.build/x86_64-apple-macosx/debug/foo.build/foo.o"))))
72057205
}
72067206
}
72077207

@@ -7593,7 +7593,7 @@ final class SwiftDriverTests: XCTestCase {
75937593
var env = ProcessEnv.block
75947594
env["PLATFORM_DIR"] = "/tmp/PlatformDir/\(platform).platform"
75957595

7596-
let workingDirectory = AbsolutePath("/tmp")
7596+
let workingDirectory = try AbsolutePath(validating: "/tmp")
75977597

75987598
var driver = try Driver(
75997599
args: ["swiftc", "-typecheck", "foo.swift", "-sdk", VirtualPath.absolute(sdkRoot).name, "-plugin-path", "PluginA", "-external-plugin-path", "Plugin~B#Bexe", "-load-plugin-library", "PluginB2", "-plugin-path", "PluginC", "-working-directory", workingDirectory.nativePathString(escaped: false)],

‎Tests/SwiftDriverTests/TripleTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1452,7 +1452,7 @@ final class TripleTests: XCTestCase {
14521452
}
14531453
}
14541454

1455-
extension Triple.Version: ExpressibleByStringLiteral {
1455+
extension Triple.Version: @retroactive ExpressibleByStringLiteral {
14561456
public init(stringLiteral value: String) {
14571457
self.init(parse: value)
14581458
}

0 commit comments

Comments
 (0)