mikecole / chocolatey-packages

A template to use when setting up to create packages both manual and automatic
Apache License 2.0
8 stars 21 forks source link

Kinda of a question, bug, or maybe even a feature request regarding plexmediaserver #159

Open brian6932 opened 2 years ago

brian6932 commented 2 years ago

So for some reason this package force restarts my PC whenever I install it or update it, this person in the Disquis comments of the package has the issue as well: image_32 Either disabling this as default, or adding an argument to disable force restarts (if this is the intended behaviour), would be really nice as this package is basically unusable for me due to this issue.

mikecole commented 2 years ago

I looked at the meager documentation for the installer, and I see nothing that mentions restarts, or avoiding restarts. I do not believe this is a chocolatey issue because I've never seen choco force a reboot. It usually just reports that a reboot is required. I'm kind of at a loss as to how to troubleshoot this.

Is this happening every time, or is it something you can easily recreate? If so perhaps we could try updating right on the command line using the same command choco does, but removing choco from the rebooting equation.

brian6932 commented 2 years ago

I looked at the meager documentation for the installer, and I see nothing that mentions restarts, or avoiding restarts.

Can confirm, doesn't occur with the standard updater

Is this happening every time, or is it something you can easily recreate?

Yes, and yes.

If there are any log files you need, or maybe a video of it just occuring lmk, idk if I can create a screen recording, since it prompts a restart, but I can try.

mikecole commented 2 years ago

Very strange. Can you attach the relevant portion of your chocolatey.log file? Mine is in C:\ProgramData\chocolatey\logs.

On Tue, Oct 19, 2021 at 5:02 PM Brian @.***> wrote:

I looked at the meager documentation for the installer, and I see nothing that mentions restarts, or avoiding restarts.

Can confirm, doesn't occur with the standard updater

Is this happening every time, or is it something you can easily recreate?

Yes, and yes.

If there are any log files you need, or maybe a video of it justoccuring lmk, idk if I can create a screen recording, since it prompts a restart, but I can try.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/mikecole/chocolatey-packages/issues/159#issuecomment-947139402, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI4X2YK6BYOGKACDGQBXFDUHXTIJANCNFSM5GKEPAOA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

brian6932 commented 2 years ago

@mikecole chocolatey.log

2021-10-19 17:02:56,286 2480 [DEBUG] - Calling command ['"C:\WINDOWS\System32\shutdown.exe" /a']
2021-10-19 17:02:56,320 2480 [DEBUG] - Command ['"C:\WINDOWS\System32\shutdown.exe" /a'] exited with '1116'

^This seems to be the part where it actually calls it (line 446-447)

mikecole commented 2 years ago

I just submitted an update to choco that properly handles the 3010 return code which the installer appears to return. I'm not sure why it's forcing a reboot, as I don't think choco is even able to force a reboot. It should be available later tonight: 1.24.4.508101

On Tue, Oct 19, 2021 at 5:53 PM Brian @.***> wrote:

@mikecole https://github.com/mikecole chocolatey.log https://github.com/mikecole/chocolatey-packages/files/7377024/chocolatey.log

2021-10-19 17:02:56,286 2480 [DEBUG] - Calling command ['"C:\WINDOWS\System32\shutdown.exe" /a'] 2021-10-19 17:02:56,320 2480 [DEBUG] - Command ['"C:\WINDOWS\System32\shutdown.exe" /a'] exited with '1116'

^This seems to be the part where it actually calls it (line 446-447)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mikecole/chocolatey-packages/issues/159#issuecomment-947164142, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI4X26WHKR25M2D47GYWLDUHXZFNANCNFSM5GKEPAOA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

brian6932 commented 2 years ago

@mikecole Can confirm, fixed in 1.24.4.508101

mikecole commented 2 years ago

Heyyy, great news! Happy that worked because I had no clue what to try next.

brian6932 commented 2 years ago

Ok, so I encountered this issue again, and realized I was wrong with what I pointed out, it seems like the issue is with exit code 3010, and after a little bit of looking it seems like this exit code is responsible for forced shutdowns.

2021-12-01 22:00:58,066 15204 [DEBUG] - Command ["C:\Users\username\AppData\Local\Temp\chocolatey\plexmediaserver\1.25.0.5282\PlexMediaServer-1.25.0.5282-2edd3c44d-x86.exe" /quiet ] exited with '3010'.
2021-12-01 22:00:58,086 15204 [ERROR] - ERROR: Running ["C:\Users\username\AppData\Local\Temp\chocolatey\plexmediaserver\1.25.0.5282\PlexMediaServer-1.25.0.5282-2edd3c44d-x86.exe" /quiet ] was not successful. Exit code was '3010'. See log for possible error messages.
2021-12-01 22:00:58,087 15204 [DEBUG] - Built-in PowerShell host called with ['[System.Threading.Thread]::CurrentThread.CurrentCulture = '';[System.Threading.Thread]::CurrentThread.CurrentUICulture = ''; & import-module -name 'C:\ProgramData\chocolatey\helpers\chocolateyInstaller.psm1'; & 'C:\ProgramData\chocolatey\helpers\chocolateyScriptRunner.ps1' -packageScript 'C:\ProgramData\chocolatey\lib\plexmediaserver\tools\chocolateyInstall.ps1' -installArguments '' -packageParameters '''] exited with '3010'.

new log files: chocolatey.log