Merge pull request #812 from zpalmtree/fix-default-build-type

Fix cmake not defaulting to release build

CMakeLists.txt

@@ -7,6 +7,9 @@ if(CCACHE_PROGRAM)
   set_property(GLOBAL PROPERTY RULE_LAUNCH_COMPILE "${CCACHE_PROGRAM}")
 endif()
 
+set(CMAKE_BUILD_TYPE Release CACHE STRING "Choose the type of build, options are: Debug, Release, RelWithDebInfo")
+set(CMAKE_CONFIGURATION_TYPES Debug RelWithDebInfo Release CACHE TYPE INTERNAL)
+
 project(TurtleCoin)
 
 # Required for finding Threads on ARM
@@ -39,8 +42,6 @@ set(VERSION "0.1")
 
 ## This section describes our general CMake setup options
 set_property(GLOBAL PROPERTY USE_FOLDERS ON)
-set(CMAKE_CONFIGURATION_TYPES Debug RelWithDebInfo Release CACHE TYPE INTERNAL)
-set(CMAKE_BUILD_TYPE Release CACHE STRING "Choose the type of build, options are: Debug, Release, RelWithDebInfo")
 set(CMAKE_SKIP_INSTALL_RULES OFF FORCE)
 set(CMAKE_SKIP_PACKAGE_ALL_DEPENDENCY ON FORCE)
 set(CMAKE_SUPPRESS_REGENERATION ON)

src/CryptoTest/main.cpp

@@ -2,8 +2,6 @@
 //
 // Please see the included LICENSE file for more information.
 
-#undef NDEBUG
-
 #include <iostream>
 #include <chrono>
 #include <assert.h>
@@ -167,7 +165,13 @@ void testHashFunction(
     }
 
     /* Verify the hash is as expected */
-    assert(CompareHashes(hash, expectedOutput));
+    if(!CompareHashes(hash, expectedOutput))
+    {
+        std::cout << "Hashes are not equal!\n" << "Expected: " << expectedOutput << "\nActual: " << hash
+                  << "\nTerminating.";
+
+        exit(1);
+    }
 }
 
 /* Bit of hackery so we can get the variable name of the passed in function.