Building a target with a Bazel aspect overwrites the Target command line printout
#10,530 建立於 2020年1月6日
倉庫指標
- Star
- (25,384 star)
- PR 合併指標
- (平均合併 22天 20小時) (30 天內合併 77 個 PR)
描述
Description of the problem / feature request:
Building a target with a Bazel aspect overwrites the Target command line printout. The Target printout previously contained information such as the path to the generated build artifact.
For example:
$ bazel build :grpc_health_v1_proto --aspects=//rules:proto_linter.bzl%proto_linter_aspect --output_groups=+proto_lint_out
INFO: Writing tracer profile to
...
INFO: Found 1 target...
Target //idl/x/go/health/proto:grpc_health_v1_proto up-to-date:
bazel-bin/idl/x/go/health/proto/grpc_health_v1_proto-descriptor-set.proto.bin
INFO: Elapsed time: 0.582s, Critical Path: 0.00s
INFO: 0 processes.
INFO: Build completed successfully, 1 total action
Has a different command line output then
$ bazel build :grpc_health_v1_proto
INFO: Writing tracer profile to
...
INFO: Found 1 target...
Aspect //rules:proto_linter.bzl%proto_linter_aspect of //idl/x/go/health/proto:grpc_health_v1_proto up-to-date:
bazel-bin/idl/xl/go/health/proto/grpc_health_v1_proto.protolint_result
INFO: Elapsed time: 0.493s, Critical Path: 0.00s
INFO: 0 processes.
INFO: Build completed successfully, 1 total action
The path to the aspect build artifact is printed, but the original build artifact path is lost.
I believe this issue is related to this [logic](https://source.bazel.build/bazel/+/master:src/main/java/com/google/devtools/build/lib/buildtool/BuildResultPrinter.java;bpv=;l=81 l) in Bazel.
Feature requests: what underlying problem are you trying to solve with this feature?
Building a Bazel target with an aspect should preserve the build target output message.
What operating system are you running Bazel on?
darwin18.0
What's the output of bazel info release?
release 1.1.0
Have you found anything relevant by searching the web?
I haven't found a relevant discussion online as of yet.
I believe this issue is related to this [logic](https://source.bazel.build/bazel/+/master:src/main/java/com/google/devtools/build/lib/buildtool/BuildResultPrinter.java;bpv=;l=81 l) in Bazel.