upload clangd index to CI and download in configure#159
Merged
Conversation
Build a clangd index using clangd-indexer in the nightly and release workflows, and attach the .idx file as a GitHub release asset. configure.py downloads the index for the matching dx-* tag on first run, stores it in .dx/, and configures .clangd to use it as an external index. This gives clangd project-wide symbols without a slow background index pass, while still allowing local edits to override stale entries.
bates64
force-pushed
the
clangd-index
branch
5 times, most recently
from
ad56bb5 to
3681ddd
Compare
Store the downloaded toolchain in .dx/windows/ instead of papermario-dx-windows/ at the repo root, keeping ephemeral data together in one gitignored directory. Also fix clangd index portability: build in YAML format and replace absolute CI paths with a $$ROOT$$ placeholder, which configure.py substitutes with the local project root at download time.
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
1 participant
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.
This PR makes it so that user IDEs (at least, those that use
clangd) do not need to reindex the entire project - can take a decent amount of time and memory - when they update dx or install it for the first time.configure.pydownloads the index from the appropriate github release whenever the user updates dx (nearest ancestor commit withdx-*tag moves to a new commit sha), and on first run..dx/windows/so we can avoid spamming the root directory with ephemeral state.