build: Remove `cmake/script/CoverageFuzz.cmake` #36161

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:260903-fuzz-cov changing 2 files +0 −54
  1. hebasto commented at 2:42 PM on September 3, 2026: member

    The gcov-based CoverageFuzz script was introduced in 8b6f1c4353836bae6aa683cbc65251165bd031ba, as a CMake's replacement for the legacy cov_fuzz target. However, neither cov_fuzz nor CoverageFuzz has a documented usage.

    Instead, #32206 documented compiling for fuzz coverage using the LLVM/Clang toolchain, which does not involve the CoverageFuzz script.

    This PR removes the never-documented CoverageFuzz script, which is likely unused.

  2. build: Remove `cmake/script/CoverageFuzz.cmake`
    The `gcov`-based `CoverageFuzz` script was introduced in
    8b6f1c4353836bae6aa683cbc65251165bd031ba, as a CMake's replacement for
    the legacy `cov_fuzz` target. However, neither `cov_fuzz` nor
    `CoverageFuzz` has a documented usage.
    
    Instead, #32206 documented compiling for fuzz coverage using the
    LLVM/Clang toolchain, which does not involve the `CoverageFuzz` script.
    
    This change removes the never-documented `CoverageFuzz` script, which is
    likely unused.
    00a5f9b737
  3. hebasto added the label Build system on Sep 3, 2026
  4. hebasto added the label Fuzzing on Sep 3, 2026
  5. DrahtBot commented at 2:42 PM on September 3, 2026: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/36161.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline and AI policy for information on the review process.

    Type Reviewers
    ACK Crypt-iQ, sedited
    Concept ACK fanquake

    If your review is incorrectly listed, please copy-paste <code>&lt;!--meta-tag:bot-skip--&gt;</code> into the comment that the bot should ignore.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  6. hebasto commented at 2:43 PM on September 3, 2026: member

    Friendly ping fuzz people: @dergoegge @marcofleon @Crypt-iQ

  7. Crypt-iQ commented at 3:07 PM on September 3, 2026: contributor

    crACK 00a5f9b7375ec8dca944e200bd9bef6a29949409

    I use the llvm-based coverage so I am ok with removing this, but if somebody relies on this we could keep it

  8. fanquake commented at 8:14 AM on September 4, 2026: member

    Concept ACK

  9. sedited approved
  10. sedited commented at 11:58 AM on September 5, 2026: contributor

    ACK 00a5f9b7375ec8dca944e200bd9bef6a29949409

    I've also been exclusively using an LLVM toolchain for coverage for some time now.

  11. DrahtBot requested review from fanquake on Sep 5, 2026
  12. sedited merged this on Sep 6, 2026
  13. sedited closed this on Sep 6, 2026

  14. hebasto deleted the branch on Sep 6, 2026

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-09-09 07:56 UTC