Page MenuHomePhabricator

Fix remaining test_bitcoin.cpp references (should be setup_common.cpp)
ClosedPublic

Authored by Fabien on Mar 19 2020, 18:00.

Details

Summary

Due to out of order landing of D5494, some reference were not updated.

Test Plan

export CC=clang CXX=clang++

../configure --enable-fuzz --with-sanitizers=fuzzer,address \
  --disable-wallet \
  --disable-bench \
  --with-utils=no \
  --with-daemon=no \
  --with-libs=no \
  --with-gui=no \
  --with-seeder=no \
  --with-miniupnpc=no  \
  --disable-zmq
make
cmake -GNinja .. -DENABLE_SANITIZERS="address;fuzzer" \
  -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++
ninja bitcoin-fuzzers

Diff Detail

Repository
rABC Bitcoin ABC
Branch
fix_setup_common
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9889
Build 17638: Default Diff Build & Tests
Build 17637: arc lint + arc unit

Event Timeline

jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
src/Makefile.test.include
62

I should point out that I did bring this up here: https://reviews.bitcoinabc.org/D5494#inline-34018

This revision is now accepted and ready to land.Mar 19 2020, 18:04
src/Makefile.test.include
62

My bad, I didn't understand the comment in D5494, just thought you were asking where the fuzz line came from