Rename fuzz_seed_corpus? #200

issue maflcko openend this issue on August 23, 2024
  1. maflcko commented at 10:36 am on August 23, 2024: contributor

    With the cmake-migration all fuzz scripts will have to be touched, which is probably the best time to rename the folder if there is desire to do so.

    The current name is minimally confusing, because seed doesn’t really apply. In reality it is a collection of fuzz input seeds, as well as fuzz inputs.

    So my recommendation would be to just drop seed. This will also allow in the future (when there is a need and desire) to provide a minimal set of possibly hand-crafted or otherwise non-fuzz-generated fuzz seed inputs to some fuzz targets (and possibly store them in a separate folder and validate that their format is still accurate and matches the fuzz target code). Even if this is never done, dropping seed may still be useful.

    Also, corpus could be renamed to corpora, to clarify that the folder holds the fuzz inputs for several fuzz targets.

    Thoughts?

  2. maflcko commented at 10:37 am on August 23, 2024: contributor
    cc @murchandamus @dergoegge yay or nay?
  3. dergoegge commented at 10:40 am on August 23, 2024: member
    ACK fuzz_corpora
  4. murchandamus commented at 3:04 pm on August 23, 2024: contributor
    Yay, fuzz_corpora!
  5. maflcko commented at 3:08 pm on August 23, 2024: contributor
    Ok, I’ll propose a pull request to do the git mv ./fuzz_seed_corpus fuzz_corpora on Monday, unless someone beats me to it.
  6. fanquake referenced this in commit ab317ad2ef on Sep 4, 2024
  7. maflcko closed this on Sep 4, 2024


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/qa-assets. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-10-30 01:25 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me