viniul commented on a change in pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#discussion_r443127661
##########
File path: CMakeLists.txt
##########
@@ -177,8 +177,14 @@ install(
# Process subdirectories
add_subdirectory(doc)
add_subdirectory(src)
-add_subdirectory(tests)
-add_subdirectory(samples)
+if (NOT (DEFINED XERCES_BUILD_FUZZERS))
Review comment:
Fuzzing requires some compile-time instrumentation of the xerces-library
code and certain libraries that need to be linked against the instrumented code
later on. Since this instrumentation is not needed for the other tests, because
I thought this way it would be cleaner. Do you still want to me to change?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscr...@xerces.apache.org
For additional commands, e-mail: c-dev-h...@xerces.apache.org
