fix: install LLVM dependencies after LLVM

aminya · Aug 12, 2024 · 315189b · 315189b
1 parent a654826
commit 315189b
Show file tree

Hide file tree

Showing 7 changed files with 11 additions and 9 deletions.
diff --git a/dist/actions/setup-cpp.js b/dist/actions/setup-cpp.js
diff --git a/dist/actions/setup-cpp.js.map b/dist/actions/setup-cpp.js.map
diff --git a/dist/legacy/setup-cpp.js b/dist/legacy/setup-cpp.js
diff --git a/dist/legacy/setup-cpp.js.map b/dist/legacy/setup-cpp.js.map
diff --git a/dist/modern/setup-cpp.js b/dist/modern/setup-cpp.js
diff --git a/dist/modern/setup-cpp.js.map b/dist/modern/setup-cpp.js.map
diff --git a/src/llvm/llvm.ts b/src/llvm/llvm.ts
@@ -23,13 +23,15 @@ export async function setupLLVM(version: string, setupDir: string, arch: string)
 }
 
 async function setupLLVMWithoutActivation_raw(version: string, setupDir: string, arch: string) {
-  // install LLVM and its dependencies in parallel
-  const [installationInfo, _1, _2] = await Promise.all([
+  // install LLVM
+  const [installationInfo, _1] = await Promise.all([
     setupLLVMOnly(version, setupDir, arch),
-    setupLLVMDeps(arch),
     addLLVMLoggingMatcher(),
   ])
 
+  // install LLVM dependencies
+  await setupLLVMDeps(arch)
+
   return installationInfo
 }
 const setupLLVMWithoutActivation = memoize(setupLLVMWithoutActivation_raw, { isPromise: true })