test: previous releases: speed up fetching sources with shallow clone #28332

pull theStack wants to merge 1 commits into bitcoin:master from theStack:test-get_previous_releases-shallow-clone changing 1 files +1 −4
  1. theStack commented at 12:40 AM on August 24, 2023: contributor

    For the sake of building previous releases, fetching the whole history of the repository for each version seems to be overkill as it takes much more time, bandwidth and disk space than necessary. Create a shallow clone instead with history truncated to the one commit of the version tag, which is directly checked out in the same command. This has the nice side-effect that we can remove the extra git checkout step after as it's not needed anymore.

    Note that it might look confusing to pass a tag to a parameter named --branch, but the git-clone manpage explicitly states that this is supported.

  2. test: previous releases: speed up fetching sources with shallow clone
    For the sake of building previous releases, fetching the whole history
    of the repository for each version seems to be overkill as it takes much
    more time, bandwidth and disk space than necessary. Create a shallow
    clone instead with history truncated to the one commit of the version
    tag, which is directly checked out in the same command. This has the
    nice side-effect that we can remove the extra `git checkout` step after
    as it's not needed anymore.
    
    Note that it might look confusing to pass a _tag_ to a parameter named
    `--branch`, but the git-clone manpage explicitly states that this is
    supported.
    360ac64b90
  3. DrahtBot commented at 12:40 AM on August 24, 2023: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK MarcoFalke

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  4. DrahtBot added the label Tests on Aug 24, 2023
  5. maflcko commented at 8:06 AM on August 24, 2023: member

    lgtm ACK 360ac64b90ee16cc24bd4c574ec7e11760515a79

  6. fanquake merged this on Aug 24, 2023
  7. fanquake closed this on Aug 24, 2023

  8. theStack deleted the branch on Aug 24, 2023
  9. Frank-GER referenced this in commit 4033a8aa18 on Sep 8, 2023
  10. bitcoin locked this on Aug 23, 2024
Labels

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-14 21:13 UTC

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