-
Notifications
You must be signed in to change notification settings - Fork 288
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
Don't inherit the same handle more than once. #1234
Don't inherit the same handle more than once. #1234
Conversation
Attempts to resolve microsoft/vcpkg#33904
I'm also up for ideas on how to reasonably test this. :( @ras0219-msft suggested trying to run git bash from our e2e tests but I really don't want to open the can of worms of trying to find where that is and/or whether it exists in the first place. (Does the git that comes with Visual Studio even have a git bash? I don't think so ....) |
Github actions images have git bash: https://github.com/actions/runner-images/blob/main/images/win/Windows2022-Readme.md#shells |
Works for me. Could we teach the vcpkg-tool CI to store the build binaries for PRs for like a week? Took me like 10 min for setup. Simply downloading it would have been faster ;) |
I think if we do that we'll be obligated to sign them which is ... a mess |
It simply expects to be executed in a git bash which does auto conversion of paths. This is with a still broken vcpkg (so not this PR):
I don't see it finding the git tar here although --debug-env shows:
|
Attempts to resolve microsoft/vcpkg#33904
@Neumann-A can you check that this fixes it for you?