[ https://issues.apache.org/jira/browse/PROTON-2086?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16911335#comment-16911335 ]
ASF GitHub Bot commented on PROTON-2086: ---------------------------------------- kpvdr commented on pull request #186: PROTON-2086: Changed API documentation from epydoc to Sphinx URL: https://github.com/apache/qpid-proton/pull/186#discussion_r315683102 ########## 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: I should have been more precise describing how this error occurred. I did include this section. The `python3-sphinx` module was installed (and found), but not the Python 2 version. The `SPHINX_MODULE_FOUND` variable returns `true`. However, when building qpid-proton with the (default) Python2, this error occurs. If we could find a way to distinguish between Python2 and Python3 versions of Sphinx, then logic can be used to prevent and error. ---------------------------------------------------------------- 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 > Move Python API docs to use Sphinx > ---------------------------------- > > Key: PROTON-2086 > URL: https://issues.apache.org/jira/browse/PROTON-2086 > Project: Qpid Proton > Issue Type: Task > Components: python-binding > Reporter: Kim van der Riet > Assignee: Kim van der Riet > Priority: Major > > Currently the Python API docs are generated using ePyDoc. These should be > switched to using Sphinx. -- This message was sent by Atlassian Jira (v8.3.2#803003) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org