Skip to content

Commit 72c150d

Browse files
committed
Merge bitcoin/bitcoin#32055: contrib: Fix deterministic-unittest-coverage tool path
893ca54 contrib: Fix deterministic-unittest-coverage tool path (janb84) Pull request description: Fix for the tooling introduced/modified in #31901 but the tool path is broken due to silent merge conflict introduced by #31161. The `deterministic-unittest-coverage` and `deterministic-fuzz-coverage` tools uses the `fuzz` and `test_bitcoind` binaries, for which the location was modified in #31161. This patch updates the location to align with that change. ACKs for top commit: maflcko: lgtm ACK 893ca54 hebasto: ACK 893ca54, I have reviewed the code and it looks OK. Tree-SHA512: efd5a75b607a6a7889333674b9a1fccf30b6a80bb2aa941f9dcc10b2e0b6158d8c870d58aa76cef2a0280782570400730828d6a4aaf806b83959e9f4bc7313f4
2 parents c20a5ce + 893ca54 commit 72c150d

File tree

2 files changed

+2
-2
lines changed
  • contrib/devtools
    • deterministic-fuzz-coverage/src
    • deterministic-unittest-coverage/src

2 files changed

+2
-2
lines changed

contrib/devtools/deterministic-fuzz-coverage/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ fn main() {
6868

6969
let build_dir = Path::new(build_dir);
7070
let corpora_dir = Path::new(corpora_dir);
71-
let fuzz_exe = build_dir.join("src/test/fuzz/fuzz");
71+
let fuzz_exe = build_dir.join("bin/fuzz");
7272

7373
sanity_check(corpora_dir, &fuzz_exe);
7474

contrib/devtools/deterministic-unittest-coverage/src/main.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ fn main() {
5858
}
5959

6060
let build_dir = Path::new(build_dir);
61-
let test_exe = build_dir.join("src/test/test_bitcoin");
61+
let test_exe = build_dir.join("bin/test_bitcoin");
6262

6363
sanity_check(&test_exe);
6464

0 commit comments

Comments
 (0)