build, ci: Fix MSVC builds and other improvements #24516

The current MSVC builds are broken due to the bug in the zeromq port. From IRC:

<sipsorcery> Looks like it's a problem downloading the zeromq dependency from https://patch-diff.githubusercontent.com/raw/zeromq/libzmq/pull/4311.diff <dhruv> sipsorcery: I'm definitely misunderstanding, i actually have no clue which file the CI is failing to download. I'll DM you more details. <sipsorcery> It's saying the hash of the patch file has changed. <dhruv> so we'd need to verify that the change is not malicious and then commit the new hash? <sipsorcery> No that dependency is managed by the vcpkg repo. Seems they might be working on it https://github.com/microsoft/vcpkg/pull/22681#issuecomment-1061312320 <dhruv> ok, thanks

This PR fixes this issue with specifying the previous port version explicitly.

The current CI task does not fail due to the cached binaries.

The second commit makes vcpkg binary cache invalid if dependencies changed.

The third commit drops double-conversion from dependencies as Qt is configured as follows:

Configure summary:

Build type: win32-msvc (x86_64, CPU features: sse sse2)
Compiler: msvc 193131104
Configuration: sse2 aesni sse3 ssse3 sse4_1 sse4_2 avx avx2 avx512f avx512bw avx512cd avx512dq avx512er avx512ifma avx512pf avx512vbmi avx512vl compile_examples f16c largefile msvc_mp precompile_header rdrnd rdseed shani silent x86SimdAlways release c++11 c++14 c++17 c++1z concurrent no-pkg-config static static_runtime stl
Build options:
...
Qt Core:
  DoubleConversion ....................... yes
    Using system DoubleConversion ........ no
...

cc @sipsorcery @dhruv

@hebasto looks good. First time I've seen the vcpkg builtin-baseline feature. Looks very handy.

I suspect you could remove the git -c advice.detachedHead=false checkout %CI_VCPKG_TAG% line from the cirrus build file now since the job of pegging the vcpkg dependencies is now being done by the builtin-baseline.

@sipsorcery

@hebasto looks good. First time I've seen the vcpkg builtin-baseline feature. Looks very handy.

I suspect you could remove the git -c advice.detachedHead=false checkout %CI_VCPKG_TAG% line from the cirrus build file now since the job of pegging the vcpkg dependencies is now being done by the builtin-baseline.

It appears that builtin-baseline is required for using of overrides. I hope we could revert the first commit when zeromq port get fixed. Therefore, suggesting to postpone the choice between git checkout and builtin-baseline.

tACK c3296b21e40be3a5cb7060ceb8f1b6db1fd79e65.

Thank you, @hebasto !