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

Bug – Scoop: Apps containing PowerShell modules get stuck when updating (when supposedly replacing the module) #207

Closed
RavenMacDaddy opened this issue Oct 6, 2022 · 7 comments
Labels
bug Something isn't working waiting-for-something This issue depends on something before we can resolve it

Comments

@RavenMacDaddy
Copy link
Contributor

RavenMacDaddy commented Oct 6, 2022

image

For reference, this is not a module that's part of Scoop; any apps containing PS modules, like scoop-completion, gets the corresponding data installed into the subdir modules – and apparently, while trying to replace the old version, WingetUI gets stuck in the process.

🔵 Current program set to 1665060355.9610548
🔵 Sending cancel signal...
🔵 Sending cancel signal...
🟢 Have permission to install, starting installation threads...
🟢 Waiting for install permission... title=scoop-completion, id=scoop-completion, installId=1665060452.285913
🟢 scoop installer assistant thread started for process <Popen: returncode: None args: 'powershell -Command scoop update scoop-compl...>
🔵 Current program set to 1665060452.285913
🔵 Sending cancel signal...
🔵 Sending cancel signal...

Hmm…no clue.

Log.txt

Even weirder, I don't see the output that WingetUI is getting stuck on when successfully installing the update through the CLI;

image

@RavenMacDaddy RavenMacDaddy changed the title Bug: Scoop modules get stuck when updating (when supposedly replacing the module) Bug: Scoop (PowerShell) modules get stuck when updating (when supposedly replacing the module) Oct 6, 2022
@RavenMacDaddy RavenMacDaddy changed the title Bug: Scoop (PowerShell) modules get stuck when updating (when supposedly replacing the module) Bug – Scoop: Apps containing PowerShell modules get stuck when updating (when supposedly replacing the module) Oct 6, 2022
@marticliment
Copy link
Owner

what happens if you run powershell -Command scoop update scoop-completion on a command prompt window?

@marticliment marticliment added bug Something isn't working needs-author-answer labels Oct 6, 2022
@RavenMacDaddy
Copy link
Contributor Author

RavenMacDaddy commented Oct 6, 2022

Here you go:

image

If CMD is being used in WingetUI, I suppose it ought to be updated to use PowerShell instead – but if I'm missing the point of its use, by all means educate me.

Both Winget and Scoop are optimized to be used with PowerShell, being the current CLI for the Windows OS – I'd guess the same applies for any other active package managers.

I also suppose that's why it just worked when executing directly through PowerShell (the most updated Core version to be specific), and it's a very possibly the reason for other apps failing to install as well.

I'm just waiting for updates to those specific apps – like Microsoft Edge – to show up again so that I can provide up-to-date information.

@ppvnf
Copy link
Contributor

ppvnf commented Oct 6, 2022

I don't know about Scoop, but Winget isn't native to PowerShell yet. See microsoft/winget-cli#221

@marticliment
Copy link
Owner

If CMD is being used in WingetUI, I suppose it ought to be updated to use PowerShell instead – but if I'm missing the point of its use, by all means educate me.

CMD is not directly used, but it is used to call a powershell instance that takes care of the scoop command. (it is python's default subprocess.Popen behaviour)

As far as I understand, the issue here is that scoop is prompting for an answer, but then (since you can't interact with the console, it gets stuck.)

I suppose this is a scoop bug, because it should take care of ran command to be unattended (as you can see, it says that the recurse parameter for whatever the function was not specified)

@marticliment
Copy link
Owner

Opened an issue on scoop, see here: ScoopInstaller/Scoop#5173

@marticliment marticliment added waiting-for-something This issue depends on something before we can resolve it and removed needs-author-answer labels Oct 6, 2022
@marticliment
Copy link
Owner

Seems like it has been already fixed
image

@marticliment
Copy link
Owner

and they'll release the next update soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working waiting-for-something This issue depends on something before we can resolve it
Projects
None yet
Development

No branches or pull requests

3 participants