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

Add and use DO_NOT_REBOOT_IN_SILENT=1 parameter for MSI to not reboot during the auto-update. #4566

Merged
merged 1 commit into from
May 19, 2022

Conversation

allexzander
Copy link
Contributor

@allexzander allexzander commented May 18, 2022

This is to fix the annoying reboot with no prompt when running the installer via auto-updater. There is no way to show the popup in the passive MSI mode, so, Windows does the restart as if the user answered "Yes" to the popup. This PR removes such behavior, as a restart is not needed when updating since a shell integration is already installed and an .exe is not busy when updating to a newer version as it gets closed automatically.

Signed-off-by: alex-z blackslayer4@gmail.com

@allexzander allexzander changed the title Add and use DO_NOT_REBOOT_IN_SILENT=1 parameter for MSI to not reboot… Add and use DO_NOT_REBOOT_IN_SILENT=1 parameter for MSI to not reboot during the auto-update. May 18, 2022
… automatically when running from the updater.

Signed-off-by: alex-z <blackslayer4@gmail.com>
@allexzander allexzander force-pushed the bugfix/autoupdate-do-not-reboot branch from c33c044 to e401428 Compare May 19, 2022 09:52
@allexzander allexzander merged commit 5523330 into master May 19, 2022
@allexzander allexzander deleted the bugfix/autoupdate-do-not-reboot branch May 19, 2022 09:52
@sonarcloud
Copy link

sonarcloud bot commented May 19, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-4566-e40142847b0b66182904eea9c61c8483c6b59f96-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@mgallien mgallien added this to the 3.6.0 milestone May 25, 2022
@espressoelf
Copy link

Thanks for adding this! I expected a prompt in the end to restart now, instead it just killed everything open and restarted. I've lost quite a bit progress there. Good to know you tackled the issue already.

@allexzander
Copy link
Contributor Author

/backport to stable-3.5

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 this pull request may close these issues.

5 participants