Fix

Author: mcbarton

.github/workflows/ci.yml

@@ -846,9 +846,7 @@ jobs:
                 ../
         fi
         os="${{ matrix.os }}"
-        cmake --build . --target check-cppinterop --parallel ${{ env.ncpus }} || true
-        cat /Users/runner/work/CppInterOp/CppInterOp/build/unittests/googletest-prefix/src/googletest-stamp/googletest-build-out.log
-        cat /Users/runner/work/CppInterOp/CppInterOp/build/unittests/googletest-prefix/src/googletest-stamp/googletest-build-err.log
+        cmake --build . --target check-cppinterop --parallel ${{ env.ncpus }} 
         if [[ ("${os}" == "ubuntu"*) ]]; then
           valgrind --show-error-list=yes --track-origins=yes --error-exitcode=1 unittests/CppInterOp/CppInterOpTests
         fi

cmake/modules/GoogleTest.cmake

@@ -20,10 +20,8 @@ elseif(APPLE)
   #       latest version of gtest on MacOS (use of -Wc++17-attribute-extensions causes 
   #       issues due to [[maybe_unused]] in gtest)
   set(EXTRA_GTEST_OPTS -DCMAKE_OSX_SYSROOT=${CMAKE_OSX_SYSROOT})
-  message(STATUS "CMAKE_CXX_FLAGS= ${CMAKE_CXX_FLAGS}")
   set(CMAKE_CXX_FLAGS_TEMP_COPY ${CMAKE_CXX_FLAGS})
   string(REPLACE "-pedantic" "" CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS}")
-  message(STATUS "CMAKE_CXX_FLAGS= ${CMAKE_CXX_FLAGS}")
 endif()
 
 include(ExternalProject)
@@ -57,10 +55,9 @@ ExternalProject_Add(
   TIMEOUT 600
   )
 
-#FIXME: Revert CMAKE_CXX_FLAGS back to include -Wall
+#FIXME: Related to fixme above which temporarily remodifies CMAKE_CXX_FLAGS
 if(APPLE)
   set(CMAKE_CXX_FLAGS ${CMAKE_CXX_FLAGS_TEMP_COPY})
-  message(STATUS "CMAKE_CXX_FLAGS= ${CMAKE_CXX_FLAGS}")
 endif()
 
 # Specify include dirs for gtest and gmock