lemire / streamvbyte

Fast integer compression in C using the StreamVByte codec
Apache License 2.0
374 stars 37 forks source link

CMake: allow parent project to disable unit tests for streamvbyte #22

Closed emaxerrno closed 6 years ago

emaxerrno commented 6 years ago

While integrating into another cmake project via add_subdirectory() the target check might already be defined by a parent directory.

In addition, on a parent project when you run its unit tests you don't want child project unit tests to run

For example, this won't build the unit tests:

cmake -DSTREAMVBYTE_ENABLE_TESTS=OFF .. 
lemire commented 6 years ago

Does STREAMVBYTE_ENALBE_TESTS include a typo? ENABLE vs. ENALBE?

emaxerrno commented 6 years ago

@lemire Yeah! good catch. I fixed the typo.

Also squashed the commits, and cleaned up the history

lemire commented 6 years ago

Merging.