-
Notifications
You must be signed in to change notification settings - Fork 80
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 packages in vcpkg cache #702
Conversation
I RDP'd into an AppVeyor instance that was failing to build zlib, but I couldn't figure out what exactly the problem was. So, I opened microsoft/vcpkg#13105. Maybe one of their pkgconfig people can figure it out. |
The vcpkg cache contains old, stale versions of several packages, which do not get updated automatically by vcpkg install. This ensures they get updated before installing new packages.
This works around issues with vcpkg's implementation of paths, resulting in failures in vcpkg_fixup_pkgconfig when installing zlib.
Now we are trying to link the release client against debug libopus 😞 |
This is needed because CMake is picking the debug opus library instead of the release library, causing linking errors when it tries to use debug CRT symbols in release builds.
Ugh, MSYS2 broke between the first and second build configuration. sigh |
This reverts commit 732ad9f, reversing changes made to fde27d9. The bug fixed by H-uru#709 was fixed by microsoft/vcpkg#13022. The upstream fix renamed the target `expat::libexpat` to `expat::expat` but also fixed the expat finder.
@zrax I've fixed all the weird failures -- are these changes OK? |
(Moved from #701 due to being in the wrong fork and thus duplicating builds)
The vcpkg cache contains old, stale versions of several packages, which do not get updated automatically by vcpkg install. This ensures they get updated before installing new packages.
This now also removes the old manually-created x86-windows-static-dyncrt triplet in favor of the upstream-supported x86-windows-static-md triplet.