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

Bump vcpkg version to fix msvc builds #2091

Merged
merged 1 commit into from
Oct 21, 2022

Conversation

olanti-p
Copy link
Contributor

@olanti-p olanti-p commented Oct 19, 2022

Summary

SUMMARY: Build "Bumped vcpkg version to fix msvc builds"

Purpose of change

Closes #2082

Describe the solution

Bump vcpkg version to latest-ish. It includes the change to zlib port file that makes it download the source code for version 1.2.12 from github mirror where it's still available.

It's a trivial change, yet should last us for another half a year or so if the trend continues.

Testing

Compiles locally, no game-breaking issues were noticed.
Opened the PR to test GitHub workflow, should work with no further changes.

Additional context

People with existing VS+vcpkg build setups will need to manually update their vcpkg.

Assuming the setup matches one described in the doc, this can be done in 2 steps:

  1. Open your vcpkg folder (usually C:\dev\vcpkg) in cmd and run:
git pull --ff origin master
  1. Open the solution in Visual Studio and do a "Clean Solution" + "Build Solution"

scarf005 added a commit to scarf005/Cataclysm-BN that referenced this pull request Oct 20, 2022
locally merge cataclysmbnteam#2091 in order to check if build succeeds

Co-authored-by: Olanti <olanti-p@yandex.ru>
@scarf005 scarf005 mentioned this pull request Oct 20, 2022
@Coolthulhu Coolthulhu merged commit b197454 into cataclysmbnteam:upload Oct 21, 2022
@olanti-p olanti-p deleted the fix-msvc-build branch October 24, 2022 19:57
@Firestorm01X2 Firestorm01X2 mentioned this pull request Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows builds failing due to zlib version change
2 participants