Skip to content

Conversation

pstratem
Copy link
Contributor

No description provided.

@@ -66,7 +66,7 @@ if BUILD_BITCOIND
endif

if BUILD_BITCOIN_UTILS
bin_PROGRAMS += bitcoin-cli bitcoin-tx
bin_PROGRAMS += bitcoin-cli bitcoin-tx bitcoin-fuzzy
Copy link
Member

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed

@laanwj
Copy link
Member

laanwj commented Apr 25, 2016

Concept ACK.
Would be useful to add documentation in doc/ on how to use this.

@laanwj laanwj added the Tests label Apr 25, 2016
@jtimon
Copy link
Contributor

jtimon commented May 20, 2016

Concept ACK

@pstratem
Copy link
Contributor Author

I still think this is something we need, but I think someone with a better understanding of autotools should do this.

@pstratem pstratem closed this Jul 11, 2016
@laanwj
Copy link
Member

laanwj commented Jul 11, 2016

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.

@laanwj
Copy link
Member

laanwj commented Aug 24, 2016

@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.

@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants