Fix option parsing issues in function

Author: buddly27

cmake/FindSphinx.cmake

@@ -54,13 +54,15 @@ if (Sphinx_FOUND AND NOT TARGET Sphinx::Build)
         cmake_parse_arguments(
             PARSE_ARGV 1 ""
             "ALL;SHOW_TRACEBACK;WRITE_ALL;FRESH_ENV;ISOLATED"
-            "BUILDER;CONFIG_DIRECTORY;SOURCE_DIRECTORY;OUTPUT_DIRECTORY"
+            "COMMENT;BUILDER;CONFIG_DIRECTORY;SOURCE_DIRECTORY;OUTPUT_DIRECTORY"
             "DEFINE;DEPENDS")
 
         # Ensure that target should be added to the default build target,
         # if required.
         if(_ALL)
-            set(_ALL ALL)
+            set(_ALL "ALL")
+        else()
+            set(_ALL "")
         endif()
 
         # Default working directory to current source path if none is provided.

doc/release/release_notes.rst

@@ -4,6 +4,17 @@
 Release Notes
 *************
 
+.. release:: Upcoming
+
+    .. change:: fixed
+
+        Fixed the :func:`sphinx_add_docs` to prevent setting incorrect
+        attribute when the "ALL" option is not set.
+
+    .. change:: fixed
+
+        Fixed the :func:`sphinx_add_docs` to add missing "COMMENT" option.
+
 .. release:: 0.2.0
     :date: 2024-08-09