fix(linux): mutually exclusive exec command args (#6384)

sohobloo · web-flow · commit 5468c188f30f · 2021-11-08T19:29:58.000-08:00
* fix(linux): If linux `executableArgs` already contains one of the mutually exclusive(%f / %u / %F / %U) code，don't append `%U`. (https://specifications.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html#exec-variables)
diff --git a/.changeset/afraid-lizards-juggle.md b/.changeset/afraid-lizards-juggle.md
@@ -0,0 +1,5 @@
+---
+"app-builder-lib": patch
+---
+
+fix(linux): If linux executableArgs already contains one of the mutually exclusive(%f / %u / %F / %U) code，don't append %U.
diff --git a/packages/app-builder-lib/src/targets/LinuxTargetHelper.ts b/packages/app-builder-lib/src/targets/LinuxTargetHelper.ts
@@ -105,7 +105,11 @@ export class LinuxTargetHelper {
         exec += " "
         exec += executableArgs.join(" ")
       }
-      exec += " %U"
+      // https://specifications.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html#exec-variables
+      const execCodes = ['%f', '%u', '%F', '%U']
+      if (executableArgs == null || executableArgs.findIndex((arg) => execCodes.includes(arg)) === -1) {
+        exec += " %U"
+      }
     }
 
     const desktopMeta: any = {

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,11 @@ export class LinuxTargetHelper {`
`105`	`105`	`exec += " "`
`106`	`106`	`exec += executableArgs.join(" ")`
`107`	`107`	`}`
`108`		`- exec += " %U"`
	`108`	`+ // https://specifications.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html#exec-variables`
	`109`	`+ const execCodes = ['%f', '%u', '%F', '%U']`
	`110`	`+ if (executableArgs == null \|\| executableArgs.findIndex((arg) => execCodes.includes(arg)) === -1) {`
	`111`	`+ exec += " %U"`
	`112`	`+ }`
`109`	`113`	`}`
`110`	`114`
`111`	`115`	`const desktopMeta: any = {`