Skip to content

Conversation

@boomanaiden154
Copy link
Contributor

The docs build action was failing with libc due to checks.rst not existing in the expected path. This patch adjusts the path to the actual path which seems to make everything happy. It seems like this did not show up before as stdfix.rst was not included in a place that actually caused it to get picked up by sphinx.

The docs build action was failing with libc due to checks.rst not
existing in the expected path. This patch adjusts the path to the actual
path which seems to make everything happy. It seems like this did not
show up before as stdfix.rst was not included in a place that actually
caused it to get picked up by sphinx.
@llvmbot
Copy link
Member

llvmbot commented Mar 6, 2025

@llvm/pr-subscribers-libc

Author: Aiden Grossman (boomanaiden154)

Changes

The docs build action was failing with libc due to checks.rst not existing in the expected path. This patch adjusts the path to the actual path which seems to make everything happy. It seems like this did not show up before as stdfix.rst was not included in a place that actually caused it to get picked up by sphinx.


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

1 Files Affected:

  • (modified) libc/docs/headers/stdfix.rst (+1-1)
diff --git a/libc/docs/headers/stdfix.rst b/libc/docs/headers/stdfix.rst
index 3b93f29a4abdc..5189439c63306 100644
--- a/libc/docs/headers/stdfix.rst
+++ b/libc/docs/headers/stdfix.rst
@@ -2,7 +2,7 @@
 StdFix Functions
 ================
 
-.. include:: ../../check.rst
+.. include:: ../check.rst
 
 Standards and Goals
 -------------------

@boomanaiden154 boomanaiden154 merged commit eac734a into llvm:main Mar 6, 2025
15 of 18 checks passed
@boomanaiden154 boomanaiden154 deleted the fix-libc-docs-3-6-25 branch March 6, 2025 22:07
@michaelrj-google
Copy link
Contributor

Thanks for fixing this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants