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

Windows NSIS One-Click Setup Launched while App already running in background #617

Closed
dharders opened this issue Jul 29, 2016 · 4 comments · May be fixed by qcif/data-curator#563
Closed

Comments

@dharders
Copy link

  • Version: 5.19.0
  • Target: win32 NSIS One-click

If the installer (AppName Setup x.x.x.exe) is launched while an application instance of AppName.exe is already running, a dialog appears stating "AppName is running. Click OK to close it and continue with install". This is good and expected behaviour.

If you click OK, then the app instance is automatically closed and setup continues. Good again. However, if the user clicks "cancel" or closes the dialog then the user is left with a hung install progress bar that is never going to move.

Expected Behaviour:

Clicking cancel (or closing the dialog) should automatically close the Setup instance also.

P.S. Keep up the great work!

@dharders
Copy link
Author

dharders commented Aug 1, 2016

Found a simple fix!

templates/nsis/allowOnlyOneInstallerInstace.nsh#L29

Change: Abort

To: Quit

cc @develar

@develar develar closed this as completed in 918a6c0 Aug 1, 2016
@justijndepover
Copy link

The link you provided is not working anymore. Can you show us a working fix?

@dharders
Copy link
Author

@justijndepover here's an updated working link

But really you shouldn't need it as it was already fixed. If you're still experiencing this in the latest version maybe create a new issue ?

@justijndepover
Copy link

The problem occured on an old project of mine. The error occures when someone tries to install my application on a new device. So there isn't an instance running in the background. Can you verify that this is the same problem?

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