ci debug#44
Merged
Merged
Conversation
|
Review changes with Changed Files
|
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
There was a problem hiding this comment.
Pull request overview
Updates the GitHub Actions CI workflow to aid debugging LLVM header/path issues and refresh the LLVM cache key used by CI.
Changes:
- Bumped the LLVM cache key version (v4 → v5) to force cache invalidation.
- Expanded the cached LLVM tarball contents to include an additional include directory.
- Added a CI step to print/debug LLVM include paths and header presence.
This reverts commit c09a12d.
* Initial plan * Fix LLVM include path mismatch in CI workflow Create symlinks to bridge Debian/Ubuntu header location (/usr/include/llvm-21) with llvm-config expectations (/usr/lib/llvm-21/include). This resolves compilation failures where cc-rs cannot find llvm-c/Target.h. Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com> * Bump cache version to v6 for LLVM with symlinks Previous v5 caches do not contain the symlinks needed to fix the Debian/Ubuntu header path mismatch. Bumping to v6 ensures fresh installation with correct symlinks. Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com> * Refactor LLVM path fix to use reusable script Extract symlink creation logic into scripts/fix-llvm-paths.sh to: - Eliminate code duplication - Make the fix more maintainable - Centralize LLVM path correction logic - Add script to cache key for invalidation on changes Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com> * Document why symlink fix runs after cache restoration Add comment explaining that the script runs unconditionally for defense in depth. The script is idempotent and provides safety against cache corruption or version mismatches. Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com> * Fix root cause: Configure C compiler include paths correctly Remove symlink workaround and fix the actual problem by setting C_INCLUDE_PATH and CPLUS_INCLUDE_PATH environment variables to point to /usr/include/llvm-21 where Debian/Ubuntu actually installs headers. This allows cc-rs (used by llvm-sys) to find LLVM headers without requiring filesystem modifications or workarounds. Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com>
* Fix LLVM cache to include llvm-c-21 headers directory Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com> * polly * Reapply "Better rom building." This reverts commit 2de7ab3. * Revert "Reapply "Better rom building."" This reverts commit b2b63cd. * test * test --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: tcrypt25519 <212655132+tcrypt25519@users.noreply.github.com> Co-authored-by: Tyler Smith <mail@tcry.pt>
tcrypt25519
added a commit
that referenced
this pull request
Feb 16, 2026
* Fix LLVM include path mismatch in CI workflow
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.