BUG: Fixes Issue #8054 and more. DOXYGEN_DOT_FOUND now exists, errant

mark_as_advanced variables removed, documentation cleaned up and OSX stuff
isolated to it's own section, support added for DOXYGEN_SKIP_DOT, support
added to call FindPackageHandleStandardArgs to avoid output on every CMake
run.

Modules/FindDoxygen.cmake

@@ -1,15 +1,25 @@
 # - This module looks for Doxygen and the path to Graphviz's dot
 # Doxygen is a documentation generation tool see http://www.doxygen.org
+#
+# This module accepts the following optional variables:
+#
+#   DOXYGEN_SKIP_DOT       = If true this module will skip trying to find Dot
+#
+# This modules defines the following variables:
+#
+#   DOXYGEN_EXECUTABLE     = The path to the doxygen command.
+#   DOXYGEN_FOUND          = Was Doxygen found or not?
+#
+#   DOXYGEN_DOT_EXECUTABLE = The path to the dot program used by doxygen.
+#   DOXYGEN_DOT_FOUND      = Was Dot found or not?
+#   DOXYGEN_DOT_PATH       = The path to dot not including the executable
+#
+# Details for OSX Users:
 #     With the OS X GUI version, it likes to be installed to /Applications and
 #     it contains the doxygen executable in the bundle. In the versions I've 
 #     seen, it is located in Resources, but in general, more often binaries are 
-# located in MacOS. This code sets the following variables:
+#     located in MacOS.
-#  DOXYGEN_EXECUTABLE     = The path to the doxygen command.
+#
-#  DOXYGEN_DOT_EXECUTABLE = The path to the dot program used by doxygen.
-#  DOXYGEN_DOT_PATH       = The path to dot not including the executable
-#  DOXYGEN = same as DOXYGEN_EXECUTABLE for backwards compatibility
-#  DOT = same as DOXYGEN_DOT_EXECUTABLE for backwards compatibility
-
 #     The official Doxygen.app that is distributed for OS X uses non-standard 
 #     conventions. Instead of the command-line "doxygen" tool being placed in
 #     Doxygen.app/Contents/MacOS, "Doxywizard" is placed there instead and 
@@ -19,10 +29,19 @@
 #     by this. Once CMake sees the bundle, it indiscrimately uses Doxywizard
 #     as the executable to use. The only work-around I have found is to disable
 #     the app-bundle feature for only this command.
-# Save the old setting
+if(APPLE)
-SET(TEMP_DOXYGEN_SAVE_CMAKE_FIND_APPBUNDLE ${CMAKE_FIND_APPBUNDLE})
+    #  Save the old setting
-# Disable the App-bundle detection feature
+    SET(TEMP_DOXYGEN_SAVE_CMAKE_FIND_APPBUNDLE ${CMAKE_FIND_APPBUNDLE})
-SET(CMAKE_FIND_APPBUNDLE "NEVER")
+    # Disable the App-bundle detection feature
+    SET(CMAKE_FIND_APPBUNDLE "NEVER")
+endif()
+#     FYI:
+#     In the older versions of OS X Doxygen, dot was included with the 
+#     Doxygen bundle. But the new versions place make you download Graphviz.app
+#     which contains dot in its bundle.
+# ============== End OSX stuff ================
+
+
 # For backwards compatibility support
 # DOXYGEN_FIND_QUIETLY, but it should have been
 # Doxygen_FIND_QUIETLY.  
@@ -30,43 +49,31 @@ IF(Doxygen_FIND_QUIETLY)
   SET(DOXYGEN_FIND_QUIETLY TRUE)
 ENDIF(Doxygen_FIND_QUIETLY)
 
-IF (NOT DOXYGEN_FIND_QUIETLY)
+#
-  MESSAGE(STATUS "Looking for doxygen...")
+# Find Doxygen...
-ENDIF (NOT DOXYGEN_FIND_QUIETLY)
+#
 
 FIND_PROGRAM(DOXYGEN_EXECUTABLE
   NAMES doxygen
-  PATHS "[HKEY_LOCAL_MACHINE\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\doxygen_is1;Inno Setup: App Path]/bin"
+  PATHS
+    "[HKEY_LOCAL_MACHINE\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\doxygen_is1;Inno Setup: App Path]/bin"
     /Applications/Doxygen.app/Contents/Resources
     /Applications/Doxygen.app/Contents/MacOS
   DOC "Doxygen documentation generation tool (http://www.doxygen.org)"
 )
 
-IF (DOXYGEN_EXECUTABLE)
+include(FindPackageHandleStandardArgs)
-  SET (DOXYGEN_FOUND "YES")
+FIND_PACKAGE_HANDLE_STANDARD_ARGS(DOXYGEN DEFAULT_MSG DOXYGEN_EXECUTABLE)
-  IF (NOT DOXYGEN_FIND_QUIETLY)
-    MESSAGE(STATUS "Looking for doxygen... - found ${DOXYGEN_EXECUTABLE}")
-  ENDIF (NOT DOXYGEN_FIND_QUIETLY)
-ELSE (DOXYGEN_EXECUTABLE)
-  IF (NOT DOXYGEN_FIND_QUIETLY)
-    IF (DOXYGEN_FIND_REQUIRED)
-      MESSAGE(FATAL_ERROR "Looking for doxygen... - NOT found")
-    ELSE (DOXYGEN_FIND_REQUIRED)
-      MESSAGE(STATUS "Looking for doxygen... - NOT found")
-    ENDIF (DOXYGEN_FIND_REQUIRED)
-  ENDIF (NOT DOXYGEN_FIND_QUIETLY)
-ENDIF (DOXYGEN_EXECUTABLE)
 
-# In the older versions of OS X Doxygen, dot was included with the 
+#
-# Doxygen bundle. But the new versions place make you download Graphviz.app
+# Find Dot...
-# which contains dot in its bundle.
+#
-IF (NOT DOXYGEN_FIND_QUIETLY)
-  MESSAGE(STATUS "Looking for dot tool...")
-ENDIF (NOT DOXYGEN_FIND_QUIETLY)
 
-FIND_PROGRAM(DOXYGEN_DOT_EXECUTABLE
+IF(NOT DOXYGEN_SKIP_DOT)
+  FIND_PROGRAM(DOXYGEN_DOT_EXECUTABLE
     NAMES dot
-  PATHS "$ENV{ProgramFiles}/Graphviz 2.21/bin"
+    PATHS 
+      "$ENV{ProgramFiles}/Graphviz 2.21/bin"
       "C:/Program Files/Graphviz 2.21/bin"
       "$ENV{ProgramFiles}/ATT/Graphviz/bin"
       "C:/Program Files/ATT/Graphviz/bin"
@@ -75,30 +82,46 @@ FIND_PROGRAM(DOXYGEN_DOT_EXECUTABLE
       /Applications/Doxygen.app/Contents/Resources
       /Applications/Doxygen.app/Contents/MacOS
     DOC "Graphviz Dot tool for using Doxygen"
-)
+  )
   
-IF (NOT DOXYGEN_FIND_QUIETLY)
+  include(FindPackageHandleStandardArgs)
-  IF (DOXYGEN_DOT_EXECUTABLE)
+  FIND_PACKAGE_HANDLE_STANDARD_ARGS(DOXYGEN_DOT DEFAULT_MSG DOXYGEN_DOT_EXECUTABLE)
-    MESSAGE(STATUS "Looking for dot tool... - found ${DOXYGEN_DOT_EXECUTABLE}")
+  
+  if(DOXYGEN_DOT_EXECUTABLE)
     # The Doxyfile wants the path to Dot, not the entire path and executable
-    GET_FILENAME_COMPONENT(DOXYGEN_DOT_PATH "${DOXYGEN_DOT_EXECUTABLE}" PATH CACHE)
+    get_filename_component(DOXYGEN_DOT_PATH "${DOXYGEN_DOT_EXECUTABLE}" PATH CACHE)
-  ELSE (DOXYGEN_DOT_EXECUTABLE)
+  endif()
-    MESSAGE(STATUS "Looking for dot tool... - NOT found")
-  ENDIF (DOXYGEN_DOT_EXECUTABLE)
-ENDIF (NOT DOXYGEN_FIND_QUIETLY)
   
+endif(NOT DOXYGEN_SKIP_DOT)
 
-# Restore the old app-bundle setting setting
+#
-SET(CMAKE_FIND_APPBUNDLE ${TEMP_DOXYGEN_SAVE_CMAKE_FIND_APPBUNDLE})
+# Backwards compatibility...
+#
+
+if(APPLE)
+  # Restore the old app-bundle setting setting
+  SET(CMAKE_FIND_APPBUNDLE ${TEMP_DOXYGEN_SAVE_CMAKE_FIND_APPBUNDLE})
+endif()
+
+# Maintain the _FOUND variables as "YES" or "NO" for backwards compatibility
+# (allows people to stuff them directly into Doxyfile with configure_file())
+if(DOXYGEN_FOUND)
+  set(DOXYGEN_FOUND "YES")
+else()
+  set(DOXYGEN_FOUND "NO")
+endif()
+if(DOXYGEN_DOT_FOUND)
+  set(DOXYGEN_DOT_FOUND "YES")
+else()
+  set(DOXYGEN_DOT_FOUND "NO")
+endif()
 
 # Backwards compatibility for CMake4.3 and less
 SET (DOXYGEN ${DOXYGEN_EXECUTABLE} )
 SET (DOT ${DOXYGEN_DOT_EXECUTABLE} )
 
 MARK_AS_ADVANCED(
-  DOXYGEN_FOUND
   DOXYGEN_EXECUTABLE
-  DOXYGEN_DOT_FOUND
   DOXYGEN_DOT_EXECUTABLE
   DOXYGEN_DOT_PATH
   )