aidengrossman added reviewers: aaron.ballman, tstellar, rnk. aidengrossman added a comment.
Posting this as when I was setting up a new dev environment with `-DLLVM_ENABLE_SPHINX`, it defaults to having `SPHINX_OUTPUT_HTML` and `SPHINX_OUTPUT_MAN` on, and the man build was broken so when I tried to build the default target, it failed. This patch fixes the clang man page build. Not sure this is the ideal approach. I'd really like to remove the redundancy between the generated targets reference in `gen_docs_depends` and their values as specified in the calls to `gen_rst_files_from_td`, but CMake doesn't really seem to have appropriate data structures (a list of tuples or something similar) to accomplish this. Very open to suggestions here though since I'm not that familiar with CMake. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D149809/new/ https://reviews.llvm.org/D149809 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
