search

bazelbuild / bazel

a fast, scalable, multi-language and extensible build system
https://bazel.build
Apache License 2.0
23.13k stars 4.05k forks source link

Bazel query docs suggest deprecated feature #5937

Closed petroseskinder closed 4 years ago

petroseskinder commented 6 years ago

What's the set of BUILD files needed to build //foo?

bazel query 'buildfiles(deps(//foo))' --output location | cut -f1 -d:

4835 deprecated this feature, and now, executing this command outputs

ERROR: Query expressions involving 'buildfiles' or 'loadfiles' cannot be used with --output=location
haxorz commented 4 years ago

Sorry for the lack of response over the past ~20 months.

I'm a little confused about the timeline here:

  1. [Jun 2017] Commit 9e26369575f04776c0416fd75a9434a22b4d5e9a (authored by me, coincidentally) bans --output=location in combinarion with buildfiles or loadfiles
  2. [Mar 2018] Commit 2d0b73e00c6ce0b43108218c7d0073e9f9ded612 (from PR #4835) updated the query-how-to.md doc.
  3. [Aug 2018] This issue was filed.

ISTM that commit 2d0b73e00c6ce0b43108218c7d0073e9f9ded612 fixed the documentation problem, and this issue is complaining about a problem that no longer exists.

Therefore I'm going to close this, but feel free to reopen if I've misunderstood you.

petroseskinder commented 4 years ago

Yeah this should have been closed.

The html still included the suggestion at the time. You linked to the markdown