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

[vcpkg] Resolve --overlay-ports is only working for relative parths since fix… #11302

Merged
merged 2 commits into from
May 11, 2020
Merged

Conversation

TobiasFunk
Copy link
Contributor

… for

Fixes #11301

Describe the pull request
Since the fix #10771 the --overlay-ports parameter does not work with absolute paths any more.

@PhoebeHui PhoebeHui changed the title Resolve --overlay-ports is only working for relative parths since fix… [vcpkg] Resolve --overlay-ports is only working for relative parths since fix… May 11, 2020
@PhoebeHui PhoebeHui self-assigned this May 11, 2020
@PhoebeHui
Copy link
Contributor

I tried the fix, it works well, thank you for the PR!

@PhoebeHui PhoebeHui added the info:reviewed Pull Request changes follow basic guidelines label May 11, 2020
@TobiasFunk TobiasFunk closed this May 11, 2020
@TobiasFunk TobiasFunk reopened this May 11, 2020
@TobiasFunk TobiasFunk closed this May 11, 2020
@TobiasFunk TobiasFunk reopened this May 11, 2020
@PhoebeHui
Copy link
Contributor

The osx failure doesn't relate to this change. please ignore it.

@TobiasFunk
Copy link
Contributor Author

@PhoebeHui when will it be possible to merge this change? since saturday no builds in our company are working because of this bug

@strega-nil
Copy link
Contributor

Cool, thanks @TobiasFunk :)

@strega-nil strega-nil merged commit 06c647e into microsoft:master May 11, 2020
strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
…ince fix… (microsoft#11302)

* Resolve --overlay-ports is only working for relative parths since fix for microsoft#10771
Fixes microsoft#11301

* use auto instead of fs::path
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
3 participants