This is an archive of the discontinued LLVM Phabricator instance.

[NFC][libunbind] Fix Sphinx error during CMake invocation
ClosedPublic

Authored by xgupta on Feb 4 2021, 9:50 PM.

Details

Summary

An error has occurred when I build libunwind with -DLLVM_BUILD_DOCS=ON.

Diff Detail

Event Timeline

xgupta created this revision.Feb 4 2021, 9:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 4 2021, 9:50 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
xgupta requested review of this revision.Feb 4 2021, 9:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 4 2021, 9:50 PM
xgupta added a comment.Feb 4 2021, 9:51 PM

Error Message was:

CMake Error at /home/xgupta/dev/llvm-project/llvm/cmake/modules/AddSphinxTarget.cmake:71 (add_dependencies):

Cannot add target-level dependencies to non-existent target "sphinx".

The add_dependencies works for top-level logical targets created by the
add_executable, add_library, or add_custom_target commands.  If you want to
add file-level dependencies see the DEPENDS option of the add_custom_target
and add_custom_command commands.

Call Stack (most recent call first):

/home/xgupta/dev/llvm-project/libunwind/docs/CMakeLists.txt:5 (add_sphinx_target)
compnerd accepted this revision.Feb 10 2021, 3:31 PM
compnerd added a subscriber: compnerd.

I suppose that this makes it difficult to generate them when libunwind is built standalone, but, as it is, there are other dependencies involved.

This revision is now accepted and ready to land.Feb 10 2021, 3:31 PM
This revision was automatically updated to reflect the committed changes.