ci: Use native platform for win-cross task #33558

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2510-native-win-cross changing 1 files +0 −1
  1. maflcko commented at 10:32 AM on October 7, 2025: member

    Forcing the architecture to amd64 is no longer required. Dropping it should have some benefits:

    • Faster CI speed on other arches (riscv64, arm, ...)
    • Unlock the CI task to run on riscv64 at all
  2. ci: Use native platform for win-cross task fa6fd16f36
  3. DrahtBot added the label Tests on Oct 7, 2025
  4. DrahtBot commented at 10:32 AM on October 7, 2025: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33558.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK hebasto

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

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  5. hebasto commented at 11:28 AM on October 7, 2025: member

    Tested fa6fd16f36e1240cda58a46e1717b02e8d3172a3:

    $ uname -mo
    riscv64 GNU/Linux
    $ docker --version
    podman version 4.3.1
    $ env -i HOME="$HOME" PATH="$PATH" USER="$USER" bash -c 'MAKEJOBS="-j3" FILE_ENV="./ci/test/00_setup_env_win64.sh" ./ci/test_run_all.sh'
    <snip>
    + echo 'Creating mirror.gcr.io/ubuntu:24.04 container to run in'
    Creating mirror.gcr.io/ubuntu:24.04 container to run in
    + docker buildx build --file /home/hebasto/dev/bitcoin/ci/test_imagefile --build-arg CI_IMAGE_NAME_TAG=mirror.gcr.io/ubuntu:24.04 --build-arg FILE_ENV=./ci/test/00_setup_env_win64.sh --build-arg BASE_ROOT_DIR=/ci_container_base --platform=linux --label=bitcoin-ci-test --tag=ci_win64 /home/hebasto/dev/bitcoin
    Error: invalid platform syntax for "linux" (use OS/ARCH[/VARIANT][,...])
    Command '['./ci/test/02_run_container.sh']' returned non-zero exit status 125.
    
  6. maflcko commented at 11:56 AM on October 7, 2025: member

    Error: invalid platform syntax for "linux" (use OS/ARCH[/VARIANT][,...])

    thanks for testing. However, this looks like an unrelated issue with your podman version. I presume you can't run any native task at all. My recommendation would be to upgrade your os (or podman) to the latest stable version.

  7. hebasto commented at 3:42 PM on October 12, 2025: member
    • Unlock the CI task to run on riscv64 at all

    Isn't this relevant to other tasks as well, for example, ci/test/00_setup_env_arm.sh?

  8. hebasto approved
  9. hebasto commented at 4:12 PM on October 12, 2025: member

    ACK fa6fd16f36e1240cda58a46e1717b02e8d3172a3, tested on Ubuntu 24.04, RISC-V.

  10. maflcko commented at 6:55 AM on October 13, 2025: member
    • Unlock the CI task to run on riscv64 at all

    Isn't this relevant to other tasks as well, for example, ci/test/00_setup_env_arm.sh?

    That task requires the aarch64 architecture and is not a native task. Not sure if it matters much if qemu-aarch64 is running on x86 or riscv, but feel free to open a separate issue or pull request. The goal here is mainly to speed up the win-cross task for non-x86 arches. Being able to run it on riscv64 is a nice cherry on top.

  11. fanquake merged this on Oct 13, 2025
  12. fanquake closed this on Oct 13, 2025

  13. maflcko deleted the branch on Oct 13, 2025
  14. fanquake referenced this in commit 20e99515a3 on Oct 13, 2025
  15. bitcoin deleted a comment on Oct 13, 2025
  16. fanquake referenced this in commit 72d1141bd8 on Oct 16, 2025
  17. sedited referenced this in commit f50d90fcbb on Oct 29, 2025
  18. stickies-v referenced this in commit a785fe6759 on Nov 5, 2025
  19. stringintech referenced this in commit b82132c4d1 on Nov 6, 2025
  20. yuvicc referenced this in commit f620a2206b on Nov 16, 2025
  21. fanquake referenced this in commit 72675b8f55 on Dec 2, 2025
  22. morozow referenced this in commit d6bb84c4a5 on May 8, 2026
  23. morozow referenced this in commit 7e13824ba7 on May 8, 2026
  24. Kino1994 referenced this in commit 00e4b4dc28 on Jun 28, 2026
  25. BigcoinBGC referenced this in commit 15dbe8b16e on Jun 30, 2026
  26. Kino1994 referenced this in commit feffe62ab4 on Aug 19, 2026
  27. bitcoin locked this on Oct 13, 2026

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-10-21 18:52 UTC

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