diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2022-03-10 15:22:03 +0100 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2022-03-10 15:22:07 +0100 |
| commit | 597ee30b5e2361a91e319fed65c0ece4ff30a5c9 (patch) | |
| tree | 5707c64c1957d9c53d052493ee6cfd0c9549968b /depends | |
| parent | 6c37eae0ad9430c6a7e0c4818b94305f271b19e2 (diff) | |
| parent | 3566353c5e8c699f5da80cb3f8081e407f1b5813 (diff) | |
Merge bitcoin/bitcoin#24522: ci: remove compiled-but-unused BDB from MSAN job
3566353c5e8c699f5da80cb3f8081e407f1b5813 ci: remove compiled-but-unused BDB from MSAN job (fanquake)
Pull request description:
Self-compiled BDB was added to this job as opposed to using depends BDB [due to linking issues](https://github.com/bitcoin/bitcoin/pull/18288#discussion_r433189350), 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](https://github.com/google/oss-fuzz/blob/master/projects/bitcoin-core/build.sh#L32) 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.
Top commit has no ACKs.
Tree-SHA512: 9e8fdd95246cafa27cda7bcf0641b428d4573f6748ecdf07cc6205a64351db22ba383ec943e88a69df3694ccb9f125d994b64345a4e44fb6fea4a014504760d1
Diffstat (limited to 'depends')
0 files changed, 0 insertions, 0 deletions