kpvdr commented on a change in pull request #186: PROTON-2086: Changed API documentation from epydoc to Sphinx URL: https://github.com/apache/qpid-proton/pull/186#discussion_r315698367
########## File path: python/CMakeLists.txt ########## @@ -112,35 +112,46 @@ endmacro(py_compile) py_compile(${CMAKE_CURRENT_BINARY_DIR} ${pysrc-generated} CPROTON_ARTIFACTS) py_compile(${CMAKE_CURRENT_SOURCE_DIR} "${pysrc}" PROTON_ARTIFACTS) -find_program(EPYDOC_EXE epydoc) -mark_as_advanced (EPYDOC_EXE) -if (EPYDOC_EXE) - foreach (py_src_doc ${pysrc}) - list(APPEND PY_DOC_FILES "${CMAKE_CURRENT_SOURCE_DIR}/${py_src_doc}") - endforeach(py_src_doc) + +find_program(SPHINX_EXE_PY2 sphinx-build-2 HINTS "/usr/bin") +mark_as_advanced (SPHINX_EXE_PY2) +find_program(SPHINX_EXE_PY3 sphinx-build-3 HINTS "/usr/bin") +mark_as_advanced (SPHINX_EXE_PY3) + +# Version of Python used to build API must match version of Sphinx used to build the docs +get_filename_component(PYTHON_EXECUTABLE_NAME ${PYTHON_EXECUTABLE} NAME) +# Try matching for Python 3 first +if ((SPHINX_EXE_PY3) AND (PYTHON_EXECUTABLE_NAME STREQUAL "python3")) + set(BUILD_SPHINX_DOCS TRUE) Review comment: ``` $ sudo yum list python?-sphinx* Installed Packages python3-sphinx.noarch 1:1.8.4-1.fc30 @fedora python3-sphinx-theme-alabaster.noarch 0.7.12-3.fc30 @fedora python3-sphinx_rtd_theme.noarch 0.4.3-1.fc30 @fedora python3-sphinxcontrib-apidoc.noarch 0.2.1-10.fc30 @fedora python3-sphinxcontrib-websupport.noarch 1.0.1-12.20180316git.fc30 @fedora ``` I repeated the test by: 1. Erasing the python2-sphinx package (`dnf erase python2-sphinx`) 2. Running `cmake ..` from the build directory 3. Running `make clean all docs` but it produced the same results as above. I'll start some debugging... ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org