cmake: Only do Java tests when unit testing on (#2551)
authorAndres Noetzli <andres.noetzli@gmail.com>
Fri, 28 Sep 2018 15:21:32 +0000 (08:21 -0700)
committerMathias Preiner <mathias.preiner@gmail.com>
Fri, 28 Sep 2018 15:21:32 +0000 (08:21 -0700)
Right now, we are adding the Java tests even when we are not building
unit tests. This commit changes the build system to only add the Java
tests when unit tests are enabled. There are two reasons for this
change:

- building a production version of CVC4 should not require JUnit
- it seems more intuitive (to me at least) to disable JUnit tests when
  unit tests are disabled

This change also simplifies building the Java bindings in our homebrew
formula.

test/CMakeLists.txt

index b85ecf9e774a2fb421698d6d48b590cc3589ef31..a79e96c6503113d2d8e29a939a0b9ea57ecdc481 100644 (file)
@@ -20,10 +20,10 @@ add_custom_target(check
 add_subdirectory(regress)
 add_subdirectory(system EXCLUDE_FROM_ALL)
 
-if(BUILD_BINDINGS_JAVA)
-  add_subdirectory(java)
-endif()
-
 if(ENABLE_UNIT_TESTING)
   add_subdirectory(unit EXCLUDE_FROM_ALL)
+
+  if(BUILD_BINDINGS_JAVA)
+    add_subdirectory(java)
+  endif()
 endif()