Skip to content

[vcpkg_find_acquire_program] Add tool powershell#15728

Closed
JackBoosY wants to merge 13 commits intomicrosoft:masterfrom
JackBoosY:dev/jack/15681
Closed

[vcpkg_find_acquire_program] Add tool powershell#15728
JackBoosY wants to merge 13 commits intomicrosoft:masterfrom
JackBoosY:dev/jack/15681

Conversation

@JackBoosY
Copy link
Copy Markdown
Contributor

Add powershell to vcpkg_find_acquire_program to fix the issue that couldn't find powershell correctly.

Fixes #15681.

@JackBoosY JackBoosY added info:internal category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly labels Jan 18, 2021
@JackBoosY JackBoosY added the depends:different-pr This PR or Issue depends on a PR which has been filed label Jan 19, 2021
@JackBoosY
Copy link
Copy Markdown
Contributor Author

Waiting for #15740.

@JackBoosY JackBoosY removed the depends:different-pr This PR or Issue depends on a PR which has been filed label Jan 20, 2021
@JackBoosY
Copy link
Copy Markdown
Contributor Author

@skywalkerisnull Can you please test this PR?

Thanks.

@JackBoosY JackBoosY marked this pull request as ready for review January 21, 2021 02:30
@PhoebeHui PhoebeHui marked this pull request as draft January 29, 2021 08:14
@JackBoosY JackBoosY marked this pull request as ready for review February 1, 2021 08:45
@JackBoosY
Copy link
Copy Markdown
Contributor Author

Regressions will be fixed in #15973.

@JackBoosY JackBoosY added the depends:different-pr This PR or Issue depends on a PR which has been filed label Feb 3, 2021
@JackBoosY JackBoosY removed the depends:different-pr This PR or Issue depends on a PR which has been filed label Feb 4, 2021
@JackBoosY
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JackBoosY JackBoosY added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Feb 5, 2021
@ras0219-msft
Copy link
Copy Markdown
Contributor

ras0219-msft commented Feb 11, 2021

Blocked on getting a response for #15681 (comment). Vcpkg should already be adding pwsh.exe to the path as part of the main invocation and it should not be needed to vcpkg_find_acquire_program().

@JackBoosY JackBoosY removed the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Feb 18, 2021
@PhoebeHui PhoebeHui marked this pull request as draft March 26, 2021 13:37
@PhoebeHui
Copy link
Copy Markdown
Contributor

Closing this PR since it seems that no progress is being made. Please reopen if work is still being done.

@PhoebeHui PhoebeHui closed this Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[vcpkg_copy_tool_dependencies][clang_cl_x64] Could not find PowerShell Core

4 participants