[PATCH] D149809: [Clang][Docs] Fix man page build

Aiden Grossman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 3 19:26:53 PDT 2023


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



More information about the cfe-commits mailing list