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

unexpected behaviour - Installation abandoned #713

Closed
Karl-WE opened this issue Jan 23, 2021 · 5 comments · Fixed by #1427 or #1814
Closed

unexpected behaviour - Installation abandoned #713

Karl-WE opened this issue Jan 23, 2021 · 5 comments · Fixed by #1427 or #1814
Labels
Area-Manifest This may require a change to the manifest Issue-Feature This is a feature request for the Windows Package Manager client.
Milestone

Comments

@Karl-WE
Copy link
Contributor

Karl-WE commented Jan 23, 2021

Brief description of your issue

upgrading PowerShell might take longer as expected. Winget causes to terminate with "Installation abandoned"

Steps to reproduce

  1. Have a previous version of PowerShell 7 installed, e.g. 7.1.0
  2. winget install powershell to upgrade to the latest version
  3. watch the output

Expected behavior

Winget should wait for the msi processes to end and check logs for exit / error codes

Actual behavior

winget is prematurely exiting with "Installation abandoned" while the uninstall / installation is still running and was overly successful.

image

Logs

MSI (s) (54:48) [10:46:37:555]: Product: PowerShell 7-x64 -- Installation completed successfully.

MSI (s) (54:48) [10:46:37:556]: Das Produkt wurde durch Windows Installer installiert. Produktname: PowerShell 7-x64. Produktversion: 7.1.1.0. Produktsprache: 1033. Hersteller: Microsoft Corporation. Erfolg- bzw. Fehlerstatus der Installation: 0.
WinGet-2021-1-22-10-44-54.016.log
WinGet-Microsoft.PowerShell.7.1.1-2021-1-22-10-45-17.287.log

Environment

Win10 Insider Dev
Powershell 7

Windows Package Manager v0.2.3162 Vorschau
Copyright (c) Microsoft Corporation. Alle Rechte vorbehalten.

Windows: Windows.Desktop v10.0.21296.1000
Paket: Microsoft.DesktopAppInstaller v1.11.3162.0

Any other software?
nothing specific.
AV: Windows Defender
System protection turned off

@ghost ghost added the Needs-Triage Issue need to be triaged label Jan 23, 2021
@denelon denelon added Issue-Bug It either shouldn't be doing this or needs an investigation. and removed Needs-Triage Issue need to be triaged labels Jan 25, 2021
@denelon
Copy link
Contributor

denelon commented Jan 25, 2021

We see this when the shell we're executing in resets itself. We are looking at solutions.

@Karl-WE
Copy link
Contributor Author

Karl-WE commented Jan 26, 2021

Thanks for this info Demetrius!

@denelon
Copy link
Contributor

denelon commented Apr 29, 2021

We will need to add another key to manifests indicating the installation or upgrade of this application could cause a "restart" of your shell/terminal. MSIX has a delayed registration process that could also be triggered with this same key.

I'm converting this "Bug" into a "Feature" to capture this request.

@denelon denelon added Issue-Feature This is a feature request for the Windows Package Manager client. Area-Manifest This may require a change to the manifest and removed Issue-Bug It either shouldn't be doing this or needs an investigation. labels Apr 29, 2021
@denelon
Copy link
Contributor

denelon commented Jul 1, 2021

We could offer a message like:
"Installer known to restart shell or terminal. The install may succeed unless you aborted the installation."

The descriptive text for the key could be:
"This installer sends abort signal ([Ctrl]+[c]) to current terminal."

How about:

Installers:
  - Architecture: x64
    InstallerUrl: https://github.com/PowerShell/PowerShell/releases/download/v7.1.3/PowerShell-7.1.3-win-x64.msi
    InstallerType: msi
    InstallerSha256: 459642D8B6D69F643794DF8394F28191F43E5ED35472899C4F0D8424F6D1317C
    InstallerAbortsTerminal: true

@denelon denelon added this to the v.Next - Windows Package Manager milestone Jul 1, 2021
@denelon
Copy link
Contributor

denelon commented Oct 1, 2021

"InstallerAbortsTerminal": {
"type": [ "boolean", "null" ],
"description": "Indicates whether the installer will abort terminal. Default is false"
},

@denelon denelon added this to WinGet Oct 1, 2021
@denelon denelon linked a pull request Oct 1, 2021 that will close this issue
@denelon denelon modified the milestones: v.Next-Client, v1.3-Client Dec 1, 2021
@denelon denelon modified the milestones: v1.3-Client, v1.4-Client Apr 19, 2022
@ghost ghost added the In-PR Issue related to a PR label Aug 5, 2022
Repository owner moved this from In Progress to Done in WinGet Aug 12, 2022
@ghost ghost removed the In-PR Issue related to a PR label Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Manifest This may require a change to the manifest Issue-Feature This is a feature request for the Windows Package Manager client.
Projects
Archived in project
2 participants