ci: Update "Win64 native" task #26891

pull hebasto wants to merge 2 commits into bitcoin:master from hebasto:230113-vcpkg changing 1 files +2 −2

hebasto commented at 9:45 PM on January 13, 2023: member
This PR bumps vcpkg and ccache versions.

Dependency changes in vcpkg (2022.09.27 - 2023.01.09):
- boost 1.80.0#0 -> 1.81.0#0
- libevent 2.1.12#6 -> libevent 2.1.12#7
- sqlite3 3.39.2#0 -> 3.40.0#1
Also see #26866 (comment).

ci: Bump vcpkg to the latest version 2023.01.09

Dependency changes (2022.09.27 - 2023.01.09):
 - boost 1.80.0#0 -> 1.81.0#0
 - libevent 2.1.12#6 -> libevent 2.1.12#7
 - sqlite3 3.39.2#0 -> 3.40.0#1

1039ed44fb

ci: Bump `ccache` version to the latest 4.7.4 in "Win64 native" task 46c31eea8c
DrahtBot commented at 9:45 PM on January 13, 2023: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Reviews

See the guideline for information on the review process. A summary of reviews will appear here.
DrahtBot added the label Tests on Jan 13, 2023
hebasto added the label Windows on Jan 15, 2023
maflcko merged this on Jan 16, 2023
maflcko closed this on Jan 16, 2023
hebasto deleted the branch on Jan 16, 2023
in .cirrus.yml:157 in 46c31eea8c
```
 152 | @@ -153,7 +153,7 @@ task:
 153 |    ccache_cache:
 154 |      folder: '%CCACHE_DIR%'
 155 |    install_tools_script:
 156 | -    - choco install --yes --no-progress ccache --version=4.6.1
 157 | +    - choco install --yes --no-progress ccache --version=4.7.4
 158 |      - choco install --yes --no-progress python3 --version=3.9.6
```
maflcko commented at 10:39 AM on January 16, 2023:

nit, realized after merge: I guess it should be fine to bump this as well, so that everything is still pinned but bumped to the latest version. (And add a comment that bumping to the latest in this file is fine?)

Finally, a two line patch could be squashed into one commit, but no big deal.

hebasto commented at 1:58 PM on May 21, 2023: member

FWIW, the recent ccache 4.8.1 seems broken. Note the cacheable calls ratio:

Cacheable calls:   264 / 617 (42.79%)
  Hits:            264 / 264 (100.0%)
    Direct:        264 / 264 (100.0%)
    Preprocessed:    0 / 264 ( 0.00%)
  Misses:            0 / 264 ( 0.00%)
Uncacheable calls: 353 / 617 (57.21%)
Local storage:
  Cache size (GB): 0.2 / 0.2 (99.87%)
  Hits:            264 / 264 (100.0%)
  Misses:            0 / 264 ( 0.00%)

hebasto commented at 1:28 PM on July 13, 2023: member

FWIW, the recent ccache 4.8.1 seems broken. Note the cacheable calls ratio:

I found the reason that ccache >= 4.8 is not working as expected. We need to have <ObjectFileName>$(IntDir)%(FileName).obj</ObjectFileName> in every project file.

However, in the light of upcoming CMake-based build system, I see no reasons to change our current build_msvc directory.
fanquake referenced this in commit c7f0e97383 on Nov 30, 2023
bitcoin locked this on Jul 12, 2024

Contributors

Labels

Linked (view graph)

#26866 ci: remove ccache version pin in MSVC CI #28938 msvc: Update vcpkg manifest

ci: Update "Win64 native" task #26891

Reviews