-
Notifications
You must be signed in to change notification settings - Fork 38.7k
[WIP] Fuzzing framework #7940
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Fuzzing framework #7940
Conversation
|
|
||
| if BUILD_BITCOIN_UTILS | ||
| bin_PROGRAMS += bitcoin-cli bitcoin-tx | ||
| bin_PROGRAMS += bitcoin-cli bitcoin-tx bitcoin-fuzzy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this should be under BITCOIN_UTILS, as that means it will be packaged as part of the binaries.
It needs to be in its own category, to be specifically enabled as a configure switch, I think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed
|
Concept ACK. |
|
Concept ACK |
|
I still think this is something we need, but I think someone with a better understanding of autotools should do this. |
|
Oh too bad, think this would have been good to have. But I guess in the mean time anyone that wants to fuzz can merge this or use your branch themselves. |
|
@theuni can you take a look here? Would be nice to have this, but looks like it ran into the cliffs of needing build system changes. |
No description provided.