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.

Sign up for GitHub

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

Jump to bottom

After update with WinSparkle, the KnobKraft Orm is not relaunched #146

Closed
christofmuc opened this issue Feb 17, 2022 · 6 comments
Closed

After update with WinSparkle, the KnobKraft Orm is not relaunched #146

christofmuc opened this issue Feb 17, 2022 · 6 comments
Labels
bug Something isn't working

Comments

@christofmuc
Copy link
Owner

Seems this is a feature and not a bug: vslavik/winsparkle#132
@christofmuc christofmuc added the bug Something isn't working label Feb 17, 2022
@christofmuc
Copy link
Owner Author

https://stackoverflow.com/questions/3767227/run-application-after-successful-install

@christofmuc
Copy link
Owner Author

Hm, disabling the FinishedPage in the Innosetup installer will re-launch. But it will also auto-launch on first install, which may not be a bad thing.

Have to test this with the 1.15.0.

@christofmuc christofmuc added the Ready for release Will be part of the next release label Apr 24, 2022
christofmuc added a commit that referenced this issue Apr 24, 2022
@christofmuc
[fix] Make sure the app launches again after auto-update via WinSpark…
046a2ab 
…le (#146)
@christofmuc christofmuc removed the Ready for release Will be part of the next release label Apr 24, 2022
@christofmuc
Copy link
Owner Author

That change did not work!? Maybe I need to look again at the InnoSetup docs.

@markusschloesser
Copy link
Collaborator

confirmed, does not work here either

@christofmuc christofmuc added the Ready for release Will be part of the next release label Jul 13, 2023
@christofmuc
Copy link
Owner Author

Tested, this started working again with 2.0.5

@markusschloesser
Copy link
Collaborator

confirmed, working fine here. Neat improvement!

@christofmuc christofmuc removed the Ready for release Will be part of the next release label Jul 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@christofmuc @markusschloesser