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 PowerShell Module Dependency Docs #2797

Merged
merged 3 commits into from
Dec 22, 2022

Conversation

ryfu-msft
Copy link
Contributor

@ryfu-msft ryfu-msft commented Dec 22, 2022

Resolves #2795

Changes:
Updates the PowerShell module documentation to explicitly identify the required dependency on the Microsoft Visual C++ Redistributable package in order to run the cmdlets

Microsoft Reviewers: Open in CodeFlow

@ryfu-msft ryfu-msft requested a review from a team as a code owner December 22, 2022 20:59
@ghost ghost added the Issue-Bug It either shouldn't be doing this or needs an investigation. label Dec 22, 2022
@@ -6,6 +6,15 @@ This project contains the source code for building the Windows Package Manager P

After building the project solution, the `Microsoft.WinGet.Client` PowerShell module can be found in the output directory in the `PowerShell` folder. For example if you built the project as x64 release, you should expect to find the module files in `$(SolutionDirectory)/src/x64/Release/PowerShell`.

## Prerequisites

The PowerShell module has a dependency on the Microsoft Visual C++ Redistributable package which can be downloaded [here](https://learn.microsoft.com/en-US/cpp/windows/latest-supported-vc-redist?view=msvc-170) or installed through WinGet by running the following command:
Copy link
Member

Choose a reason for hiding this comment

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

What version do I need to download?

Copy link

@robinmalik robinmalik Dec 22, 2022

Choose a reason for hiding this comment

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

@florelis I've got the Microsoft Visual C++ 2015-2022 Redistributable, v14.34.31931 on my W11 machine, both x86 and x64, and it works for me (well, it imports and Get-WinGetPackage runs). I suspect you can just go with x64 though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added the word "latest" and the id in the winget example points to 2015.

Copy link
Member

Choose a reason for hiding this comment

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

Thanks @robinmalik. I actually meant it as a rhetorical question for @ryfu-msft to specify it in the doc, which he did

@ryfu-msft ryfu-msft merged commit b906673 into microsoft:master Dec 22, 2022
@ryfu-msft ryfu-msft deleted the updatePowerShellDocs branch December 22, 2022 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Bug It either shouldn't be doing this or needs an investigation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The type initializer for 'Microsoft.WinGet.Client.Common.BaseClientCommand' threw an exception.
4 participants