cmake --build assumes our build directory is build_fuzz, but cmake didn't use -B build_fuzz, so simply copying and pasting the commands didn't work.
doc: Fix missing cmake -B option in fuzzing quickstart guide #34876
pull ekzyis wants to merge 1 commits into bitcoin:master from ekzyis:fix-fuzzing-quickstart-guide-missing-cmake-B-option changing 1 files +1 −1-
ekzyis commented at 9:49 AM on March 20, 2026: none
-
doc: Fix missing cmake -B option in fuzzing quickstart guide c70bab28c7
- DrahtBot added the label Docs on Mar 20, 2026
-
DrahtBot commented at 9:50 AM on March 20, 2026: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process. A summary of reviews will appear here.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
- ekzyis renamed this:
doc: Fix missing -B option for cmake in fuzzing quickstart guide
doc: Fix missing cmake -B option in fuzzing quickstart guide
on Mar 20, 2026 -
maflcko commented at 10:36 AM on March 20, 2026: member
I don't understand this. The preset literally has it set to that:
"binaryDir": "${sourceDir}/build_fuzz", -
maflcko commented at 10:37 AM on March 20, 2026: member
What is the output before and after the changes here?
-
hebasto commented at 12:00 PM on March 20, 2026: member
cmake --buildassumes our build directory is build_fuzz, butcmakedidn't use-B build_fuzz... -
ekzyis commented at 3:01 PM on March 20, 2026: none
Oh, I just checked my bash history. I did not actually copy and paste the commands. :sweat_smile:
I manually edited my
cmakecommand for clean rebuilds to include--preset=libfuzzerand--build build_fuzzinstead of--build build:$ rm -rf build && cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=1 --preset=libfuzzer -B build && cmake --build build_fuzz -j8This obviously did not create build_fuzz, but build.
When I run
cmake --preset=libfuzzer, like in the documentation, it does create build_fuzz.Sorry! I was too quick to assume inconsistent documentation and push a fix. Should have looked into
--presetfirst.I guess I'm too used to inconsistent documentation (in other projects) haha :sweat_smile:
- ekzyis closed this on Mar 20, 2026
- ekzyis deleted the branch on Mar 20, 2026