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

Fail with error if user attempts to uninstall NVDA while it is running #2890

Open
nvaccessAuto opened this issue Dec 22, 2012 · 10 comments
Open
Labels
bug component/installer good first issue github features these at https://github.com/nvaccess/nvda/contribute z goodForNewDev (archived)

Comments

@nvaccessAuto
Copy link

Reported by elliott94 on 2012-12-22 23:40
Currently, if eSpeak is set as the default synth during an uninstall, it stops producing output after the process has started, i.e. the Uninstall button is pressed. However, a dialog is presented at the end of the uninstall which prompts the user as to whether they want to reboot now or later. However, this prompt doesn't get any feedback. This doesn't occur if another synth is selected.

@nvaccessAuto
Copy link
Author

Comment 1 by elliott94 on 2012-12-22 23:40
Changes:
Changed title from "Allow a synthesizer to be used for the entirety of uninstallation" to "Allow eSpeak to be used for the entirety of uninstallation"

@nvaccessAuto
Copy link
Author

Comment 2 by mdcurran on 2012-12-23 04:48
I fear that other features of NVDA would not work while on that final dialog. The reason espeak stops is because many of its files have already been deleted, though it seems eSpeak is not caching or locking those files.
I'm more likely to accept that we simply throw up an error at the start, or kill NVDA off, if its running when trying to uninstall.

@nvaccessAuto
Copy link
Author

Comment 3 by jteh on 2012-12-26 03:09
NVDA can't be expected to run during uninstallation, since it's being uninstalled. I'd probably vote for presenting an error, as users might still expect it to work somehow. Note that we can't present the error from NSIS due to localisation issues, so it'll probably have to be slave.

@bhavyashah
Copy link

@jcsteh Could you please modify the ticket title accordingly to reflect our mostly agreed decision that a warning/error should be presented to mitigate any potential confusion as noted in yours and Mick's comments?

@jcsteh jcsteh changed the title Allow eSpeak to be used for the entirety of uninstallation Fail with error if user attempts to uninstall NVDA while it is running Aug 15, 2017
@Adriani90
Copy link
Collaborator

@elliott94 can you still reproduce this with NVDA 2018.4 Beta 1?

@jcsteh
Copy link
Contributor

jcsteh commented Nov 25, 2018

Even if the original issue can't be reproduced (and I suspect it can), the idea here is still valid; i.e. fail with an error if the user attempts to uninstall while NVDA is running.See #2890 (comment) and #2890 (comment).

@LeonarddeR
Copy link
Collaborator

It should be not too difficult to do this, see https://nsis.sourceforge.io/Check_whether_your_application_is_running
I'd go for the mutex approach. SInce NSIS isn't very hard to dive into and the reference is pretty neat, marking this as good for new dev.

@seanbudd seanbudd added the good first issue github features these at https://github.com/nvaccess/nvda/contribute label Apr 30, 2021
@Adriani90
Copy link
Collaborator

cc: @seanbudd

@seanbudd
Copy link
Member

@Adriani90 could you please explain the tag?

@Adriani90
Copy link
Collaborator

Ah sorry, I just thought you might want to give it a priority. It still affects users, especially when the dialog after uninstall is not spoken at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component/installer good first issue github features these at https://github.com/nvaccess/nvda/contribute z goodForNewDev (archived)
Projects
None yet
Development

No branches or pull requests

6 participants