fix: add missing %U in desktop file of appimages (#4909)

* fix missing %U in desktop file of appimages this should fix #4035 * fix test
electron-userland · Sep 22, 2020 · d96b3d3 · d96b3d3
1 parent b8c9cd0
commit d96b3d3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/app-builder-lib/src/targets/AppImageTarget.ts b/packages/app-builder-lib/src/targets/AppImageTarget.ts
@@ -19,7 +19,7 @@ export default class AppImageTarget extends Target {
   constructor(ignored: string, private readonly packager: LinuxPackager, private readonly helper: LinuxTargetHelper, readonly outDir: string) {
     super("appImage")
 
-    this.desktopEntry = new Lazy<string>(() => helper.computeDesktopEntry(this.options, "AppRun", {
+    this.desktopEntry = new Lazy<string>(() => helper.computeDesktopEntry(this.options, "AppRun %U", {
       "X-AppImage-Version": `${packager.appInfo.buildVersion}`,
     }))
   }
@@ -90,4 +90,4 @@ export default class AppImageTarget extends Target {
       updateInfo: await executeAppBuilderAsJson(args),
     })
   }
-}
+}
diff --git a/test/snapshots/linux/linuxPackagerTest.js.snap b/test/snapshots/linux/linuxPackagerTest.js.snap
@@ -3,7 +3,7 @@
 exports[`AppImage - default icon, custom executable and custom desktop 1`] = `
 "[Desktop Entry]
 Name=Test App ßW
-Exec=AppRun
+Exec=AppRun %U
 Terminal=true
 Type=Application
 Icon=Foo