commit 721b82f257afae0170a60ba355efceef6cff2355
author Rob Hughes <robert.hughes@arm.com> Fri Nov 15 09:04:17 2019 +0000
committer Derek Lamberti <derek.lamberti@arm.com> Fri Nov 15 13:10:56 2019 +0000
tree 8beb627dee8979e284210024c1bd329821174325
parent db1a2834b4e9d74ed538943634212eccbd4a789b

Print CMake messages on stdout rather than stderr

The default version of message("...") print to stderr, which is inappropriate
for informational messages such as the ones we are printing in these cases.

Using message(STATUS "...") makes these messages appear on stdout instead
which is more appropriate.

Change-Id: I02f41e6b4948e6938566f06d7164444bd5b8199e
Signed-off-by: Robert Hughes <robert.hughes@arm.com>

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 89f0e5a..fe3240d 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -518,7 +518,7 @@
 # object libs armnn to include in the build
 include(src/backends/backends.cmake)
 foreach(lib ${armnnLibraries})
-    message("Adding object library dependency to armnn: ${lib}")
+    message(STATUS "Adding object library dependency to armnn: ${lib}")
     list(APPEND armnn_sources $<TARGET_OBJECTS:${lib}>)
 endforeach()
 
@@ -826,7 +826,7 @@
     endif()
 
     foreach(lib ${armnnUnitTestLibraries})
-        message("Adding object library dependency to UnitTests: ${lib}")
+        message(STATUS "Adding object library dependency to UnitTests: ${lib}")
         list(APPEND unittest_sources $<TARGET_OBJECTS:${lib}>)
     endforeach()
 
@@ -838,7 +838,7 @@
 
     if(VALGRIND_FOUND)
         if(HEAP_PROFILING OR LEAK_CHECKING)
-            message("Valgrind is disabled for heap profiling and leak checking builds.")
+            message(STATUS "Valgrind is disabled for heap profiling and leak checking builds.")
         else()
             # Valgrind works with gperftools version number <= 2.4
             target_compile_definitions(UnitTests PRIVATE "WITH_VALGRIND=1")

cmake/GlobalConfig.cmake

diff --git a/cmake/GlobalConfig.cmake b/cmake/GlobalConfig.cmake
index af57948..191cd9d 100644
--- a/cmake/GlobalConfig.cmake
+++ b/cmake/GlobalConfig.cmake
@@ -328,7 +328,7 @@
             add_definitions("-DARMNN_LEAK_CHECKING_ENABLED=1")
         endif()
     else()
-        message("Heap profiling and leak checking are disabled for referencetests")
+        message(STATUS "Heap profiling and leak checking are disabled for referencetests")
     endif()
 else()
     # Valgrind only works with gperftools version number <= 2.4

src/backends/backends.cmake

diff --git a/src/backends/backends.cmake b/src/backends/backends.cmake
index 473de48..aca3b63 100644
--- a/src/backends/backends.cmake
+++ b/src/backends/backends.cmake
@@ -10,13 +10,13 @@
 
 # prefer to include common code first
 foreach(includeFile ${commonIncludes})
-    message("Including backend common library into the build: ${includeFile}")
+    message(STATUS "Including backend common library into the build: ${includeFile}")
     include(${includeFile})
 endforeach()
 
 # now backends can depend on common code included first
 foreach(includeFile ${backendIncludes})
-    message("Including backend into the build: ${includeFile}")
+    message(STATUS "Including backend into the build: ${includeFile}")
     include(${includeFile})
 endforeach()
 

src/backends/cl/backend.cmake

diff --git a/src/backends/cl/backend.cmake b/src/backends/cl/backend.cmake
index eb62147..9484b94 100644
--- a/src/backends/cl/backend.cmake
+++ b/src/backends/cl/backend.cmake
@@ -10,5 +10,5 @@
     list(APPEND armnnLibraries armnnClBackendWorkloads)
     list(APPEND armnnUnitTestLibraries armnnClBackendUnitTests)
 else()
-    message("CL backend is disabled")
+    message(STATUS "CL backend is disabled")
 endif()

src/backends/neon/backend.cmake

diff --git a/src/backends/neon/backend.cmake b/src/backends/neon/backend.cmake
index c8ff488..801ba7c 100644
--- a/src/backends/neon/backend.cmake
+++ b/src/backends/neon/backend.cmake
@@ -10,5 +10,5 @@
     list(APPEND armnnLibraries armnnNeonBackendWorkloads)
     list(APPEND armnnUnitTestLibraries armnnNeonBackendUnitTests)
 else()
-    message("NEON backend is disabled")
+    message(STATUS "NEON backend is disabled")
 endif()

src/backends/reference/backend.cmake

diff --git a/src/backends/reference/backend.cmake b/src/backends/reference/backend.cmake
index 01a4ee1..e8654d8 100644
--- a/src/backends/reference/backend.cmake
+++ b/src/backends/reference/backend.cmake
@@ -10,5 +10,5 @@
     list(APPEND armnnLibraries armnnRefBackendWorkloads)
     list(APPEND armnnUnitTestLibraries armnnRefBackendUnitTests)
 else()
-    message("Reference backend is disabled")
+    message(STATUS "Reference backend is disabled")
 endif()