Skip to content

More fuzztest cleanups #547

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

Merged
merged 5 commits into from
May 9, 2023
Merged

Conversation

apoelstra
Copy link
Member

Followup to #538. Copies the new fuzzing scripts from rust-bitcoin and cleans some things up.

Confirmed that I can run all these fuzztests in my local nix-based CI.

@apoelstra
Copy link
Member Author

apoelstra commented May 8, 2023

Blocked on resolving rust-bitcoin/rust-bitcoin#1842 (since this PR has the same problem, and Github similarly is providing zero indication of error on the PR itself) should be good to go now

Copy link
Member

@sanket1729 sanket1729 left a comment

Choose a reason for hiding this comment

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

utACK a6405f8

@sanket1729 sanket1729 merged commit be27637 into rust-bitcoin:master May 9, 2023
@apoelstra apoelstra deleted the 2023-05--fuzz branch May 9, 2023 17:06
gruve-p pushed a commit to Groestlcoin/rust-miniscript that referenced this pull request Aug 28, 2023
a6405f8 fuzz: replace README.md (Andrew Poelstra)
7b04860 fuzz: delete old ./travis-fuzz.sh (Andrew Poelstra)
8e8dcb4 fuzz: remove afl support, run clippy and fmt on fuzz targets (Andrew Poelstra)
14ceccd fuzz: remove existing CI; run fuzz/generate-files.sh to make new one (Andrew Poelstra)
6fea17f fuzz: copy *.sh files from rust-bitcoin; tweak generate-files.sh (Andrew Poelstra)

Pull request description:

  Followup to rust-bitcoin#538. Copies the new fuzzing scripts from rust-bitcoin and cleans some things up.

  Confirmed that I can run all these fuzztests in my local nix-based CI.

ACKs for top commit:
  sanket1729:
    utACK a6405f8

Tree-SHA512: f19fa7a62e244aaebf3ac0a0bcfe1686392eca3679d71edd8d6504888bf1a5004a7527fa46e8ee6296be4a27dfc20408a8ea258569f6337a73f41118d751a451
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants