Kornel,

Would it be ok to add the STATUS 'tag' to the messages in the script.

When you happen to use the cmake-gui, you're confronted with all messages that don't have this 'tag'. That's a bit annoying when it is an empty string.. :S...

See attached patch, ok ?

Vincent
Index: development/cmake/CMakeLists.txt
===================================================================
--- development/cmake/CMakeLists.txt    (revision 28570)
+++ development/cmake/CMakeLists.txt    (working copy)
@@ -66,11 +66,11 @@
 endforeach(_c_l)
 
 if(UseVersionSuffix)
-  message("-- Using versioned PACKAGE. Disable with -DUseVersionSuffix=0")
+  message(STATUS "-- Using versioned PACKAGE. Disable with 
-DUseVersionSuffix=0")
   set(PACKAGE ${PACKAGE_BASE}${LYX_INSTALL_SUFFIX})
   set(PROGRAM_SUFFIX "${LYX_INSTALL_SUFFIX}")
 else()
-  message("-- PACKAGE not versioned, to enable use -DUseVersionSuffix=1")
+  message(STATUS "-- PACKAGE not versioned, to enable use 
-DUseVersionSuffix=1")
   set(PACKAGE ${PACKAGE_BASE})
   set(PROGRAM_SUFFIX "")
 endif()
@@ -194,20 +194,20 @@
        add_definitions(-DHAVE_GETTEXT)
 endif()
 
-message("")
+message(STATUS "")
 if(nls OR all)
        set(nls TRUE CACHE TYPE STRING)
        add_definitions(-DENABLE_NLS=1)
-       message("----- Building with ENABLE_NLS")
+       message(STATUS "----- Building with ENABLE_NLS")
 else() 
-       message("----- No nls, to enable use -Dnls=1")
+       message(STATUS "----- No nls, to enable use -Dnls=1")
 endif()
 set(nls)
 if(ASPELL_FOUND)
        add_definitions(-DUSE_ASPELL=1)
-       message("----- Building with USE_ASPELL")
+       message(STATUS "----- Building with USE_ASPELL")
 else() 
-       message("----- No aspell, to get more information use -Daspell=1")
+       message(STATUS "----- No aspell, to get more information use 
-Daspell=1")
 endif()
 set(all)
 
@@ -216,9 +216,9 @@
                set(noconsole TRUE CACHE TYPE STRING)
                set(WIN32_CONSOLE WIN32)
                set(LYX_QTMAIN_LIBRARY ${QT_QTMAIN_LIBRARY})
-               message("----- Console disabled")
+               message(STATUS "----- Console disabled")
        else()
-               message("----- Console enabled, disable it with -Dnoconsole=1")
+               message(STATUS "----- Console enabled, disable it with 
-Dnoconsole=1")
        endif()
        if(MSVC)
                add_definitions(-D_CRT_SECURE_NO_DEPRECATE 
-D_CRT_NONSTDC_NO_DEPRECATE -DNOMINMAX)      
@@ -229,7 +229,7 @@
        endif()
 endif()
 
-message("")
+message(STATUS "")
 foreach(_v PACKAGE PACKAGE_VERSION
     PROGRAM_SUFFIX LYX_DATE LYX_DIR_VER LYX_USERDIR_VER
     LYX_ABS_TOP_SRCDIR
@@ -238,17 +238,17 @@
     message(FATAL_ERROR "${_v} not defined")
   endif()
   string(SUBSTRING "${_v}                            " 0 28 _v1)
-  message("----- ${_v1}: ${${_v}}")
+  message(STATUS "----- ${_v1}: ${${_v}}")
 endforeach(_v)
-message("")
+message(STATUS "")
 
 
 add_definitions(-DBOOST_USER_CONFIG="<config.h>")
        
 if(MSVC AND NOT disable-pch AND NOT MERGE_FILES)
-       message("-----")
-       message("----- using precompiled headers, disable with -Ddisable-pch=1")
-       message("-----")
+       message(STATUS "-----")
+       message(STATUS "----- using precompiled headers, disable with 
-Ddisable-pch=1")
+       message(STATUS "-----")
        configure_file(${CMAKE_SOURCE_DIR}/pcheaders.h
                ${CMAKE_BINARY_DIR}/pcheaders.h)
        configure_file(${CMAKE_SOURCE_DIR}/config.cpp.cmake
@@ -270,9 +270,9 @@
                "${CMAKE_CXX_FLAGS_MINSIZEREL} /DLYX_PCH_STL /DLYX_PCH_BOOST")
 else()
        if(MSVC)
-               message("-----")
-               message("----- precompiled headers disabled, enable with 
-Ddisable-pch=0")
-               message("-----")
+               message(STATUS "-----")
+               message(STATUS "----- precompiled headers disabled, enable with 
-Ddisable-pch=0")
+               message(STATUS "-----")
        endif()
        set(disable-pch TRUE CACHE TYPE STRING)
        macro(lyx_add_msvc_pch)
@@ -348,10 +348,10 @@
                
        endif()
        
-       message("----- Warning level      : ${CMAKE_CXX_WARNING_LEVEL}  
${WARNING_LEVEL_MESSAGE}")
-       message("----- Warnings as errors : ${MSVC_W_ERROR}")
-       message("----- Warnings disabled  : ${MSVC_W_DISABLE}")
-       message("")
+       message(STATUS "----- Warning level      : ${CMAKE_CXX_WARNING_LEVEL}  
${WARNING_LEVEL_MESSAGE}")
+       message(STATUS "----- Warnings as errors : ${MSVC_W_ERROR}")
+       message(STATUS "----- Warnings disabled  : ${MSVC_W_DISABLE}")
+       message(STATUS "")
        
 endif()
 

Reply via email to