test: add aarch64-apple-darwin platform entry to get_previous_releases #22790

pull Zero-1729 wants to merge 1 commits into bitcoin:master from Zero-1729:add-aarch64-apple-darwin-get-prev-releases changing 1 files +1 −0
  1. Zero-1729 commented at 10:57 AM on August 24, 2021: contributor

    Over the course of reviewing a PR, I had to edit test/get_previous_releases.py (after I ran git clean -xdff) to run the backwards compatibility tests (e.g. wallet_upgradewallet, feature_backwards_compatibility, etc.), as currently on master, running the script as indicated in test/README.md, for example, on an M1 machine results in the following error, as the aarch64-apple-darwin* platform entry is presently not recognised:

    Output from an M1 machine running macOS v11.5.2

    $ test/get_previous_releases.py -b v0.20.1 v0.19.1 v0.18.1 v0.17.2 v0.16.3 v0.15.2
    Releases directory: releases
    Not sure which binary to download for aarch64-apple-darwin20.6.0
    

    As a quick fix, this PR adds the missing aarch64-apple-darwin* platform entry. Running the script now results in fetching the old binaries, as expected:

    $ test/get_previous_releases.py -b v0.20.1 v0.19.1 v0.18.1 v0.17.2 v0.16.3 v0.15.2
    
    Releases directory: releases
    Fetching: https://bitcoincore.org/bin/bitcoin-core-0.20.1/bitcoin-0.20.1-osx64.tar.gz
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
      0 20.9M    0     0    0     0      0      0 --:--:--  0:00:02 --:--:--     0
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
    100 20.9M  100 20.9M    0     0   136k      0  0:02:37  0:02:37 --:--:-- 95607
    Checksum matched
    
    …
    
    Checksum matched
    

    After this patch, the backwards compatibility tests also run successfully, as expected.

    Note: I am open to other possible solutions.


    Steps to reproduce:

    Ensure you take out the binaries in releases if they already exist.

    Try running test/get_previous_releases.py -b v0.15.2 or similar to fetch the old release binaries.

  2. test: add aarch64-apple-darwin platform entry to get_previous_releases f6e4db27ce
  3. DrahtBot added the label Tests on Aug 24, 2021
  4. Zero-1729 commented at 11:10 AM on August 27, 2021: contributor

    cc @hebasto @jarolrod @fanquake

    Would love feedback from other M1 users as well.

  5. luke-jr commented at 4:08 PM on September 20, 2021: member

    So these are run via emulation?

  6. Zero-1729 commented at 3:36 AM on September 21, 2021: contributor

    So these are run via emulation?

    I haven't tried running it via emulation, I ran it locally on my machine.

  7. luke-jr commented at 3:40 AM on September 21, 2021: member

    You're downloading x86-64 binaries, not AArch64 binaries. That only makes sense if they actually work.

  8. Zero-1729 commented at 5:09 PM on September 21, 2021: contributor

    You're downloading x86-64 binaries, not AArch64 binaries. That only makes sense if they actually work.

    Yes, I am aware running the script fetches the old x86-64 osx binaries. The good thing is these old binaries work on the machine (thanks to rosetta 2 translation). Hence, adding the 'aarch64' platform to fetch these old binaries is an easy solution for Apple Silicon machines to run those binaries and the functional tests that require them.

  9. RandyMcMillan commented at 5:42 PM on September 21, 2021: contributor

    Works as expected.

    Screen Shot 2021-09-21 at 1 33 44 PM

    arm64
    

    Screen Shot 2021-09-21 at 1 39 57 PM

  10. luke-jr commented at 8:02 PM on September 21, 2021: member

    The good thing is these old binaries work on the machine (thanks to rosetta 2 translation).

    Great, that's what I was asking :)

  11. MarcoFalke merged this on Sep 22, 2021
  12. MarcoFalke closed this on Sep 22, 2021

  13. sidhujag referenced this in commit 549173fc70 on Sep 22, 2021
  14. DrahtBot locked this on Oct 30, 2022

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-27 00:14 UTC

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