don't pass driver to generateExplicitModuleDependenciesBuildJobs

Author: sina-mahdavi

Sources/SwiftDriver/ExplicitModuleBuilds/ExplicitDependencyBuildPlanner.swift

@@ -152,23 +152,23 @@ public typealias ExternalTargetModuleDetailsMap = [ModuleDependencyId: ExternalT
   /// module's target version (PCM Args).
   /// 2. Generate jobs for all Clang modules, now that we have each module's set of PCM versions it must be
   /// built against.
-  public mutating func generateExplicitModuleDependenciesBuildJobs(driver: Driver) throws -> [Job] {
+  public mutating func generateExplicitModuleDependenciesBuildJobs() throws -> [Job] {
     let mainModuleId: ModuleDependencyId = .swift(dependencyGraph.mainModuleName)
     guard let mainModuleDependencies = reachabilityMap[mainModuleId] else {
       fatalError("Expected reachability information for the main module.")
     }
     let swiftDependenciesJobs =
-      try generateSwiftDependenciesBuildJobs(for: mainModuleDependencies, driver: driver)
+      try generateSwiftDependenciesBuildJobs(for: mainModuleDependencies)
 
     // Generate build jobs for all Clang modules
     let clangDependenciesJobs =
-      try generateClangDependenciesBuildJobs(for: mainModuleDependencies, driver: driver)
+      try generateClangDependenciesBuildJobs(for: mainModuleDependencies)
 
     return swiftDependenciesJobs + clangDependenciesJobs
   }
 
   /// Generate a build job for each Swift module in the set of supplied `dependencies`
-  private mutating func generateSwiftDependenciesBuildJobs(for dependencies: Set<ModuleDependencyId>, driver: Driver)
+  private mutating func generateSwiftDependenciesBuildJobs(for dependencies: Set<ModuleDependencyId>)
   throws -> [Job] {
     var jobs: [Job] = []
     let swiftDependencies = dependencies.filter {
@@ -245,7 +245,7 @@ public typealias ExternalTargetModuleDetailsMap = [ModuleDependencyId: ExternalT
   }
 
   /// Generate a build job for each Clang module in the set of supplied `dependencies`.
-  private mutating func generateClangDependenciesBuildJobs(for dependencies: Set<ModuleDependencyId>, driver: Driver)
+  private mutating func generateClangDependenciesBuildJobs(for dependencies: Set<ModuleDependencyId>)
   throws -> [Job] {
     var jobs: [Job] = []
     let clangDependencies = dependencies.filter {

Sources/SwiftDriver/Jobs/Planning.swift

@@ -226,7 +226,7 @@ extension Driver {
     guard var resolvedModulePlanner = explicitModulePlanner else {
       return
     }
-    let modulePrebuildJobs = try resolvedModulePlanner.generateExplicitModuleDependenciesBuildJobs(driver: self)
+    let modulePrebuildJobs = try resolvedModulePlanner.generateExplicitModuleDependenciesBuildJobs()
     // If asked, add jobs to precompile module dependencies. Otherwise exit.
     // We may have a dependency graph but not be required to add pre-compile jobs to the build plan,
     // for example when `-explain-dependency` is being used.

Tests/SwiftDriverTests/ExplicitModuleBuildTests.swift

@@ -177,7 +177,7 @@ final class ExplicitModuleBuildTests: XCTestCase {
                                            toolchain: driver.toolchain,
                                            supportsScannerPrefixMapPaths: driver.isFrontendArgSupported(.scannerPrefixMapPaths))
       let modulePrebuildJobs =
-        try explicitDependencyBuildPlanner.generateExplicitModuleDependenciesBuildJobs(driver: driver)
+        try explicitDependencyBuildPlanner.generateExplicitModuleDependenciesBuildJobs()
       XCTAssertEqual(modulePrebuildJobs.count, 4)
       for job in modulePrebuildJobs {
         XCTAssertEqual(job.outputs.count, 1)