Eliminate double-negative in option name / value

pjohnmeyer · pjohnmeyer · commit 4d7ae2e1fcf1 · 2016-05-07T02:30:26.000-05:00
UTPP_SKIP_TESTS_AS_BUILD_STEP with a default value of OFF was an
unfortunate choice, so this changes it to UTPP_RUN_TESTS_AS_BUILD_STEP
with a default value of ON.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -4,9 +4,9 @@ project(UnitTest++)
 option(UTPP_USE_PLUS_SIGN
     "Set this to OFF if you wish to use '-cpp' instead of '++' in lib/include paths"
     ON)
-option(UTPP_SKIP_TESTS_AS_BUILD_STEP
-    "Set this to ON if you do not wish unit tests to run as part of cmake --build"
-    OFF)
+option(UTPP_RUN_TESTS_AS_BUILD_STEP
+    "Set this to OFF if you do not wish unit tests to run as part of cmake --build"
+    ON)
 
 if(MSVC14 OR MSVC12)
     # has the support we need
@@ -60,14 +60,14 @@ endif()
 
 target_link_libraries(TestUnitTest++ UnitTest++)
 
-if(${UTPP_SKIP_TESTS_AS_BUILD_STEP})
+if(${UTPP_RUN_TESTS_AS_BUILD_STEP})
+    # run unit tests as post build step
     add_custom_command(TARGET TestUnitTest++
-        POST_BUILD COMMAND echo "TestUnitTest++ was not run as a build step because UTPP_SKIP_TESTS_AS_BUILD_STEP is ON")
+        POST_BUILD COMMAND TestUnitTest++
+        COMMENT "Running unit tests")
 else()
-    # run unit tests as post build step
     add_custom_command(TARGET TestUnitTest++
-    	POST_BUILD COMMAND TestUnitTest++
-    	COMMENT "Running unit tests")
+        POST_BUILD COMMAND echo "TestUnitTest++ was not run as a build step because UTPP_RUN_TESTS_AS_BUILD_STEP is OFF")
 endif()
 
 # add install targets
