ci: remove compiled-but-unused BDB from MSAN job #24522

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:remove_unused_bdb_msan changing 2 files +1 −9
  1. fanquake commented at 1:14 pm on March 10, 2022: member

    Self-compiled BDB was added to this job as opposed to using depends BDB due to linking issues, however the compiled BDB is not actually used. Remove it for now, given we don’t actually lose any coverage (note that BDB is also not currently used in the naitve MSAN fuzz job or for OSS Fuzz builds).

    In future, we can use depends BDB, however introducing it now will cause false positives, which can be fixed by upgrading the versions of Clang / LLVM we use, however upgrading to those newer versions causes other issues, which appear in standard library code, and require more involved suppressing, which can be solved in a follow up or another PR i.e #23008.

  2. ci: remove compiled-but-unused BDB from MSAN job
    Self-compiled BDB was added to this job as opposed to using depends BDB
    due to linking issues, however the compiled BDB is not actually used.
    Remove it for now, given we don't actually lose any coverage (note that
    BDB is also no used the MSAN fuzz job), and in future, we can use
    depends BDB.
    3566353c5e
  3. fanquake added the label Tests on Mar 10, 2022
  4. MarcoFalke commented at 1:25 pm on March 10, 2022: member
  5. MarcoFalke commented at 1:27 pm on March 10, 2022: member
  6. MarcoFalke merged this on Mar 10, 2022
  7. MarcoFalke closed this on Mar 10, 2022

  8. fanquake deleted the branch on Mar 10, 2022
  9. sidhujag referenced this in commit 7a13f8c89f on Mar 11, 2022
  10. DrahtBot locked this on Mar 10, 2023

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: 2024-11-17 00:12 UTC

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