travis: Run unit tests --with-sanitizers=undefined #14036

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:Mf1803-travisAsan changing 2 files +25 −14
  1. MarcoFalke commented at 7:42 PM on August 23, 2018: member

    See issue #12691

  2. MarcoFalke added the label Tests on Aug 23, 2018
  3. MarcoFalke force-pushed on Aug 23, 2018
  4. MarcoFalke force-pushed on Aug 23, 2018
  5. MarcoFalke force-pushed on Aug 23, 2018
  6. MarcoFalke force-pushed on Aug 23, 2018
  7. MarcoFalke force-pushed on Aug 23, 2018
  8. MarcoFalke force-pushed on Aug 23, 2018
  9. depends: allow CC/CXX to be overridden during configure 0d00fd5901
  10. DrahtBot commented at 10:02 PM on August 23, 2018: member

    <!--e57a25ab6845829454e8d69fc972939a-->Note to reviewers: This pull request conflicts with the following ones:

    • #13863 (travis: move script sections to files in .travis/ subject to shellcheck by scravy)
    • #13827 (wip, depends: Add native_nsis to support unicode by ken2812221)
    • #13515 (travis: avoid timeout without saving caches, also enable all qt by ken2812221)
    • #12134 (Build previous releases and run functional tests by Sjors)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  11. practicalswift commented at 11:13 PM on August 23, 2018: contributor

    This is excellent!

    Strongest possible concept ACK (assuming we’re not hitting build timeout issues blocking this – that the builds take too long to finish)

  12. ken2812221 commented at 7:54 AM on August 24, 2018: contributor

    Concept ACK

  13. scravy commented at 10:15 AM on August 24, 2018: contributor

    Concept ACK

  14. MarcoFalke force-pushed on Aug 24, 2018
  15. MarcoFalke renamed this:
    travis: Compile with sanitizers and clang in one of the linux jobs
    travis: Compile --with-sanitizers=undefined --enable-werror
    on Aug 24, 2018
  16. scravy commented at 12:25 PM on August 24, 2018: contributor

    From the build log:

    checking whether we are using Intel C compiler... no
checking whether we are using Sun Studio C compiler... no
checking whether we are using clang C compiler... no
checking whether we are using gcc >= 4 C compiler... yes
checking whether the C++ compiler works... yes
checking whether we are using Intel C++ compiler... no
checking whether we are using Sun Studio C++ compiler... no
checking whether we are using clang C++ compiler... no
checking whether we are using gcc >= 4 C++ compiler... yes

    In https://travis-ci.org/bitcoin/bitcoin/jobs/420091638:

    Pull Request #14036 travis: Compile with sanitizers and clang in one of the linux jobs

    doesn't seem to be using clang at all.

  17. MarcoFalke commented at 12:28 PM on August 24, 2018: member

    Yes, it is not possible to set the compiler when building from depends. I will add this back, when it is possible.

  18. MarcoFalke force-pushed on Aug 24, 2018
  19. MarcoFalke force-pushed on Aug 24, 2018
  20. MarcoFalke renamed this:
    travis: Compile --with-sanitizers=undefined --enable-werror
    travis: Compile with Werror, clang on some linux jobs
    on Aug 24, 2018
  21. MarcoFalke closed this on Aug 24, 2018
  22. MarcoFalke deleted the branch on Aug 24, 2018
  23. MarcoFalke restored the branch on Aug 24, 2018
  24. MarcoFalke reopened this on Aug 24, 2018
  25. MarcoFalke force-pushed on Aug 24, 2018
  26. MarcoFalke renamed this:
    travis: Compile with Werror, clang on some linux jobs
    travis: Run unit tests --with-sanitizers=undefined
    on Aug 24, 2018
  27. MarcoFalke added the label Build system on Aug 24, 2018
  28. MarcoFalke force-pushed on Aug 25, 2018
  29. MarcoFalke force-pushed on Aug 25, 2018
  30. MarcoFalke force-pushed on Aug 25, 2018
  31. MarcoFalke force-pushed on Aug 25, 2018
  32. MarcoFalke force-pushed on Aug 25, 2018
  33. MarcoFalke force-pushed on Aug 25, 2018
  34. MarcoFalke closed this on Aug 25, 2018
  35. MarcoFalke deleted the branch on Aug 25, 2018
  36. MarcoFalke restored the branch on Aug 26, 2018
  37. MarcoFalke reopened this on Aug 26, 2018
  38. MarcoFalke force-pushed on Aug 26, 2018
  39. travis: Run unit tests --with-sanitizers=undefined fad8afa388
  40. MarcoFalke force-pushed on Aug 26, 2018
  41. laanwj commented at 9:05 AM on August 27, 2018: member

    utACK fad8afa3883675880dfa7fa6664fe658f10b11b5

  42. laanwj merged this on Aug 27, 2018
  43. laanwj closed this on Aug 27, 2018
  44. laanwj referenced this in commit 0ae15bd05f on Aug 27, 2018
  45. MarcoFalke deleted the branch on Aug 27, 2018
  46. Munkybooty referenced this in commit 0c2de3a954 on Jun 30, 2021
  47. Munkybooty referenced this in commit d973e50968 on Jun 30, 2021
  48. Munkybooty referenced this in commit 8fe64f740e on Jul 1, 2021
  49. Munkybooty referenced this in commit 49e86df289 on Jul 2, 2021
  50. Munkybooty referenced this in commit 3e72c81f70 on Jul 2, 2021
  51. Munkybooty referenced this in commit 19fc1dcf01 on Jul 4, 2021
  52. UdjinM6 referenced this in commit 3de8dd48e8 on Jul 7, 2021
  53. Munkybooty referenced this in commit 216eb6fb69 on Jul 7, 2021
  54. UdjinM6 referenced this in commit 5bef8db181 on Jul 7, 2021
  55. Munkybooty referenced this in commit a24a351158 on Jul 7, 2021
  56. Munkybooty referenced this in commit 7327a0b2ec on Jul 8, 2021
  57. MarcoFalke locked this on Sep 8, 2021
Contributors
MarcoFalkeDrahtBotpracticalswiftken2812221scravylaanwj
Labels
Build systemTests
Linked (view graph)
#12691 Enable -fsanitize flags in Travis
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-14 21:14 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me