Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[libc++][CI] Fixes documentation builder. #78327

Merged
merged 1 commit into from
Jan 16, 2024
Merged

Conversation

mordante
Copy link
Member

@mordante mordante commented Jan 16, 2024

The documentation CI no longer builds. This is likely introduced by
8f90e69. This fixes the issue.

@mordante mordante marked this pull request as ready for review January 16, 2024 19:02
@mordante mordante merged commit 7e909d5 into llvm:main Jan 16, 2024
6 checks passed
@mordante mordante deleted the fix_doc_ci branch January 16, 2024 19:02
@llvmbot
Copy link
Collaborator

llvmbot commented Jan 16, 2024

@llvm/pr-subscribers-github-workflow

Author: Mark de Wever (mordante)

Changes

The documentation CI no longer builds. This is likely introduced by
8f90e69. This fixes the issue.


Full diff: https://github.com/llvm/llvm-project/pull/78327.diff

1 Files Affected:

  • (modified) .github/workflows/docs.yml (+1-1)
diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
index bf559964ff97b80..5936cdc46a902d5 100644
--- a/.github/workflows/docs.yml
+++ b/.github/workflows/docs.yml
@@ -138,7 +138,7 @@ jobs:
       - name: Build libcxx docs
         if: steps.docs-changed-subprojects.outputs.libcxx_any_changed == 'true'
         run: |
-          cmake -B libcxx-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_RUNTIMES="libcxxabi;libcxx" -DLLVM_ENABLE_SPHINX=ON ./runtimes
+          cmake -B libcxx-build -GNinja -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_RUNTIMES="libcxxabi;libcxx;libunwind" -DLLVM_ENABLE_SPHINX=ON ./runtimes
           TZ=UTC ninja -C libcxx-build docs-libcxx-html
       - name: Build libc docs
         if: steps.docs-changed-subprojects.outputs.libc_any_changed == 'true'

justinfargnoli pushed a commit to justinfargnoli/llvm-project that referenced this pull request Jan 28, 2024
The documentation CI no longer builds. This is likely introduced by
8f90e69. This fixes the issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants