Remove the unused build option, which was dangerous (as the name implies). Also remove the fuzzbuzz config, which was never used as part of this repo and seems redundant now that we integrate with oss-fuzz.
fuzz: Remove unused --enable-danger-fuzz-link-all option #21884
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2105-fuzzNoDanger changing 4 files +0 −56-
MarcoFalke commented at 7:41 AM on May 8, 2021: member
-
fuzz: Remove unused --enable-danger-fuzz-link-all option fa27d6d3ac
- MarcoFalke requested review from adamjonas on May 8, 2021
- MarcoFalke requested review from practicalswift on May 8, 2021
- fanquake added the label Tests on May 8, 2021
-
practicalswift commented at 9:47 AM on May 8, 2021: contributor
cr ACK fa27d6d3ac065684a1219e9a948514d27929cf7c: patch looks correct and rationale makes sense
Thanks for cleaning up!
- practicalswift approved
-
hebasto commented at 1:44 PM on May 8, 2021: member
Concept ACK.
- The fuzzbuzz integration configuration file was added in #18634, and it is really not useful in this repo.
- The
--enable-danger-fuzz-link-alloption was added in #20560 for backwards compatibility (see #20560 (comment) and #20560 (comment)).
- hebasto approved
-
hebasto commented at 1:48 PM on May 8, 2021: member
ACK fa27d6d3ac065684a1219e9a948514d27929cf7c, I have reviewed the code and it looks OK, I agree it can be merged.
- fanquake merged this on May 9, 2021
- fanquake closed this on May 9, 2021
- MarcoFalke deleted the branch on May 9, 2021
- sidhujag referenced this in commit ae49676b7a on May 9, 2021
- gwillen referenced this in commit 1254ad9cdb on Jun 1, 2022
- DrahtBot locked this on Aug 16, 2022