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

Rebooted without warning #1671

Closed
Y-Less opened this issue Nov 1, 2018 · 3 comments
Closed

Rebooted without warning #1671

Y-Less opened this issue Nov 1, 2018 · 3 comments

Comments

@Y-Less
Copy link

Y-Less commented Nov 1, 2018

Ensure you have read over Submitting Issues -
https://github.com/chocolatey/choco#submitting-issues

Please check to see if your issue already exists with a quick search of the issues. Start with one relevant term and then add if you get too many results.

NOTE: Keep in mind we have an etiquette regarding communication that we expect folks to observe when they are looking for support in the Chocolatey community. https://github.com/chocolatey/choco/blob/master/README.md#etiquette-regarding-communication
-->

What You Are Seeing?

I was installing a package (node) while doing some other work (a calculation for which had been running for several hours). Suddenly my machine rebooted without warning.

What is Expected?

Not reboot without warning!

How Did You Get This To Happen? (Steps to Reproduce)

Output Log

Full Log Output

PLACE LOG CONTENT HERE - WE NEED _ALL_ DETAILED OUTPUT BASED ON THE ABOVE TO BE ABLE TO PROVIDE SUPPORT (YOU WILL FIND THAT IN THE $env:ChocolateyInstall\logs\chocolatey.log between the `=====`)

@gep13
Copy link
Member

gep13 commented Nov 2, 2018

@Y-Less thank you for taking the time to raise this issue, and I am sorry to hear that you have had problems.

The issue is actually outwith the confines of this repository. This is actually being discussed, along with other issues regarding the changes to the Node installer here:

nodejs/node#23838

There is also a Pull Request to add additional warnings about what will happen during the installation here:

nodejs/node#23987

@gep13 gep13 closed this as completed Nov 2, 2018
@Y-Less
Copy link
Author

Y-Less commented Nov 2, 2018

This should be restricted at the package manager level, such that nothing installed via Chocolatey even has the ability to disrupt people in this way. It isn't an issue with just one repo, even though that is the one I experienced it on.

@gep13
Copy link
Member

gep13 commented Nov 2, 2018

@Y-Less at the package manager level, i.e. Chocolatey, Chocolatey doesn't reboot the machine. It may provide a warning that a reboot will be necessary for a package that was just installed to work correctly, but Chocolatey won't reboot your machine.

However, in this case, the NodeJS developers have included the usage of Boxstarter, which is written on top of Chocolatey to provide the ability to automatically reboot your machine when required. However, even in this scenario, Boxstarter can be instructed to NOT reboot your machine.

So again, this isn't the right place to raise this issue. The issue needs to be raised, or perhaps further discussed, on the NodeJS repository, in how they have chosen to use these tools.

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