-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
infra/rust/coverage: copy right toolchain sources #11501
Conversation
When a project uses another toolchain than nightly-x86_64-unknown-linux-gnu we should get sources from this other toolchain
@DonggeLiu could this be taken in to fix suricata fuzzing coverage ? (and maybe other rust projects using a custom toolchain) |
Requesting help from @jonathanmetzman : ) |
/gcbrun trial_build.py rust --fuzzing-engines libfuzzer --sanitizers address |
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.
LGTM if testing passes.
So, gcb says this fixes suricata but breaks surrealdb Looking into it |
Got the fix easy. |
so as to get coverage
e28667a
to
d516ecf
Compare
friendly ping @jonathanmetzman as ClusterFuzz-External reminds me that suricata Coverage build is still failing ;-) |
/gcbrun trial_build.py rust --fuzzing-engines libfuzzer --sanitizers address |
trial build all green :-) |
@jonathanmetzman is the CFLite failure related/expected to this PR? |
@DonggeLiu this PR does not change anything for python ;-) |
Fixes https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=65817