[PackageLoading] Use temporary directory when compiling manifest

Avoid using temporary file in compiled manifest as that opens a file and
causes issues on Linux. We can use a temporary directory and let
compiler do all of the file handling.

<rdar://problem/59472703>

(cherry picked from commit 2500ac0)

Author: aciidgh

Sources/PackageLoading/ManifestLoader.swift

@@ -533,11 +533,10 @@ public final class ManifestLoader: ManifestLoaderProtocol {
 
             cmd += [manifestPath.pathString]
 
-            try withTemporaryFile { file in
+            try withTemporaryDirectory(removeTreeOnDeinit: true) { tmpDir in
                 // Set path to compiled manifest executable.
-                cmd += ["-o", file.path.pathString]
-
-                try Process.popen(arguments: cmd)
+                let file = tmpDir.appending(components: "\(packageIdentity)-manifest")
+                cmd += ["-o", file.pathString]
 
                 // Compile the manifest.
                 let compilerResult = try Process.popen(arguments: cmd)
@@ -550,7 +549,7 @@ public final class ManifestLoader: ManifestLoaderProtocol {
                 }
 
                 // Pass the fd in arguments.
-                cmd = [file.path.pathString, "-fileno", "1"]
+                cmd = [file.pathString, "-fileno", "1"]
 
               #if os(macOS)
                 // If enabled, use sandbox-exec on macOS. This provides some safety against