Skip to content

fuzzing.yml: use prebuilt cargo-fuzz for faster CI - #297

Merged
sylvestre merged 1 commit into
uutils:mainfrom
oech3:patch-1
Oct 12, 2026
Merged

sylvestre merged 1 commit into
uutils:mainfrom
oech3:patch-1

Conversation

@oech3

@oech3 oech3 commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

No description provided.

continue-on-error: ${{ !matrix.test-target.should_pass }}
run: |
cargo fuzz run ${{ matrix.test-target.name }} -- -max_total_time=${{ env.RUN_FOR }} -detect_leaks=0
cargo fuzz run --target host-tuple ${{ matrix.test-target.name }} -- -max_total_time=${{ env.RUN_FOR }} -detect_leaks=0

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

does cargo fuzz run find the binary with host-tuple?
i think it builds the path as fuzz/target//release/..., so it would look in fuzz/target/host-tuple/
we only run on ubuntu-latest, could you please just use x86_64-unknown-linux-gnu?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

host-tuple is same with x86_64-unknown-linux-gnu. No?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fuzz/target/host-tuple/

It is confusing with --profile=profile-name

@sylvestre
sylvestre merged commit 2d605f6 into uutils:main Oct 12, 2026
31 checks passed
@oech3
oech3 deleted the patch-1 branch October 12, 2026 06:24
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