This repository has been archived by the owner on Dec 18, 2023. It is now read-only.
make bazel query
try harder to get a list of dependencies
#10
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.
If I run
bazel build "//examples/macos/..."
inhttps://github.com/bazelbuild/rules_apple then it works fine, but
if I run
bazel query "deps(//examples/macos/...)"
then it blows upin my face, complaining about "no such package '@com_google_protobuf//'".
This causes rebazel to have empty list of files to watch.
This patch adds the
--keep_going
flag tobazel query
, so we atleast end up with a partial list of files to watch. In practice, this was
enough to get rebuild-on-save to work for the example files that I
was editing.
I've not used bazel before, so I'm not sure how common this inability
to get deps is. Is this a reasonable hack, or should we instead listen
to the exit code of
bazel query
and abort, or trigger some kind offallback behaviour?