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

[11/11/22]: [Register with windows restart manager] #327

Open
DevRCRun opened this issue Nov 11, 2022 · 1 comment
Open

[11/11/22]: [Register with windows restart manager] #327

DevRCRun opened this issue Nov 11, 2022 · 1 comment

Comments

@DevRCRun
Copy link

Is your feature request related to a problem? Please describe.
The nodejs installer on windows appears to be inconsistent version to version in whether it will proceed while node-windows originated services are running. On some versions it goes through fine, and the services (or machine) can then be restarted to pick up the new version, on others it errors out with "RESTART MANAGER: Failed to shut down all applications in the service's session"

Describe the solution you'd like
My supposition is this is occurring because nodejs is in use / the services are not registered with windows restart manager https://learn.microsoft.com/en-us/windows/win32/rstmgr/about-restart-manager https://www.advancedinstaller.com/user-guide/qa-vista-restart-manager.html (I could be completely wrong about this and something else is at fault).

Describe alternatives you've considered
Some hideous powershell scripted solution.

Additional context
It would be useful to know if any other node-windows users experience the same problem (if not I'm likely way off track with what the issue actually is). I'm aware that the issue, if indeed that's what it is, may need to actually be addressed in winsw, but I'm aware that you're thinking of switching away from winsw, so thought it worth posting here. That's also why I've gone for a feature request. If this is invalid, happy to close as such!

Thanks

Please use the reactions to cast a vote in favor of or against this feature suggestion »
@coreybutler
Copy link
Owner

Marking as an enhancement request so I can address it (or at least test it) in the next version.

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

2 participants