fix: add timeout for the LLVM installer call

Author: aminya

action.yml

@@ -86,7 +86,7 @@ inputs:
     required: false
   timeout:
     description: "The timeout for installation of one tool (in minutes)."
-    default: "10"
+    default: "20"
     required: false
 
 runs:

dist/actions/setup-cpp.js

@@ -5,6 +5,7 @@ import { hasNala, isPackageInstalled, setupAptPack } from "../utils/setup/setupA
 import { InstallationInfo } from "../utils/setup/setupBin"
 import { promises } from "fs"
 import { info } from "console"
+import { DEFAULT_TIMEOUT } from "../tool"
 const { readFile, writeFile, chmod } = promises
 
 export async function setupLLVMApt(majorVersion: number): Promise<InstallationInfo> {
@@ -19,6 +20,7 @@ export async function setupLLVMApt(majorVersion: number): Promise<InstallationIn
   await execRoot("bash", ["/tmp/llvm-setup-cpp.sh", `${majorVersion}`, "all"], {
     stdio: "inherit",
     shell: true,
+    timeout: DEFAULT_TIMEOUT,
   })
 
   await addPath(`${installationFolder}/bin`)

dist/actions/setup-cpp.js.map

@@ -96,7 +96,7 @@ async function main(args: string[]): Promise<number> {
         setupCppDir,
         successMessages,
         errorMessages,
-        parseFloat(opts.timeout ?? "10"),
+        parseFloat(opts.timeout ?? "20") * 60 * 1000,
       )
       time2 = Date.now()
       info(`took ${timeFormatter.format(time1, time2) || "0 seconds"}`)

dist/legacy/setup-cpp.js

@@ -69,6 +69,8 @@ export type ToolName = keyof typeof setups
 /** The tools that can be installed */
 export const tools = Object.keys(setups) as Array<ToolName>
 
+export const DEFAULT_TIMEOUT = 20 * 60 * 1000 // 20 minutes
+
 export async function installTool(
   tool: ToolName,
   version: string,
@@ -77,7 +79,7 @@ export async function installTool(
   setupCppDir: string,
   successMessages: string[],
   errorMessages: string[],
-  timeout: number = 0.1,
+  timeout: number = DEFAULT_TIMEOUT,
 ) {
   startGroup(`Installing ${tool} ${version}`)
   let hasLLVM = false