summaryrefslogtreecommitdiff
path: root/sysprop
diff options
context:
space:
mode:
author Trevor Radcliffe <tradical@google.com> 2022-06-08 16:16:31 +0000
committer Trevor Radcliffe <tradical@google.com> 2022-08-15 20:10:58 +0000
commit2db49e32695470be11a3d12b10890aa39700dbb2 (patch)
treee3bc4eee86c4642e7a51b152f5a3aeeebed5171f /sysprop
parentffd6014a6836258b7f1c44b0988cceedab295fcf (diff)
Delete SyspropMutator
This CL and its associated CLs point cc modules that rely on sysprop_library modules to the auto-generated cc_library module, and then remove the code in Soong which maps the sysprop_library target to the generated target. This is being done because Bazel will not be supporting the existing behavior in Soong. Bug: 226199990 Test: m Change-Id: I117c5a4a9bf7c38e6fb7b92272a390df87507e30
Diffstat (limited to 'sysprop')
-rw-r--r--sysprop/sysprop_library.go5
-rw-r--r--sysprop/sysprop_test.go10
2 files changed, 9 insertions, 6 deletions
diff --git a/sysprop/sysprop_library.go b/sysprop/sysprop_library.go
index a29d4c3ac..409bca422 100644
--- a/sysprop/sysprop_library.go
+++ b/sysprop/sysprop_library.go
@@ -363,7 +363,10 @@ func (m *syspropLibrary) ShouldSupportSdkVersion(ctx android.BaseModuleContext,
// sysprop_library creates schematized APIs from sysprop description files (.sysprop).
// Both Java and C++ modules can link against sysprop_library, and API stability check
// against latest APIs (see build/soong/scripts/freeze-sysprop-api-files.sh)
-// is performed.
+// is performed. Note that the generated C++ module has its name prefixed with
+// `lib`, and it is this module that should be depended on from other C++
+// modules; i.e., if the sysprop_library module is named `foo`, C++ modules
+// should depend on `libfoo`.
func syspropLibraryFactory() android.Module {
m := &syspropLibrary{}
diff --git a/sysprop/sysprop_test.go b/sysprop/sysprop_test.go
index 88ef61524..80b86e059 100644
--- a/sysprop/sysprop_test.go
+++ b/sysprop/sysprop_test.go
@@ -209,32 +209,32 @@ func TestSyspropLibrary(t *testing.T) {
cc_library {
name: "cc-client-platform",
srcs: ["d.cpp"],
- static_libs: ["sysprop-platform"],
+ static_libs: ["libsysprop-platform"],
}
cc_library_static {
name: "cc-client-platform-static",
srcs: ["d.cpp"],
- whole_static_libs: ["sysprop-platform"],
+ whole_static_libs: ["libsysprop-platform"],
}
cc_library {
name: "cc-client-product",
srcs: ["d.cpp"],
product_specific: true,
- static_libs: ["sysprop-platform-on-product", "sysprop-vendor-on-product"],
+ static_libs: ["libsysprop-platform-on-product", "libsysprop-vendor-on-product"],
}
cc_library {
name: "cc-client-vendor",
srcs: ["d.cpp"],
soc_specific: true,
- static_libs: ["sysprop-platform", "sysprop-vendor"],
+ static_libs: ["libsysprop-platform", "libsysprop-vendor"],
}
cc_binary_host {
name: "hostbin",
- static_libs: ["sysprop-platform"],
+ static_libs: ["libsysprop-platform"],
}
`)