Manifest loading logging

jmschonfeld · jmschonfeld · commit c0969aeb51a2 · 2024-07-01T20:21:04.000-07:00
diff --git a/Sources/PackageLoading/ManifestLoader.swift b/Sources/PackageLoading/ManifestLoader.swift
@@ -1081,7 +1081,7 @@ public final class ManifestLoader: ManifestLoaderProtocol {
                                 if runResult.exitStatus != .terminated(code: 0) {
                                     // TODO: should this simply be an error?
                                     // return completion(.failure(AsyncProcessResult.Error.nonZeroExit(runResult)))
-                                    evaluationResult.errorOutput = evaluationResult.compilerOutput
+                                    evaluationResult.errorOutput = (evaluationResult.compilerOutput ?? "<no compiler output>") + " running the manifest executable exited with a non-zero exit code: \(runResult.exitStatus)"
                                     return completion(.success(evaluationResult))
                                 }
 

Original file line number	Diff line number	Diff line change
`@@ -1081,7 +1081,7 @@ public final class ManifestLoader: ManifestLoaderProtocol {`
`1081`	`1081`	`if runResult.exitStatus != .terminated(code: 0) {`
`1082`	`1082`	`// TODO: should this simply be an error?`
`1083`	`1083`	`// return completion(.failure(AsyncProcessResult.Error.nonZeroExit(runResult)))`
`1084`		`- evaluationResult.errorOutput = evaluationResult.compilerOutput`
	`1084`	`+ evaluationResult.errorOutput = (evaluationResult.compilerOutput ?? "<no compiler output>") + " running the manifest executable exited with a non-zero exit code: \(runResult.exitStatus)"`
`1085`	`1085`	`return completion(.success(evaluationResult))`
`1086`	`1086`	`}`
`1087`	`1087`