-
Notifications
You must be signed in to change notification settings - Fork 2.6k
infra: remove suppression of unknown-warning-option #14468
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
base: master
Are you sure you want to change the base?
Conversation
maflcko
left a comment
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.
thanks!
Let's see how many project need a project-specific flag.
Ref: #14457 Signed-off-by: David Korczynski <[email protected]>
|
/gcbrun trial_build.py all --fuzzing-engines libfuzzer --sanitizers address |
|
The CI says: |
I'm fixing this here https://github.com/google/oss-fuzz/pull/14470/changes |
|
This shouldn't be merged just yet. Likely we want to add some changes to a set of projects accompanying this PR. Will do that once we have the restults from the analysis of the projects |
|
Should be fine to rebase and re-run now? |
yep, I'll start the trial on after landing this one: #14313 |
|
/gcbrun trial_build.py all --fuzzing-engines libfuzzer --sanitizers address |
|
The infra build was passing now, but I can't see the logs, because for me they are either truncated or have a corrupt encoding. |
Ref: #14457