-
-
Notifications
You must be signed in to change notification settings - Fork 431
Postponed update is installed on exit #874
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
Comments
I see. This could be an issue when the user tries to open/modify the IDE when the installation is in progress. I'm not sure if/how we can leverage electron-updater to postpone the installation until the next start-up. |
I think the best fix is to always run the interactive installer. This will clearly communicate to the user what is happening and also give them the option to cancel the process. With that change, I don't think any adjustments to the dialog text are needed. I found it odd that I got a different installation behavior from clicking the Close and Install button than I did from clicking the Not now button and then closing the IDE. |
Well, actually they're not different behaviours. Once the update is downloaded, the installation will always take place when the IDE is closed. I think it's not a good idea to quit the IDE automatically immediately after the download finishes, we must warn the user before taking such an action. |
They are different behaviors. With one you get an interactive installation. With the other you get a silent installation. Close and Install
The IDE closes and you get the interactive NSIS installation for the update: Not now
The IDE is updated silently: |
Of course. I think I did not explain my proposal clearly enough. Here is the current behavior:
Here is the behavior I am proposing:
|
Oh, sorry! I didn't get it at all. So, as far as I understand, this is an issue on Windows only. @per1234 is it right? |
That is correct.
Well, the Linux behavior also does not match what was described ("the changes will be applied at the next start-up"). However, the install on close is not a problem because it is only a replacement of a single file and happens almost instantly even when I tried it on a slow old laptop. The update takes a significant amount of time on Windows and is not atomic. So this is where the silent update caught me by surprise. |
Nice one @per1234! I tried getting a grasp on how vscode does automatic updates, but it's quite complex. I'll definitely get back to it when I'll have more time for this issue. For reference: updateService.win32.ts |
Describe the problem
Arduino IDE now has an auto update capability (#797).
After downloading an update, the user is offered two options:
From #797 (comment)
🐛 In contradiction to the described behavior, after you select Not now and then close the Arduino IDE, the postponed update is installed immediately.
To reproduce
In order to force an update offer, download an outdated nightly build, using the variant that supports auto-update for your OS.
Latest outdated URLs at the time of writing are here:
These download URLs can be derived from that of the current nightly by decrementing the date component.
Install the downloaded application as usual
Start the installed Arduino IDE.
Open the "Command Palette".
(Ctrl+Shift+P or Command+Shift+P by default)
Run the "Preferences: Open Settings (UI)" command.
Change the "Arduino › Ide: Update Channel" (
arduino.ide.updateChannel
) setting to "nightly".Select File > Quit from the Arduino IDE menus.
Start the installed Arduino IDE.
Wait for the IDE to finish loading.
Click the Download button on the "Update Available" dialog.
Wait for the download to finish.
Click the Not now button on the "Arduino IDE ___ has been downloaded." dialog.
Select File > Quit from the Arduino IDE menus.
Watch the Arduino IDE installation folder.
🐛 The IDE is updated immediately after exit, not on the next start-up.
The installation takes some time and there is no feedback to the user that an operation is in progress, so they might try to open the IDE again or shut down their machine during the installation.
Expected behavior
Postponed update behavior matches described behavior: update occurs on the next start-up.
Arduino IDE version
Original report
2.0.0-rc4-snapshot-354b74c
Last verified with
9a6a457
Operating system
Operating system version
Issue checklist
The text was updated successfully, but these errors were encountered: