Travis should build and test arm asm #598

issue gmaxwell openend this issue on March 9, 2019
  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
  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: 2024-10-30 05:15 UTC

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