Skip to content

Commit 764c7ae

Browse files
[Github] Remove CMake options from docs CI resetting defaults
The SPHINX_OUTPUT_HTML and SPHINX_OUTPUT_MAN CMake flags are by default set to ON/True in ./llvm/cmake/modules/FindSphinx.cmake, so we don't need to set them true again within the CMake invocation.
1 parent 09b02f5 commit 764c7ae

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.github/workflows/docs.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,11 +62,11 @@ jobs:
6262
- name: Build LLVM docs
6363
if: steps.docs-changed-subprojects.outputs.llvm_any_changed == 'true'
6464
run: |
65-
cmake -B llvm-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_SPHINX=ON -DSPHINX_OUTPUT_HTML=ON -DSPHINX_OUTPUT_MAN=ON ./llvm
65+
cmake -B llvm-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_SPHINX=ON ./llvm
6666
TZ=UTC ninja -C llvm-build docs-llvm-html docs-llvm-man
6767
- name: Build Clang docs
6868
if: steps.docs-changed-subprojects.outputs.clang_any_changed == 'true'
6969
run: |
70-
cmake -B clang-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PROJECTS="clang" -DLLVM_ENABLE_SPHINX=ON -DSPHINX_OUTPUT_HTML=ON -DSPHINX_OUTPUT_MAN=ON ./llvm
70+
cmake -B clang-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_PROJECTS="clang" -DLLVM_ENABLE_SPHINX=ON ./llvm
7171
TZ=UTC ninja -C clang-build docs-clang-html docs-clang-man
7272

0 commit comments

Comments
 (0)