-
Notifications
You must be signed in to change notification settings - Fork 280
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
Fix appveyor vcpkg cache directory #447
Conversation
Was the test successful? |
By default, saving cache is disabled in Pull Request builds so I am now trying with the main branch in my fork, still wip. |
vcpkg in manifest mode creates the vcpkg_installed directory in the build directory
Without cache https://ci.appveyor.com/project/scribam/libzip/builds/50295040/job/allxlkipr79aaqr6
With cache https://ci.appveyor.com/project/scribam/libzip/builds/50295188/job/qpdoo4m7jqs6yglg
|
@@ -65,8 +66,6 @@ before_build: | |||
cmd: >- | |||
py -m pip install nihtest | |||
|
|||
vcpkg install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not required in manifest mode, CMake will take care of this install automatically
Looks good, thanks for diving into this! |
vcpkg in manifest mode creates the vcpkg_installed directory in the build directory
Amend commit d0ab655
CMake configure step exposed the vcpkg_installed directory in appveyor logs:
-- Found ZLIB: optimized;C:/projects/libzip/build/vcpkg_installed/x64-windows/lib/zlib.lib;debug;C:/projects/libzip/build/vcpkg_installed/x64-windows/debug/lib/zlibd.lib (found suitable version "1.3.1", minimum required is "1.1.2")