aspect-cli icon indicating copy to clipboard operation
aspect-cli copied to clipboard

aspect error messages reference bazel

Open mattem opened this issue 3 years ago • 2 comments

When an info / warn / error message is thrown by bazel, these hint to the user to run bazel, however we likely want to hint to the user to run aspect instead as this may lead to confusion.

eg:

$ bazel-bin/cmd/aspect/aspect_/aspect build
WARNING: Usage: bazel build <options> <targets>.
Invoke `bazel help build` for full description of usage and options.
Your request is correct, but requested an empty set of targets. Nothing will be built.
INFO: Analyzed 0 targets (0 packages loaded, 0 targets configured).
INFO: Found 0 targets...
INFO: Elapsed time: 0.155s, Critical Path: 0.04s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action

mattem avatar Mar 20 '22 21:03 mattem

To be precise I think devx teams will commonly install under another name, so we should use $0 everywhere and/or a product_name config setting. Some might actually install as 'bazel' like we do with bazelisk

alexeagle avatar Mar 21 '22 00:03 alexeagle

Yah good point, $0 or configure a product name is better here.

mattem avatar Mar 21 '22 13:03 mattem