Create test_fuzz library from ./src/test/fuzz/fuzz.cpp #18527

issue MarcoFalke openend this issue on April 4, 2020
  1. MarcoFalke commented at 9:53 pm on April 4, 2020: member

    We have one wrapper file around the fuzzing engine (src/test/fuzz/fuzz.cpp), so that fuzz tests can be written independent of the engine. This file is compiled for each fuzz target (harness), so with currently more than 114 fuzz targets this is quite an overhead. All the wrapper does is provide the main function for the fuzz target.

    I am not sure, but I believe the main function can also be provided by a library. Moving this file to a library would also compile it only once, thus speeding up compilation of the fuzz tests.

    This issue can probably be solved similar to #17542

    Useful skills:

    • Background in the autotools build system

    Want to work on this issue?

    The purpose of the good first issue label is to highlight which issues are suitable for a new contributor without a deep understanding of the codebase.

    You do not need to request permission to start working on this. You are encouraged to comment on the issue if you are planning to work on it. This will help other contributors monitor which issues are actively being addressed and is also an effective way to request assistance if and when you need it.

    For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.

  2. MarcoFalke added the label good first issue on Apr 4, 2020
  3. MarcoFalke added the label Build system on Apr 4, 2020
  4. brakmic commented at 10:19 pm on April 4, 2020: contributor
    Will look into it.
  5. MarcoFalke closed this on Apr 5, 2020

  6. DrahtBot locked this on Feb 15, 2022

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: 2024-11-23 21:12 UTC

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