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

Update VMware.WorkstationPro to 17.6.1 #190690

Closed
wants to merge 2 commits into from

Conversation

FaserF
Copy link
Contributor

@FaserF FaserF commented Nov 12, 2024

Checklist for Pull Requests

Manifests

  • Have you checked that there aren't other open pull requests for the same manifest update/change?
  • This PR only modifies one (1) manifest
  • Have you validated your manifest locally with winget validate --manifest <path>?
  • Have you tested your manifest locally with winget install --manifest <path>?
  • Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.


Microsoft Reviewers: Open in CodeFlow

@wingetbot
Copy link
Collaborator

Service Badge  Service Badge  

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@wingetbot wingetbot added the URL-Validation-Error Microsoft Defender SmartScreen triggered during automatic validation label Nov 12, 2024
@wingetbot
Copy link
Collaborator

Url validation error

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Nov 12, 2024
@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Needs-Author-Feedback This needs a response from the author. URL-Validation-Error Microsoft Defender SmartScreen triggered during automatic validation labels Nov 12, 2024
@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Nov 12, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Nov 12, 2024
@stephengillie
Copy link
Collaborator

Automatic Validation ended with:

Installation failed with exit code -1978335140
2024-11-11 23:56:25.339 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\Archive.cpp(33)\WindowsPackageManager.dll!00007FFBDE6A413B: (caller: 00007FFBDE61EBB5) ReturnHr(1) tid(590) 80004005 Unspecified error
2024-11-11 23:56:25.340 [CLI ] Failed to extract archive with code -2147467259
MSI (s) (30:D0) [23:55:51:481]: PROPERTY CHANGE: Adding OutOfDiskSpace property. Its value is '0'.
MSI (s) (30:D0) [23:55:51:481]: PROPERTY CHANGE: Adding OutOfNoRbDiskSpace property. Its value is '0'.
MSI (s) (30:D0) [23:55:51:481]: PROPERTY CHANGE: Adding PrimaryVolumeSpaceAvailable property. Its value is '0'.
MSI (s) (30:D0) [23:55:51:481]: PROPERTY CHANGE: Adding PrimaryVolumeSpaceRequired property. Its value is '0'.
MSI (s) (30:D0) [23:55:51:481]: PROPERTY CHANGE: Adding PrimaryVolumeSpaceRemaining property. Its value is '0'.
Property(S): OutOfDiskSpace = 0
Property(S): OutOfNoRbDiskSpace = 0
Property(S): PrimaryVolumeSpaceAvailable = 0
Property(S): PrimaryVolumeSpaceRequired = 0
Property(S): PrimaryVolumeSpaceRemaining = 0
MSI (s) (30:80) [23:55:52:231]: PROPERTY CHANGE: Adding OutOfDiskSpace property. Its value is '0'.
MSI (s) (30:80) [23:55:52:231]: PROPERTY CHANGE: Adding OutOfNoRbDiskSpace property. Its value is '0'.
MSI (s) (30:80) [23:55:52:231]: PROPERTY CHANGE: Adding PrimaryVolumeSpaceAvailable property. Its value is '0'.
MSI (s) (30:80) [23:55:52:231]: PROPERTY CHANGE: Adding PrimaryVolumeSpaceRequired property. Its value is '0'.
MSI (s) (30:80) [23:55:52:231]: PROPERTY CHANGE: Adding PrimaryVolumeSpaceRemaining property. Its value is '0'.

Automated error analysis suggests -1978335140 may mean Failed to extract archive. Does the InstallerUrl actually end in .zip - or does it have a different extension?

Automated error analysis suggests 80004005 may mean Resource missing

(Automated response - build 897.)

@FaserF
Copy link
Contributor Author

FaserF commented Nov 14, 2024

.tar extractions should be possible with winget 1.9.25180 due to #4541

I have not found any docs, describing on specifically setting anything within the installer.yaml to use tar.exe - so for me I thought it should work that way?
Are the tests running with an older winget version?

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Nov 14, 2024
@ItzLevvie
Copy link
Contributor

@wingetbot run

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Needs-Attention This work item needs to be reviewed by a member of the core team. Validation-Installation-Error Error installing application labels Nov 29, 2024
@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Nov 29, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Nov 29, 2024
@FaserF
Copy link
Contributor Author

FaserF commented Nov 29, 2024

Same thing as mentioned here: #190690 (comment)

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Dec 6, 2024
@ItzLevvie
Copy link
Contributor

@wingetbot run

@wingetbot
Copy link
Collaborator

/AzurePipelines run

@microsoft-github-policy-service microsoft-github-policy-service bot removed Needs-Author-Feedback This needs a response from the author. Validation-Completed Validation passed Validation-Installation-Error Error installing application labels Dec 8, 2024
@wingetbot wingetbot added the Validation-Installation-Error Error installing application label Dec 9, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback This needs a response from the author. label Dec 9, 2024
@FaserF
Copy link
Contributor Author

FaserF commented Dec 9, 2024

Same thing as mentioned here: #190690 (comment)

Just writing this to remove the "needs author feedback"

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Dec 9, 2024
@stephengillie
Copy link
Collaborator

Manual Validation ended with:

1:13:16 PM:
2024-12-13 13:13:16.265 [CLI ] Failed to extract archive with code -2147467259 1:13:16 PM: [237C:023C][
2024-12-13T13:12:46]w363: Could not create system restore point, error: 0x80070422. Continuing... 1:13:16 PM: MSI (s) (64:78) [13:12:48:313]: Windows Installer installed the product. Product Name: Microsoft Visual C++ 2022 X64 Minimum Runtime - 14.42.34433. Product Version: 14.42.34433. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 0.

(Automated response - build 904.)

@denelon
Copy link
Contributor

denelon commented Dec 18, 2024

WinGet doesn't support compression formats other than .zip. It's a specific setting for users in system context to leverage tar.exe to decompress .zip files as opposed to the shell API for decompression in the referenced PR:

@denelon
Copy link
Contributor

denelon commented Dec 18, 2024

Supporting .tar would most likely be part of:

@denelon
Copy link
Contributor

denelon commented Dec 18, 2024

Is there an uncompressed version available or something that isn't .tar?

@denelon denelon added Needs-Author-Feedback This needs a response from the author. and removed Needs-Attention This work item needs to be reviewed by a member of the core team. labels Dec 18, 2024
@FaserF
Copy link
Contributor Author

FaserF commented Dec 19, 2024

Is there an uncompressed version available or something that isn't .tar?

Only behind a sign in wall from what I have found :(

But what about this mergered PR? microsoft/winget-cli#4541 With this from what I read extracting tar files should be available and supported since 1.9.25180

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention This work item needs to be reviewed by a member of the core team. and removed Needs-Author-Feedback This needs a response from the author. labels Dec 19, 2024
@denelon
Copy link
Contributor

denelon commented Dec 19, 2024

That PR only switches to use tar.exe to decompress .zip files when the shell APIs aren't available. There is more work needed to support those formats through the normal code flow. We've had work ongoing towards other compression/decompression libraries for multiple formats, but that's been gated by some challenges related to making the feature work on Windows 10.

@denelon
Copy link
Contributor

denelon commented Dec 19, 2024

There would also be some similar additional work necessary in wingetcreate to support the other compression formats as well.

@FaserF
Copy link
Contributor Author

FaserF commented Dec 19, 2024

Thanks for the clarification, then I am going to close this PR for now.

@FaserF FaserF closed this Dec 19, 2024
auto-merge was automatically disabled December 19, 2024 16:40

Pull request was closed

@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Attention This work item needs to be reviewed by a member of the core team. label Dec 19, 2024
Copy link
Contributor

@rovabunw rovabunw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these are the correct urls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Validation-Installation-Error Error installing application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants