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: Machine restart required to completely uninstall app #71

Closed
eomere opened this issue Jan 17, 2020 · 7 comments
Closed

Windows: Machine restart required to completely uninstall app #71

eomere opened this issue Jan 17, 2020 · 7 comments

Comments

@eomere
Copy link

eomere commented Jan 17, 2020

Issue overview

The windows version of the app does not completely uninstall when prompted.

Current Behavior

The directory containing all originally downloaded information does not get deleted. This is because it holds a bin file that won't delete. Upon restarting the computer, both the directory and bin file were deleted.

Expected Behavior

When uninstalled, all items associated with the app should be deleted.

Steps to Reproduce

Install the app, then uninstall it.

@eomere eomere added the Triage Issue needs to be assessed and labeled, further information on reported might be needed label Jan 17, 2020
@jmarrec
Copy link
Collaborator

jmarrec commented Jan 23, 2020

Which bin file won't delete?

@eomere
Copy link
Author

eomere commented Jan 24, 2020 via email

@jmarrec
Copy link
Collaborator

jmarrec commented Jan 30, 2020

I just don't really use windows unless I really have to. Anyways, I tried it out today:

Try 1:

  1. Install OS App
  2. Uninstall OS App. works fine

Try 2:

  1. Install OS App
  2. Launch OS App.
  3. Close OS App. Seems like I have a hanging "openstudio.exe" process
  4. Uninstall OS App. uninstaller says it's fine. But I still have C:\openstudioapplication-1.0.0-pre1\bin\ (the "bin" folder) on my disk, and that folder is plain empty. I still see openstudio.exe as Running in the Task Manager. I also see QtWebEngineProcess.exe (3 of them) which could be related.

@jmarrec jmarrec added component - Installer/Platform 📦 severity - Minor Bug 🐤 and removed Triage Issue needs to be assessed and labeled, further information on reported might be needed labels Jan 30, 2020
@jmarrec
Copy link
Collaborator

jmarrec commented Jan 30, 2020

Ok, so I have a lead:

If I open OS App, and wait a few seconds before I close it, then OpenStudioApp.exe and openstudio.exe are both killed.
If I open OS App and close it right after, openstudio.exe gets orphaned.

So it looks like in a way if I don't wait out for the measure manager to start properly, then closing the OS App doesn't make the cli stop.

The measure manager is started in a non-blocking way, and we use a future to see when it's started and then use connectOSDocumentSignals, so perhaps that makes some sense.

https://github.com/NREL/OpenStudioApplication/blob/aaff5fd8a72ef81af90c4226b551d12a2d4c7f0e/src/openstudio_app/OpenStudioApp.cpp#L216-L225


I can also confirm that when I log into my session, I don't have any "QtWebEngineProcess.exe", then after launching OS App the first time I do have 3. Closing OS app leaves 3 of them running. Opening two instances of OS App doesn't change the fact that 3 are running.


@jmarrec
Copy link
Collaborator

jmarrec commented Jan 30, 2020

I'm probably not the right person to track down this problem, but I'll label the issue accordingly.

@macumber
Copy link
Collaborator

It's possible that orphan CLI processes were holding this folder locked. When this happens next can you use unlocker to see which process it is? If it is openstudio then there is a good chance that will be fixed in #369

@jmarrec
Copy link
Collaborator

jmarrec commented Jun 1, 2021

Assuming it's fixed by #369, and if not need more info to reproduce. Happy to reopen if bug is confirmed with 1.2.0 (or 1.2.0-rc6) and more info is provided

@jmarrec jmarrec closed this as completed Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants