ci: Use Multi-ToolTask in "Win64 native" task #26299

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:221012-mtt changing 1 files +1 −1
  1. hebasto commented at 4:51 PM on October 12, 2022: member

    See https://devblogs.microsoft.com/cppblog/improved-parallelism-in-msbuild/.

    Build time (ccache cache has been invalidated to ensure equal initial conditions):

    • on the master branch: image

    • this PR changes: image

    Also consider "CPU Usage" charts provided by CI.

  2. ci: Use Multi-ToolTask in "Win64 native" task
    See https://devblogs.microsoft.com/cppblog/improved-parallelism-in-msbuild/
    98c389ccff
  3. DrahtBot added the label Tests on Oct 12, 2022
  4. aureleoules approved
  5. aureleoules commented at 5:00 PM on October 12, 2022: member

    ACK 98c389ccff2d808524fa6d8b8091465e7b86ea1c :tada:

  6. aureleoules commented at 5:08 PM on October 12, 2022: member

    By default Cirrus CI will store logs and caches for 90 days but it can be changed by manually configuring a lifecycle rule for a Google Cloud Storage bucket that Cirrus CI is using.

    from https://cirrus-ci.org/guide/supported-computing-services/

    To be sure, this means this change is mostly useful for pulls that are inactive for > 90 days? Or are there other reasons why the cache could be invalidated?

  7. hebasto commented at 5:14 PM on October 12, 2022: member

    To be sure, this means this change is mostly useful for pulls that are inactive for > 90 days? Or are there other reasons why the cache could be invalidated?

    ccache has been invalidated only for benchmarking reason to compare build time with the same initial conditions. The PR description has been amended.

    All "Win64 native" CI tasks will benefit from this PR changes.

  8. DrahtBot commented at 10:38 PM on October 12, 2022: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #25797 (build: Add CMake-based build system 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.

  9. jarolrod commented at 3:02 AM on October 13, 2022: member

    ACK 98c389ccff2d808524fa6d8b8091465e7b86ea1c

    The doc does mention it's experimental, but this does seem useful. We can at the very least give this a trial run to see if any issues appear over several pr's. The speed-up is substantial.

  10. hebasto commented at 10:35 AM on October 13, 2022: member

    Friendly ping @sipsorcery :)

  11. maflcko merged this on Oct 13, 2022
  12. maflcko closed this on Oct 13, 2022

  13. hebasto deleted the branch on Oct 13, 2022
  14. sipsorcery commented at 6:53 PM on October 13, 2022: member

    Nice find!

    tACK 98c389ccff2d808524fa6d8b8091465e7b86ea1c.

  15. bitcoin locked this on Oct 13, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-24 21:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me