fuzz: Add option to merge input dir to test runner #18300

pull MarcoFalke wants to merge 3 commits into bitcoin:master from MarcoFalke:Mf1902-fuzzMerge changing 2 files +37 −19

MarcoFalke commented at 3:17 PM on March 9, 2020: member

This is mainly useful for myself to merge pull requests like https://github.com/bitcoin-core/qa-assets/pull/4

I thought it wouldn't hurt to share the code.

Also remove the --disable-ccache from the docs to speed up builds when developing fuzzers.
doc: Remove --disable-ccache from docs fa4fa88d76
fuzz: Add option to merge input dir to test runner aaaa055ff7
fanquake added the label Tests on Mar 9, 2020
practicalswift commented at 5:28 PM on March 9, 2020: contributor

Concept ACK

Nice to have!
MarcoFalke commented at 5:48 PM on March 9, 2020: member

re-run ci
MarcoFalke closed this on Mar 9, 2020
MarcoFalke reopened this on Mar 9, 2020
fuzz: Remove option --export_coverage from test_runner
The coverage statistics are not stable across clang versions
fa3fa27c45
MarcoFalke closed this on Mar 11, 2020
MarcoFalke reopened this on Mar 11, 2020
DrahtBot cross-referenced this on Mar 13, 2020 from issue doc: Add fuzzing quickstart guides for libFuzzer and afl-fuzz by practicalswift
DrahtBot commented at 5:45 PM on March 13, 2020: contributor


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #18342 (doc: Add fuzzing quickstart guides for libFuzzer and afl-fuzz by practicalswift)
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.
in test/fuzz/test_runner.py:140 in aaaa055ff7 outdated
```
 135 | @@ -121,6 +136,22 @@ def main():
 136 |      )
 137 |  
 138 |  
 139 | +def merge_inputs(*, corpus, test_list, build_dir, merge_dir):
 140 | +    logging.info("Merge the inputs in the passed dir into the seed_dir. Passed dir {}".format(merge_dir))
```
sipa commented at 6:31 PM on March 13, 2020:

This looks like leftover debugging.

MarcoFalke commented at 6:38 PM on March 13, 2020:

Merging is currently a manual operation, so I like the extra verbosity. Happy to remove if you have strong feelings about this.

sipa commented at 7:02 PM on March 13, 2020:

If it's intentional, no problem; It just seemed written strangely as a log message, so I wanted to make sure it wasn't a leftover.
sipa commented at 6:34 PM on March 13, 2020: member

Concept ACK
MarcoFalke merged this on Mar 18, 2020
MarcoFalke closed this on Mar 18, 2020
MarcoFalke deleted the branch on Mar 18, 2020
sidhujag referenced this in commit e951bb1270 on Mar 18, 2020
MarkLTZ cross-referenced this on Apr 4, 2020 from issue Bitcoin PR tracking by MarkLTZ
deadalnix referenced this in commit e4b4c79490 on Nov 5, 2020
sidhujag referenced this in commit 4b6caad829 on Nov 10, 2020
furszy cross-referenced this on Mar 23, 2021 from issue Fuzzing framework support by furszy
random-zebra referenced this in commit 44b5327e61 on May 28, 2021
kwvg referenced this in commit db36f4efff on Aug 15, 2021
bitcoin locked this on Feb 15, 2022

Contributors

Labels

fuzz: Add option to merge input dir to test runner #18300

Conflicts