-
-
Notifications
You must be signed in to change notification settings - Fork 475
OSSFuzz Integration #830
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
OSSFuzz Integration #830
Conversation
Changed titlecase to lowercase for project name in build.sh Don't build examples or test for Crow fuzzing Signficiantly simplified corpus Added cifuzz.yml Clean up More fixes
Building the fuzzers on GitHub has a problem, do you have an idea what is going wrong? |
@gittiver The OSSFuzz CI build will fail until this PR is merged and until I get approval for a subsequent, dependent PR to google/oss-fuzz. It is currently trying to retrieve these fuzz files from the default branch of the repository, but it is not yet there (bit of a chicken and the egg problem). |
so we have to wait for the other approval? |
This PR needs to be merged first. Once you merge it, I will work with
OSS-Fuzz to begin fuzzing!
…On Thu, Jun 20, 2024 at 12:33 PM gittiver ***@***.***> wrote:
so we have to wait for the other approval?
—
Reply to this email directly, view it on GitHub
<#830 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHXFHTM6D2XJYRZZMDRZIMLZIL74TAVCNFSM6AAAAABJBKAUNSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBRGEYDIMRWGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
|
It will try to build on merges as is, so it will show as a fail for the
CIFuzz job. It won't have an affect on any other jobs, though- so you could
just ignore the CIFuzz job until it is fully integrated with OSS-Fuzz
(Hopefully within a week's time).
…On Thu, Jun 20, 2024 at 1:00 PM gittiver ***@***.***> wrote:
This PR needs to be merged first. Once you merge it, I will work with
OSS-Fuzz to begin fuzzing!
So it will not break the build on main branch on merge? It's hard to check
if it will work before merging int.
—
Reply to this email directly, view it on GitHub
<#830 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHXFHTORNHIG4EAOUF67FDDZIMDDDAVCNFSM6AAAAABJBKAUNSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBRGE2DSNRXGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
The harnesses and build.sh have already been integrated into master, as seen in this this [PR](CrowCpp/Crow#830) This pull request solely adds the necessary Dockerfile
Added cifuzz.yml Don't build examples or test for Crow fuzzing Changed titlecase to lowercase for project name in build.sh
Hello! Apologies for the long delay in getting this implemented, as discussed in Issue 744 with @gittiver.
I would greatly appreciate your review and integration of this PR.
As an overview of what next steps are for me to integrate this project fully into OSSFuzz:
Please let me know if there are any questions or concerns!