From 5a34e72d9f113eb5d028e740c2def1f944619595 Mon Sep 17 00:00:00 2001
From: Paul Fultz II <pfultz2@yahoo.com>
Date: Tue, 12 Sep 2023 13:50:30 -0500
Subject: [PATCH] Dont require sphinx to be installed (#144)

* Dont require sphinx to be installed

* Dont require doxygen

* Fix doxygen variable name
---
 share/rocm/cmake/ROCMSphinxDoc.cmake | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/share/rocm/cmake/ROCMSphinxDoc.cmake b/share/rocm/cmake/ROCMSphinxDoc.cmake
index 70b509c8..a4f0f618 100755
--- a/share/rocm/cmake/ROCMSphinxDoc.cmake
+++ b/share/rocm/cmake/ROCMSphinxDoc.cmake
@@ -12,8 +12,7 @@ find_program(
     NAMES sphinx-build
     HINTS "$ENV{SPHINX_DIR}"
     PATH_SUFFIXES bin
-    DOC "Sphinx documentation generator"
-    REQUIRED)
+    DOC "Sphinx documentation generator")
 
 find_program(
     DOXYGEN_EXECUTABLE
@@ -61,14 +60,10 @@ function(rocm_add_sphinx_doc SRC_DIR)
         list(APPEND VARS -A "${VAR}")
     endforeach()
 
-    if(PARSE_USES_DOXYGEN)
-        if(NOT DOXYGEN_EXECUTABLE)
-            message(FATAL_ERROR
-                "rocm_add_sphinx_doc has USES_DOXYGEN set but DOXYGEN_EXECUTABLE is set to ${DOXYGEN_EXECUTABLE}.")
-        endif()
-        set(USES_DOXYGEN -D "doxygen_executable=${DOXYGEN_EXECUTABLE}")
+    if(PARSE_USE_DOXYGEN)
+        set(USE_DOXYGEN -D "doxygen_executable=${DOXYGEN_EXECUTABLE}")
     else()
-        set(USES_DOXYGEN)
+        set(USE_DOXYGEN)
     endif()
 
     if(NOT TARGET sphinx-${BUILDER})
@@ -81,7 +76,7 @@ function(rocm_add_sphinx_doc SRC_DIR)
             "${SPHINX_EXECUTABLE}"
             -b ${PARSE_BUILDER}
             -d "${CMAKE_CURRENT_BINARY_DIR}/doctrees"
-            ${USES_DOXYGEN}
+            ${USE_DOXYGEN}
             ${VARS}
             "${SRC_DIR}"
             "${OUTPUT_DIR}"