CI msvc: only build vcpkg dependencies for release (not debug) to reduce build times #20489

pull sipsorcery wants to merge 1 commits into bitcoin:master from sipsorcery:msvc_vcpkg_relonly changing 1 files +1 −0

sipsorcery commented at 11:29 AM on November 25, 2020: member

This change to the appveyor CI config for msvc builds reverses a change introduced in #19960. It re-applies a setting to inform vcpkg to only build release versions of the dependencies rather than the default of debug and release.

It had been expected that the vcpkg manifest mechanism introduced in #19960 would do this automatically but it turns out not to be the case.
This change to the appveyor CI config for msvc builds reverses a change introduced in #19960. It re-applies a setting to inform vcpkg to only build release vesions of the dependencies rather than the default of debug and release.
It had been expected that the vcpkg manifest mechanism introduced in #19960 would do this automatically but it turns out not to be the case.
fa18e7cbc5
fanquake added the label Windows on Nov 25, 2020
fanquake added the label Tests on Nov 25, 2020
hebasto commented at 11:37 AM on November 25, 2020: member

This change to the appveyor CI config for msvc builds reverses a change introduced in #19960.

You mean the partial change reversing, right?
sipsorcery commented at 11:43 AM on November 25, 2020: member

You mean the partial change reversing, right?

Yes. The vcpkg manifest mechanism stays (from what I can tell it's the way of the future for vcpkg) but a previous tweak to adjust the vcpkg settings is being put back.
MarcoFalke commented at 11:48 AM on November 25, 2020: member

ACK fa18e7cbc5ea6aaba94dca4ebdc850c9db141f89 if green
hebasto approved
hebasto commented at 3:23 PM on November 25, 2020: member

ACK fa18e7cbc5ea6aaba94dca4ebdc850c9db141f89, AppVeyor build takes 5 minutes less.
MarcoFalke merged this on Nov 25, 2020
MarcoFalke closed this on Nov 25, 2020
sidhujag referenced this in commit a985056063 on Nov 25, 2020
MarcoFalke referenced this in commit e7b53d4721 on Dec 16, 2020
hebasto referenced this in commit c80c8c250b on Dec 16, 2020
DrahtBot locked this on Feb 15, 2022

Contributors

Labels

Linked (view graph)

#19960 build: The vcpkg tool has introduced a proper way to use manifests