Primary shared cache file UX improvements#8188
Merged
Merged
Conversation
fuzyll
added
the
File Format: SharedCache
bdash
reviewed
May 20, 2026
fuzyll
added a commit
that referenced
this pull request
May 20, 2026
Addresses feedback from PR #8188.
emesare
approved these changes
May 27, 2026
Addresses feedback from PR #8188.
fuzyll
force-pushed
the
test_shared_cache_ux_improvements
branch
from
31c8f4e to
27f966a
Compare
Contributor
Author
|
This was merged in 5.4.9718-dev. |
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
3 participants
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 addresses the following issues:
It also adds some clarification on how analysis databases for shared caches are handled differently from regular analysis databases. Had an Enterprise customer that lost some time because their expectations didn't account for this difference and figured it would be good to spell it out more clearly.
One of the primary changes here is the introduction of a load setting,
loader.dsc.primaryFilePath, which is intended to be used in headless settings to specify where the primary shared cache file is. An example of using this is now also in the documentation.This is still pretty annoying, though, so I've changed and expanded how we search for the primary shared cache file. We should now handle cases like accidentally choosing a
.bndbfile better. We should also be able to handle shared caches being stored within projects more gracefully as well. I tried to keep Enterprise collaboration in mind with these changes and I don't believe there should be any strange conflicts introduced.