drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Convert statsd_test and statsd_benchmark to proto lite

Browse Source 
libplatformprotos for the device is compiled using proto lite, and
statsd_test and statsd_benchmark have protos that reference the
libplatformprotos.  With protobuf 3.9.1 compiling a full proto
against a lite proto doesn't work.  Convert them to lite.  Requires
adding the protobuf internal protos to the sources,
libprotobuf-cpp-lite does not contain the compiled version of
field_options.proto.

Bug: 117607748
Test: m checkbuild
Change-Id: I0b101555ff604df17b2e3e06f3b3ce25adb1c9fb
Merged-In: I0b101555ff604df17b2e3e06f3b3ce25adb1c9fb
This commit is contained in:
Colin Cross 2018-10-26 23:28:02 -07:00
parent f74defb808
commit 959cc442a1
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
cmds/statsd/Android.bp
View File

@ -209,6 +209,10 @@ cc_test {
],
srcs: [
// atom_field_options.proto needs field_options.proto, but that is
// not included in libprotobuf-cpp-lite, so compile it here.
":libprotobuf-internal-protos",
"src/atom_field_options.proto",
"src/atoms.proto",
"src/stats_log.proto",
@ -270,11 +274,11 @@ cc_test {
],
proto: {
type: "full",
type: "lite",
include_dirs: ["external/protobuf/src"],
},
shared_libs: ["libprotobuf-cpp-full"],
shared_libs: ["libprotobuf-cpp-lite"],
}
@ -287,6 +291,10 @@ cc_benchmark {
defaults: ["statsd_defaults"],
srcs: [
// atom_field_options.proto needs field_options.proto, but that is
// not included in libprotobuf-cpp-lite, so compile it here.
":libprotobuf-internal-protos",
"src/atom_field_options.proto",
"src/atoms.proto",
"src/stats_log.proto",
@ -301,7 +309,7 @@ cc_benchmark {
],
proto: {
type: "full",
type: "lite",
include_dirs: ["external/protobuf/src"],
},
@ -323,7 +331,7 @@ cc_benchmark {
shared_libs: [
"libgtest_prod",
"libstatslog",
"libprotobuf-cpp-full",
"libprotobuf-cpp-lite",
],
}