liblog is a platform library that provides stable C API. There is no
need to include the library, especialy by statically linking to it, in
any APEX. It not only wastes the storage/ram, but also is incorrect
because the socket interface to logd which is implemented in liblog is
not guaranteed to be stable.
Fixing this issue by converting static_libs: ["liblog"] into
shared_libs: ["liblog"], in which case the dependency to the library
is satisfied via the stub variant of the library.
As a result, we could restrict the availablity of the library to
the platform and the runtime APEX.
Exempt-From-Owner-Approval: already approved when this was in internal
master (ag/10572699)
Bug: http://b/151051671
Bug: http://b/150827719
Test: m
Merged-In: I5aab863cb12b8767b6979255c247000a59355b0e
(cherry picked from commit 95b6f45b0e)
Change-Id: I5aab863cb12b8767b6979255c247000a59355b0e
64 lines
1.8 KiB
Text
64 lines
1.8 KiB
Text
//
|
|
// Copyright (C) 2019 The Android Open Source Project
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
//
|
|
|
|
// =========================================================================
|
|
// Native library that toggles between the old and new statsd socket
|
|
// protocols. This library should only be used by DNS resolver or other
|
|
// native modules on Q that log pushed atoms to statsd.
|
|
// =========================================================================
|
|
cc_defaults {
|
|
name: "libstatspush_compat_defaults",
|
|
srcs: [
|
|
"statsd_writer.c",
|
|
"stats_event_list.c",
|
|
"StatsEventCompat.cpp"
|
|
],
|
|
cflags: [
|
|
"-Wall",
|
|
"-Werror",
|
|
"-DWRITE_TO_STATSD=1",
|
|
"-DWRITE_TO_LOGD=0",
|
|
],
|
|
header_libs: ["libstatssocket_headers"],
|
|
static_libs: [
|
|
"libbase",
|
|
],
|
|
shared_libs: [
|
|
"liblog",
|
|
"libutils",
|
|
],
|
|
}
|
|
|
|
cc_library {
|
|
name: "libstatspush_compat",
|
|
defaults: ["libstatspush_compat_defaults"],
|
|
export_include_dirs: ["include"],
|
|
export_header_lib_headers: [
|
|
"libstatssocket_headers",
|
|
],
|
|
static_libs: ["libgtest_prod"],
|
|
}
|
|
|
|
cc_test {
|
|
name: "libstatspush_compat_test",
|
|
defaults: ["libstatspush_compat_defaults"],
|
|
test_suites: ["device_tests"],
|
|
srcs: [
|
|
"tests/StatsEventCompat_test.cpp",
|
|
],
|
|
static_libs: ["libgmock"],
|
|
}
|
|
|