Travis should build and test arm asm #598

1. 
   gmaxwell commented at 9:20 pm on March 9, 2019: contributor
2. gmaxwell added this to the milestone initial release on May 23, 2019
3. 
   gmaxwell commented at 0:33 am on May 24, 2019: contributor

   Travis should also test make install.
4. 
   benma commented at 7:59 am on July 19, 2019: contributor

   Related: Travis does not seem to run ./tests. Is there a reason for that?
5. 
   real-or-random commented at 8:43 am on July 19, 2019: contributor

   It does, see https://travis-ci.org/bitcoin-core/secp256k1/jobs/560256756#L708 for example. It just runs the entire testsuite using make check-TESTS. Maybe we should just update the README.
6. 
   benma commented at 12:02 pm on July 19, 2019: contributor

   Of course!

   Updated README here: https://github.com/bitcoin-core/secp256k1/pull/652

7. 
   real-or-random commented at 3:56 pm on August 17, 2019: contributor

   Travis should also test different window size settings.
8. 
   real-or-random commented at 5:14 pm on October 8, 2019: contributor

   Travis has now (alpha) support for ARM: https://blog.travis-ci.com/2019-10-07-multi-cpu-architecture-support
9. 
   elichai commented at 10:32 am on November 8, 2019: contributor

   FWIW it’s arm64, and we have asm only for arm, not arm64.
10. 
    laanwj commented at 10:39 am on November 8, 2019: member

    It can build and run ARM executables on ARM64 (FWIW this is what we do in bitcoin in the ARM64 travis run, see https://github.com/bitcoin/bitcoin/pull/17233)
11. 
    elichai commented at 10:44 am on November 8, 2019: contributor

    Didn’t know arm64 can execute arm binaries. i’ll give this a try
12. 
    laanwj commented at 11:06 am on November 8, 2019: member

    I think it’s a feature that many SoCs have, including the one Travis happens to use, not a general requirement for arm64
13. MarcoFalke cross-referenced this on Dec 5, 2019 from issue travis: Add some arm builds by MarcoFalke
14. real-or-random cross-referenced this on Jan 6, 2020 from issue General CI discussion by real-or-random
15. jonasnick removed this from the milestone initial release on Apr 28, 2020
16. 
    gmaxwell commented at 5:02 am on July 28, 2020: contributor

    Even just having arm64 would be a big improvement. Sure, that wouldn’t test that the arm ASM builds and works– which is the critical thing motivating this issue, but there isn’t anything blocking that as far as I know.

    Similarly, there appears to be no BE host in this project’s travis configuration. s390x is be, I believe. (lol)

17. 
    elichai commented at 8:12 am on July 28, 2020: contributor

    CC https://github.com/bitcoin-core/secp256k1/pull/696
18. real-or-random cross-referenced this on May 3, 2021 from issue Add ARM32/ARM64 CI by sipa
19. 
    real-or-random commented at 11:43 am on December 20, 2021: contributor

    This has been solved by #930.
20. real-or-random closed this on Dec 20, 2021

    ---

Contributors
gmaxwell benma real-or-random elichai laanwj