Merge pull request #617 from artemcm/OlderCompilerCompatFix

artemcm · web-flow · commit 480f6f0210d4 · 2021-04-27T09:34:22.000-07:00
Small fix to `emitModuleJob` for compatibility with &lt;=5.3 compilers.
diff --git a/Sources/SwiftDriver/Jobs/Planning.swift b/Sources/SwiftDriver/Jobs/Planning.swift
@@ -185,9 +185,9 @@ extension Driver {
 
   private mutating func addEmitModuleJob(addJobBeforeCompiles: (Job) -> Void, addJobAfterCompiles: (Job) -> Void) throws {
     if shouldCreateEmitModuleJob {
-      let emitModuleJob = try emitModuleJob()
-      addJobBeforeCompiles(emitModuleJob)
-      try addVerifyJobs(emitModuleJob: emitModuleJob, addJob: addJobAfterCompiles)
+      let emitJob = try emitModuleJob()
+      addJobBeforeCompiles(emitJob)
+      try addVerifyJobs(emitModuleJob: emitJob, addJob: addJobAfterCompiles)
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -185,9 +185,9 @@ extension Driver {`
`185`	`185`
`186`	`186`	`private mutating func addEmitModuleJob(addJobBeforeCompiles: (Job) -> Void, addJobAfterCompiles: (Job) -> Void) throws {`
`187`	`187`	`if shouldCreateEmitModuleJob {`
`188`		`- let emitModuleJob = try emitModuleJob()`
`189`		`- addJobBeforeCompiles(emitModuleJob)`
`190`		`- try addVerifyJobs(emitModuleJob: emitModuleJob, addJob: addJobAfterCompiles)`
	`188`	`+ let emitJob = try emitModuleJob()`
	`189`	`+ addJobBeforeCompiles(emitJob)`
	`190`	`+ try addVerifyJobs(emitModuleJob: emitJob, addJob: addJobAfterCompiles)`
`191`	`191`	`}`
`192`	`192`	`}`
`193`	`193`