Skip to content

Commit 1ebacc3

Browse files
committed
downgrading electron-updater version to prevent breaking the Windows installer
1 parent fa34b01 commit 1ebacc3

File tree

3 files changed

+48
-14
lines changed

3 files changed

+48
-14
lines changed

electron/app/js/appUpdater.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ const i18n = require('./i18next.config');
1111
const { getLogger } = require('./wktLogging');
1212
const errorUtils = require('./errorUtils');
1313
const { sendToWindow } = require('./windowUtils');
14+
const osUtils = require('./osUtils');
1415

1516
let _isDevMode;
1617
let _downloadWindow;
@@ -34,6 +35,15 @@ function registerAutoUpdateListeners() {
3435
autoUpdater.logger.info('Download complete, install type: ' + _installType);
3536
// quit and install in this handler so MacOS updater can process the event first
3637
if(_installType === 'now') {
38+
// Working around https://github.com/electron-userland/electron-builder/issues/6418.
39+
//
40+
// This issue is fixed in [email protected] but that version suffers from
41+
// https://github.com/electron-userland/electron-builder/issues/6425, which causes
42+
// a Windows installer regression.
43+
//
44+
if (osUtils.isMac()) {
45+
autoUpdater.autoInstallOnAppQuit = false;
46+
}
3747
autoUpdater.quitAndInstall();
3848
}
3949
});

electron/package-lock.json

Lines changed: 37 additions & 13 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

electron/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@
5757
"proxyquire": "^2.1.3"
5858
},
5959
"dependencies": {
60-
"electron-updater": "^4.6.2",
60+
"electron-updater": "^4.3.9",
6161
"extract-zip": "^2.0.1",
6262
"gunzip-maybe": "^1.4.2",
6363
"https-proxy-agent": "^5.0.0",

0 commit comments

Comments
 (0)