Skip to content
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

use cquery for resolving files and dependencies #672

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tobiaskohlbau
Copy link

This takes antoher stub at #594 by introducing cquery back in. In difference to the previous attempt this splits up the resolving of buildfiles. CQuery does not support buildfiles but without cquery bazel-watcher could watch files which are not reflected in the build for e.g. dependencies behind a select statement. In order to avoid depending on such cases first a cquery is issued which resolves dependencies of selected targets. These dependencies are then injected into the buildfiles query in order to get dependent build files.

This takes antoher stub at bazelbuild#594 by introducing cquery back in. In
difference to the previous attempt this splits up the resolving of
buildfiles. CQuery does not support buildfiles but without cquery
bazel-watcher could watch files which are not reflected in the build
for e.g. dependencies behind a select statement. In order to avoid
depending on such cases first a cquery is issued which resolves
dependencies of selected targets. These dependencies are then injected
into the buildfiles query in order to get dependent build files.

Signed-off-by: Tobias Kohlbau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant