fuzz: copy some more files from rust-bitcoin#285
Merged
Conversation
If we want to keep seeds from these directories we should move them to the qa-assets repo and make some effort to minimize them. It's easy to get 10s of 000s of these files if you're just running the fuzzer without making an effort to clean them up.
You should be able to run this script cleanly. Update it to include a recent Cargo.toml change, and add a big header which will hopefully remind me not to edit the Cargo.toml file directly.
The README has a few references to rust-bitcoin. I left these alone because all the content of the doc should be the same between the two repos, and it is probably a useful hint to somebody reading the README that the fuzz infrastructure was copied from rust-bitcoin.
Member
Author
|
On 53efbb2 successfully ran local tests |
delta1
approved these changes
Jun 29, 2026
| # 1. Generate fuzz/Cargo.toml | ||
| cat > "$REPO_DIR/fuzz/Cargo.toml" <<EOF | ||
| ###### | ||
| ## DO NOT EDIT THIS FILE DIRECTLY. It is generated by generate-fuzz.sh. |
Member
There was a problem hiding this comment.
nit: can be done in a follow-up if necessary
Suggested change
| ## DO NOT EDIT THIS FILE DIRECTLY. It is generated by generate-fuzz.sh. | |
| ## DO NOT EDIT THIS FILE DIRECTLY. It is generated by generate-files.sh. |
Member
Author
There was a problem hiding this comment.
Will do in a followup so we can get the CI job running (or not) ASAP.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This should complete the transition to cargo-fuzz.
Fixes #271.