Comments on the attached patch?

I think it provides a little nicer default message when
the package cannot  be found.
Index: FindPackageHandleStandardArgs.cmake
===================================================================
--- FindPackageHandleStandardArgs.cmake	(revision 703829)
+++ FindPackageHandleStandardArgs.cmake	(working copy)
@@ -20,7 +20,11 @@
 MACRO(FIND_PACKAGE_HANDLE_STANDARD_ARGS _NAME _FAIL_MSG _VAR1 )
 
   IF("${_FAIL_MSG}" STREQUAL "DEFAULT_MSG")
-    SET(_FAIL_MESSAGE "Could NOT find ${_NAME}")
+    IF (${_NAME}_FIND_REQUIRED)
+      SET(_FAIL_MESSAGE "Could not find REQUIRED package ${_NAME}")
+    ELSE (${_NAME}_FIND_REQUIRED)
+      SET(_FAIL_MESSAGE "Could not find OPTIONAL package ${_NAME}")
+    ENDIF (${_NAME}_FIND_REQUIRED)
   ELSE("${_FAIL_MSG}" STREQUAL "DEFAULT_MSG")
     SET(_FAIL_MESSAGE "${_FAIL_MSG}")
   ENDIF("${_FAIL_MSG}" STREQUAL "DEFAULT_MSG")
_______________________________________________
Kde-buildsystem mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-buildsystem

Reply via email to