search

bazelbuild / bazel-watcher

Tools for building Bazel targets when source files change.
Apache License 2.0
440 stars 116 forks source link

BUILD file changes not detected anymore #603

Closed purkhusid closed 1 year ago

purkhusid commented 1 year ago

Describe the bug BUILD file changes are not detected by bazel-watcher and when bazel-watcher is started it non-fatally emits the following error:

ERROR: Error doing post analysis query: Evaluation failed: buildfiles() doesn't make sense for the configured target graph

The issue is very likely related to this change: https://github.com/bazelbuild/bazel-watcher/commit/ac2ce9f7406798b10c584efbac14ca05ba8cd818

The change moves the queries bazel-watcher does over to cquery but cquery does not support buildfiles in it's queries.

Expected behavior BUILD file changes should be detected

Version (please complete the following information):

achew22 commented 1 year ago

Can you confirm this is fixed in https://github.com/bazelbuild/bazel-watcher/releases/tag/v0.23.0?

purkhusid commented 1 year ago

The Linux binaries are missing from the release. Will test it as soon as they get published.

achew22 commented 1 year ago

Sorry about that, they're sitting in the GitHub actions queue here https://github.com/bazelbuild/bazel-watcher/actions/runs/4736205730

jakeleventhal commented 1 year ago

This issue can close