Skip to content

ContextManager needs to watch for appearance of non-existent analysis roots #24305

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

Closed
stereotype441 opened this issue Sep 8, 2015 · 1 comment
Labels
devexp-server Issues related to some aspect of the analysis server legacy-area-analyzer Use area-devexp instead. P4 type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@stereotype441
Copy link
Member

From the analysis server API documentation, in the analysis.setAnalysisRoots section:

Note that this request determines the set of requested analysis roots. The actual set of analysis roots at any given time is the intersection of this set with the set of files and directories actually present on the filesystem. When the filesystem changes, the actual set of analysis roots is automatically updated, but the set of requested analysis roots is unchanged. This means that if the client sets an analysis root before the root becomes visible to server in the filesystem, there is no error; once the server sees the root in the filesystem it will start analyzing it. Similarly, server will stop analyzing files that are removed from the file system but they will remain in the set of requested roots.

Currently, if an analysis root does not exist in the file system at the time that setAnalysisRoots is invoked, the analysis server will not watch for its appearance. This means that, for instance, if the user's editor supports a single operation that creates a folder and sets it as an analysis root, race conditions in the file system may prevent that folder from being analyzed.

This should be fixed as part of the re-work of the ContextManager (#24133).

@stereotype441 stereotype441 added legacy-area-analyzer Use area-devexp instead. devexp-server Issues related to some aspect of the analysis server labels Sep 8, 2015
@stereotype441 stereotype441 self-assigned this Sep 8, 2015
@kevmoo kevmoo added P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed Priority-Medium labels Mar 1, 2016
@stereotype441 stereotype441 removed their assignment Aug 15, 2016
@srawlins srawlins added P4 and removed P2 A bug or feature request we're likely to work on labels Apr 13, 2023
@keertip
Copy link
Contributor

keertip commented Apr 18, 2023

Likely not valid anymore as there has been rewrites/refactors in this area. Closing as stale.

@keertip keertip closed this as completed Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devexp-server Issues related to some aspect of the analysis server legacy-area-analyzer Use area-devexp instead. P4 type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Development

No branches or pull requests

5 participants