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
Move some test data generation to main CMakeLists.txt (/ fix fuzzing build failures) #9122
Move some test data generation to main CMakeLists.txt (/ fix fuzzing build failures) #9122
Conversation
Move the generation of tests/src/test_certs.h and tests/src/test_keys.h to the main CMakeLists.txt. This is required because these files are needed both by tests and programs, whereas tests/CMakeLists.txt is only included when ENABLE_TESTING is on. Signed-off-by: David Horstmann <david.horstmann@arm.com>
47b0af9
to
1238b37
Compare
Check that we can build under CMake with ENABLE_TESTING=OFF but ENABLE_PROGRAMS=ON. Signed-off-by: David Horstmann <david.horstmann@arm.com>
We are testing that we don't break OSS-Fuzz, primarily. Signed-off-by: David Horstmann <david.horstmann@arm.com>
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 - thanks
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 - the note helps :)
Question: Does this need a ChangeLog? It is somewhat user-facing, so I suppose it probably does? |
Oopsie. I would say so yeah. It is a bug fix right? |
Yep that's right! I'll add a ChangeLog |
Oh hang on, it was never released! So we don't need a ChangeLog after all :) |
Huzzah! |
Marking |
Fixes a fuzzing build failure.
Move test data file generation to the main CMakeLists.txt and perform it whenever either
ENABLE_TESTING
orENABLE_PROGRAMS
is on.PR checklist
Please tick as appropriate and edit the reasons (e.g.: "backport: not needed because this is a new feature")