Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linux AppImage update unable to move downloaded file #8698

Open
flexopticalHenno opened this issue Nov 20, 2024 · 1 comment · May be fixed by #8706
Open

Linux AppImage update unable to move downloaded file #8698

flexopticalHenno opened this issue Nov 20, 2024 · 1 comment · May be fixed by #8706

Comments

@flexopticalHenno
Copy link

  • Electron-Builder Version: 25.0.4 (also 25.1.7 / 25.1.8)
  • Node Version: 20.16.0 / 18.20.5
  • Electron Version: 32.1.2 / 32.2.5
  • Electron Type (current, beta, nightly): current
  • Electron-Updater Version: 6.3.3 (also 6.3.9)
  • Target: AppImage

Building the app works fine, but when the app finds an update, it downloads it to the cache folder, removes the old AppImage and then fails to move the downloaded file to the original AppImage location. The error states (translated) "mv: Executing stat not possible for '/home/username/.cache/app-name-updater/pending/APP Name.1.2.3-latest.4.AppImage': File not found". Copying the failed command into a terminal moves that file successfully.

Stack trace includes AppImageUpdater.doInstall in AppImageUpdater:88:42 and AppImageUpdater.install in BaseUpdater.js:61:25 and AppImageUpdater.quitAndInstall in BaseUpdater.js:15:34

The app name contains spaces, so the issue may be related to the fix that was introduced in 1c14820

The update was still working with electron-builder 25.0.3 and electron-updater 6.3.2. Tested on Ubuntu 24.04 and Debian 12 VMs.

@beyondkmp
Copy link
Collaborator

This should be caused by

execFileSync("mv", ["-f", options.installerPath, destination])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants