perfetto: improve bazel BUILD file generation with protos
This CL removes our legacy behaviour of automatically creating
source_set targets for proto libraries without the associated source_set
targets being present in GN. Instead, we now require that the target
exist.
There are also a few other improvements:
* fix naming of a few scattered classes to match the general pattern
* don't generate lite targets when not necessary; this removes a bunch
of private lite targets which aren't used by anyone
* the same source_set target is not generated multiple times anymore
meaning that we can get rid of the deduplication hack which preserved
public visibility for proto targets
Change-Id: I0b39dbacc3a422479ad1a2d7e5f62089f8f54578
diff --git a/Android.bp b/Android.bp
index 8d92551..79c7103 100644
--- a/Android.bp
+++ b/Android.bp
@@ -925,7 +925,7 @@
":perfetto_src_ipc_client",
":perfetto_src_ipc_common",
":perfetto_src_perfetto_cmd_perfetto_cmd",
- ":perfetto_src_perfetto_cmd_protos_gen",
+ ":perfetto_src_perfetto_cmd_protos_cpp_gen",
":perfetto_src_perfetto_cmd_trigger_producer",
":perfetto_src_protozero_protozero",
":perfetto_src_tracing_common",
@@ -986,7 +986,7 @@
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_src_base_version_gen_h",
"perfetto_src_perfetto_cmd_gen_cc_config_descriptor",
- "perfetto_src_perfetto_cmd_protos_gen_headers",
+ "perfetto_src_perfetto_cmd_protos_cpp_gen_headers",
],
defaults: [
"perfetto_defaults",
@@ -2921,9 +2921,9 @@
],
}
-// GN: //protos/perfetto/config:perfetto_config_descriptor
+// GN: //protos/perfetto/config:merged_config_descriptor
genrule {
- name: "perfetto_protos_perfetto_config_perfetto_config_descriptor",
+ name: "perfetto_protos_perfetto_config_merged_config_descriptor",
srcs: [
"protos/perfetto/config/perfetto_config.proto",
],
@@ -2932,7 +2932,7 @@
],
cmd: "mkdir -p $(genDir)/external/perfetto/ && $(location aprotoc) --proto_path=external/perfetto --descriptor_set_out=$(out) $(in)",
out: [
- "perfetto_protos_perfetto_config_perfetto_config_descriptor.bin",
+ "perfetto_protos_perfetto_config_merged_config_descriptor.bin",
],
}
@@ -7221,7 +7221,7 @@
genrule {
name: "perfetto_src_perfetto_cmd_gen_cc_config_descriptor",
srcs: [
- ":perfetto_protos_perfetto_config_perfetto_config_descriptor",
+ ":perfetto_protos_perfetto_config_merged_config_descriptor",
],
cmd: "$(location tools/gen_cc_proto_descriptor.py) --gen_dir=$(genDir) --cpp_out=$(out) $(in)",
out: [
@@ -7245,9 +7245,9 @@
],
}
-// GN: //src/perfetto_cmd:protos
+// GN: //src/perfetto_cmd:protos_cpp
genrule {
- name: "perfetto_src_perfetto_cmd_protos_gen",
+ name: "perfetto_src_perfetto_cmd_protos_cpp_gen",
srcs: [
"src/perfetto_cmd/perfetto_cmd_state.proto",
],
@@ -7261,9 +7261,9 @@
],
}
-// GN: //src/perfetto_cmd:protos
+// GN: //src/perfetto_cmd:protos_cpp
genrule {
- name: "perfetto_src_perfetto_cmd_protos_gen_headers",
+ name: "perfetto_src_perfetto_cmd_protos_cpp_gen_headers",
srcs: [
"src/perfetto_cmd/perfetto_cmd_state.proto",
],
@@ -9453,7 +9453,7 @@
":perfetto_src_kallsyms_kallsyms",
":perfetto_src_kallsyms_unittests",
":perfetto_src_perfetto_cmd_perfetto_cmd",
- ":perfetto_src_perfetto_cmd_protos_gen",
+ ":perfetto_src_perfetto_cmd_protos_cpp_gen",
":perfetto_src_perfetto_cmd_trigger_producer",
":perfetto_src_perfetto_cmd_unittests",
":perfetto_src_profiling_common_callstack_trie",
@@ -9689,7 +9689,7 @@
"perfetto_src_ipc_test_messages_cpp_gen_headers",
"perfetto_src_ipc_test_messages_ipc_gen_headers",
"perfetto_src_perfetto_cmd_gen_cc_config_descriptor",
- "perfetto_src_perfetto_cmd_protos_gen_headers",
+ "perfetto_src_perfetto_cmd_protos_cpp_gen_headers",
"perfetto_src_protozero_testing_messages_cpp_gen_headers",
"perfetto_src_protozero_testing_messages_lite_gen_headers",
"perfetto_src_protozero_testing_messages_zero_gen_headers",
@@ -10302,7 +10302,7 @@
":perfetto_src_base_unix_socket",
":perfetto_src_ipc_client",
":perfetto_src_ipc_common",
- ":perfetto_src_perfetto_cmd_protos_gen",
+ ":perfetto_src_perfetto_cmd_protos_cpp_gen",
":perfetto_src_perfetto_cmd_trigger_perfetto_cmd",
":perfetto_src_perfetto_cmd_trigger_producer",
":perfetto_src_protozero_protozero",
@@ -10361,7 +10361,7 @@
"perfetto_protos_perfetto_trace_track_event_cpp_gen_headers",
"perfetto_protos_perfetto_trace_track_event_zero_gen_headers",
"perfetto_src_base_version_gen_h",
- "perfetto_src_perfetto_cmd_protos_gen_headers",
+ "perfetto_src_perfetto_cmd_protos_cpp_gen_headers",
],
defaults: [
"perfetto_defaults",