cc_cmake_snapshot: drop Modules field
Use Modules_host, Modules_system or Modules_vendor instead
Also, fix test by adding libc++demangle to the ignored list.
Bug: 285204695
Test: m binder_sdk
Test: cd build/soong/cc && go test
Change-Id: I6c3eda83584bc910f8b73ffa0f36c3d7fce5f10b
diff --git a/cc/cmake_snapshot.go b/cc/cmake_snapshot.go
index b797764..fb2924a 100644
--- a/cc/cmake_snapshot.go
+++ b/cc/cmake_snapshot.go
@@ -69,6 +69,7 @@
"libc",
"libc++",
"libc++_static",
+ "libc++demangle",
"libc_musl",
"libc_musl_crtbegin_so",
"libc_musl_crtbegin_static",
@@ -96,9 +97,6 @@
}
type CmakeSnapshotProperties struct {
- // TODO: remove
- Modules []string
-
// Host modules to add to the snapshot package. Their dependencies are pulled in automatically.
Modules_host []string
@@ -289,7 +287,6 @@
deviceVendorVariations := append(deviceVariations, blueprint.Variation{"image", "vendor"})
hostVariations := ctx.Config().BuildOSTarget.Variations()
- ctx.AddVariationDependencies(hostVariations, cmakeSnapshotModuleTag, m.Properties.Modules...)
ctx.AddVariationDependencies(hostVariations, cmakeSnapshotModuleTag, m.Properties.Modules_host...)
ctx.AddVariationDependencies(deviceSystemVariations, cmakeSnapshotModuleTag, m.Properties.Modules_system...)
ctx.AddVariationDependencies(deviceVendorVariations, cmakeSnapshotModuleTag, m.Properties.Modules_vendor...)