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

ci: Bump vcpkg release tag #23310

Merged
merged 1 commit into from
Oct 20, 2021
Merged

ci: Bump vcpkg release tag #23310

merged 1 commit into from
Oct 20, 2021

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Oct 19, 2021

In the new vcpkg release the "Binary Caching" and "Manifest Mode" features are available by default.

Suggested in #23215 (comment) and #23215 (comment).

In the new release the "Binary Caching" and "Manifest Mode" features are
available by default.
@sipsorcery
Copy link
Contributor

tACK 13ae568.

@DrahtBot
Copy link
Contributor

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #23215 (ci: Add vcpkg downloads cache by hebasto)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@fanquake
Copy link
Member

In the new vcpkg release the "Binary Caching"

I assume this speeds up our build in some way?

@fanquake fanquake merged commit 16df28c into bitcoin:master Oct 20, 2021
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 20, 2021
@hebasto
Copy link
Member Author

hebasto commented Oct 20, 2021

In the new vcpkg release the "Binary Caching"

I assume this speeds up our build in some way?

Build time:

  • unpopulated or invalidated binary cache: 27:51
  • fully populated binary cache: 15:04

@hebasto hebasto deleted the 211019-vcpkg branch October 20, 2021 13:21
fanquake added a commit that referenced this pull request Oct 23, 2021
f778845 ci: Add vcpkg tools cache (Hennadii Stepanov)

Pull request description:

  On master (02feae5) vcpkg downloads some tools that are used internally:
  ```
  ...
    A suitable version of cmake was not found (required v3.20.0). Downloading portable cmake v3.20.0...
    Downloading cmake...
      https://github.com/Kitware/CMake/releases/download/v3.20.2/cmake-3.20.2-windows-i386.zip -> C:\Users\ContainerAdministrator\AppData\Local\Temp\vcpkg\downloads\cmake-3.20.2-windows-i386.zip
    Extracting cmake...
    A suitable version of 7zip was not found (required v18.1.0). Downloading portable 7zip v18.1.0...
    Downloading 7zip...
      https://www.nuget.org/api/v2/package/7-Zip.CommandLine/18.1.0 -> C:\Users\ContainerAdministrator\AppData\Local\Temp\vcpkg\downloads\7-zip.commandline.18.1.0.nupkg
    Extracting 7zip...
    A suitable version of nuget was not found (required v5.5.0). Downloading portable nuget v5.5.0...
    Downloading nuget...
      https://dist.nuget.org/win-x86-commandline/v5.5.1/nuget.exe -> C:\Users\ContainerAdministrator\AppData\Local\Temp\vcpkg\downloads\22ea847d-nuget.exe
    Detecting compiler hash for triplet x64-windows-static...
    A suitable version of powershell-core was not found (required v7.1.0). Downloading portable powershell-core v7.1.0...
    Downloading powershell-core...
      https://github.com/PowerShell/PowerShell/releases/download/v7.1.3/PowerShell-7.1.3-win-x86.zip -> C:\Users\ContainerAdministrator\AppData\Local\Temp\vcpkg\downloads\PowerShell-7.1.3-win-x86.zip
    Extracting powershell-core...
  ...
  ```

  If any of downloads above fails the whole CI task fails (see #23107). The most recent failure examples in the master branch:
  - c001da3, [log](https://api.cirrus-ci.com/v1/task/5182966176415744/logs/build.log)
  - 12ff899, [log](https://api.cirrus-ci.com/v1/task/5367684129882112/logs/build.log)

  This PR adds vcpkg tools cache.

  Closes #23107.

  ---

  Note for reviewers. Some patches from the initial PR were split into separated PRs: #23310 and #23329. Therefore, a discussion here could be outdated or irrelevant until the recent [push](#23215 (comment)).

ACKs for top commit:
  sipsorcery:
    ACK f778845.

Tree-SHA512: 201f4e4d38c00cbec6aaeca4f3e1b60f1c65289fb68b82cead47f37f6af5ac42dd539cf8ed3566f5bd9afe4a7762d42bbabb316d58f47352d4e7bfc2214806fe
@bitcoin bitcoin locked and limited conversation to collaborators Oct 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants