Travis should build and test arm asm #598

issue gmaxwell opened this issue on March 9, 2019
  1. gmaxwell commented at 9:20 PM on March 9, 2019: contributor

    No description provided.

  2. gmaxwell added this to the milestone initial release on May 23, 2019
  3. gmaxwell commented at 12: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
  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
  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


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-22 22:15 UTC

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