fix: make LLVM platform keywords optional

aminya · Jan 12, 2025 · d98f950 · d98f950
1 parent 95adc35
commit d98f950
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 9 deletions.
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.mjs b/dist/modern/setup-cpp.mjs
diff --git a/dist/modern/setup-cpp.mjs.map b/dist/modern/setup-cpp.mjs.map
diff --git a/src/llvm/llvm_url.ts b/src/llvm/llvm_url.ts
@@ -83,9 +83,9 @@ async function getAssetKeywords(platform: string, arch: string) {
 
   switch (platform) {
     case "win32": {
-      keywords.push("windows", "Windows")
+      optionalKeywords.push("windows", "Windows")
       if (x86_64.includes(arch)) {
-        keywords.push("win64", "x86_64", "X64")
+        optionalKeywords.push("win64", "x86_64", "X64")
         // TODO fallback to win32 if win64 is not available (e.g. for LLVM 3.6.2 and older)
       } else if (x86.includes(arch)) {
         keywords.push("win32")
@@ -98,7 +98,7 @@ async function getAssetKeywords(platform: string, arch: string) {
       break
     }
     case "linux": {
-      keywords.push("linux", "Linux")
+      optionalKeywords.push("linux", "Linux")
 
       if (isUbuntu()) {
         optionalKeywords.push("ubuntu")
@@ -116,7 +116,7 @@ async function getAssetKeywords(platform: string, arch: string) {
       }
 
       if (x86_64.includes(arch)) {
-        keywords.push("x86_64", "X64")
+        optionalKeywords.push("x86_64", "X64")
       } else if (x86.includes(arch)) {
         keywords.push("x86")
       } else if (arm64.includes(arch)) {
@@ -135,7 +135,7 @@ async function getAssetKeywords(platform: string, arch: string) {
       break
     }
     case "darwin": {
-      keywords.push("apple", "macos", "macOS")
+      optionalKeywords.push("apple", "macos", "macOS")
 
       if (x86_64.includes(arch)) {
         keywords.push("x86_64", "X64")