Fix Doxygen workflow (#4372)

.github/workflows/doxygen.yml

@@ -1,4 +1,5 @@
 name: Build and publish Doxygen documentation
+# To test this workflow, push your changes to your fork's `develop` branch.
 on:
   push:
     branches:
@@ -11,12 +12,18 @@ jobs:
       image: docker://rippleci/rippled-ci-builder:2944b78d22db
     steps:
       - name: checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v3
+      - name: check environment
+        run: |
+          echo ${PATH} | tr ':' '\n'
+          cmake --version
+          doxygen --version
+          env
       - name: build
         run: |
           mkdir build
           cd build
-          cmake -DBoost_NO_BOOST_CMAKE=ON ..
+          cmake -Donly_docs=TRUE ..
           cmake --build . --target docs --parallel $(nproc)
       - name: publish
         uses: peaceiris/actions-gh-pages@v3

.github/workflows/nix.yml

@@ -74,7 +74,7 @@ jobs:
         run: |
           mkdir ${build_dir}
           cd ${build_dir}
-          conan install .. --build missing --settings build_type=${{ matrix.configuration }} --profile:build default --profile:host default
+          conan install .. --build missing --settings build_type=${{ matrix.configuration }}
       - name: configure
         run: |
           cd ${build_dir}

Builds/CMake/RippledDocs.cmake

@@ -2,25 +2,24 @@
    docs target (optional)
 #]===================================================================]
 
-# Early return if the `docs` directory is missing,
-# e.g. when we are building a Conan package.
-if(NOT EXISTS docs)
+option(with_docs "Include the docs target?" FALSE)
+
+if(NOT (with_docs OR only_docs))
   return()
 endif()
 
-if (tests)
-  find_package (Doxygen)
-  if (NOT TARGET Doxygen::doxygen)
-    message (STATUS "doxygen executable not found -- skipping docs target")
-    return ()
-  endif ()
+find_package(Doxygen)
+if(NOT TARGET Doxygen::doxygen)
+  message(STATUS "doxygen executable not found -- skipping docs target")
+  return()
+endif()
 
-  set (doxygen_output_directory "${CMAKE_BINARY_DIR}/docs")
-  set (doxygen_include_path "${CMAKE_CURRENT_SOURCE_DIR}/src")
-  set (doxygen_index_file "${doxygen_output_directory}/html/index.html")
-  set (doxyfile "${CMAKE_CURRENT_SOURCE_DIR}/docs/Doxyfile")
+set(doxygen_output_directory "${CMAKE_BINARY_DIR}/docs")
+set(doxygen_include_path "${CMAKE_CURRENT_SOURCE_DIR}/src")
+set(doxygen_index_file "${doxygen_output_directory}/html/index.html")
+set(doxyfile "${CMAKE_CURRENT_SOURCE_DIR}/docs/Doxyfile")
 
-  file (GLOB_RECURSE doxygen_input
+file(GLOB_RECURSE doxygen_input
   docs/*.md
   src/ripple/*.h
   src/ripple/*.cpp
@@ -28,48 +27,48 @@ if (tests)
   src/test/*.h
   src/test/*.md
   Builds/*/README.md)
-  list (APPEND doxygen_input
+list(APPEND doxygen_input
   README.md
   RELEASENOTES.md
   src/README.md)
-  set (dependencies "${doxygen_input}" "${doxyfile}")
+set(dependencies "${doxygen_input}" "${doxyfile}")
 
-  function (verbose_find_path variable name)
+function(verbose_find_path variable name)
   # find_path sets a CACHE variable, so don't try using a "local" variable.
-    find_path (${variable} "${name}" ${ARGN})
-    if (NOT ${variable})
-      message (NOTICE "could not find ${name}")
-    else ()
-      message (STATUS "found ${name}: ${${variable}}/${name}")
-    endif ()
-  endfunction ()
+  find_path(${variable} "${name}" ${ARGN})
+  if(NOT ${variable})
+    message(NOTICE "could not find ${name}")
+  else()
+    message(STATUS "found ${name}: ${${variable}}/${name}")
+  endif()
+endfunction()
 
-  verbose_find_path (doxygen_plantuml_jar_path plantuml.jar PATH_SUFFIXES share/plantuml)
-  verbose_find_path (doxygen_dot_path dot)
+verbose_find_path(doxygen_plantuml_jar_path plantuml.jar PATH_SUFFIXES share/plantuml)
+verbose_find_path(doxygen_dot_path dot)
 
-  # https://en.cppreference.com/w/Cppreference:Archives
-  # https://stackoverflow.com/questions/60822559/how-to-move-a-file-download-from-configure-step-to-build-step
-  set (download_script "${CMAKE_BINARY_DIR}/docs/download-cppreference.cmake")
-  file (WRITE
+# https://en.cppreference.com/w/Cppreference:Archives
+# https://stackoverflow.com/questions/60822559/how-to-move-a-file-download-from-configure-step-to-build-step
+set(download_script "${CMAKE_BINARY_DIR}/docs/download-cppreference.cmake")
+file(WRITE
   "${download_script}"
-    "file (DOWNLOAD \
+  "file(DOWNLOAD \
     http://upload.cppreference.com/mwiki/images/b/b2/html_book_20190607.zip \
     ${CMAKE_BINARY_DIR}/docs/cppreference.zip \
     EXPECTED_HASH MD5=82b3a612d7d35a83e3cb1195a63689ab \
   )\n \
-    execute_process ( \
+  execute_process( \
     COMMAND \"${CMAKE_COMMAND}\" -E tar -xf cppreference.zip \
   )\n"
-  )
-  set (tagfile "${CMAKE_BINARY_DIR}/docs/cppreference-doxygen-web.tag.xml")
-  add_custom_command (
+)
+set(tagfile "${CMAKE_BINARY_DIR}/docs/cppreference-doxygen-web.tag.xml")
+add_custom_command(
   OUTPUT "${tagfile}"
   COMMAND "${CMAKE_COMMAND}" -P "${download_script}"
   WORKING_DIRECTORY "${CMAKE_BINARY_DIR}/docs"
-  )
-  set (doxygen_tagfiles "${tagfile}=http://en.cppreference.com/w/")
+)
+set(doxygen_tagfiles "${tagfile}=http://en.cppreference.com/w/")
 
-  add_custom_command (
+add_custom_command(
   OUTPUT "${doxygen_index_file}"
   COMMAND "${CMAKE_COMMAND}" -E env
     "DOXYGEN_OUTPUT_DIRECTORY=${doxygen_output_directory}"
@@ -80,7 +79,6 @@ if (tests)
     "${DOXYGEN_EXECUTABLE}" "${doxyfile}"
   WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
   DEPENDS "${dependencies}" "${tagfile}")
-  add_custom_target (docs
+add_custom_target(docs
   DEPENDS "${doxygen_index_file}"
   SOURCES "${dependencies}")
-endif ()

CMakeLists.txt

@@ -58,6 +58,12 @@ endif ()
 include(RippledCompiler)
 include(RippledInterface)
 
+option(only_docs "Include only the docs target?" FALSE)
+include(RippledDocs)
+if(only_docs)
+  return()
+endif()
+
 ###
 
 include(deps/Boost)
@@ -119,5 +125,4 @@ include(RippledCore)
 include(RippledInstall)
 include(RippledCov)
 include(RippledMultiConfig)
-include(RippledDocs)
 include(RippledValidatorKeys)