diff options
author | Jiyong Park <jiyong@google.com> | 2017-06-16 12:03:55 +0900 |
---|---|---|
committer | Steven Moreland <smoreland@google.com> | 2017-06-21 12:33:46 -0700 |
commit | 9875edb04f939727b431d0645d20c9a986c517cd (patch) | |
tree | 8764f40723727b0e4169267e409a99d1f03d75d7 | |
parent | c8451eb3d0558ea6ed30f692a8b3c408343ea850 (diff) | |
download | build_soong-9875edb04f939727b431d0645d20c9a986c517cd.tar.gz build_soong-9875edb04f939727b431d0645d20c9a986c517cd.tar.bz2 build_soong-9875edb04f939727b431d0645d20c9a986c517cd.zip |
add exclude_shared_libs for vendor_available:true libs
Adding a mechanism to conditionally exclude some shared library
dependencies when a lib is built for vendors.
Without this, some libraries cannot be earily marked as vendor_available
if they are depending on shared libs can shouldn't be marked as
vendor_available.
By using exclude_shared_libs with exclude_srcs (or __ANDROID_VNDK__
macro), we can eliminate the unnecessary dependency for vendors.
Bug: 62471389
Test: build
(cherry picked from commit 44cf1a777916fbe2dc03760b0f2076c4cd3a0f63)
Merged-In: If94277b45c3769223cea371d0028e75277640356
Change-Id: If94277b45c3769223cea371d0028e75277640356
-rw-r--r-- | cc/linker.go | 12 | ||||
-rw-r--r-- | cc/util.go | 10 |
2 files changed, 22 insertions, 0 deletions
diff --git a/cc/linker.go b/cc/linker.go index 5a3b4781..2c391320 100644 --- a/cc/linker.go +++ b/cc/linker.go @@ -87,6 +87,14 @@ type BaseLinkerProperties struct { // group static libraries. This can resolve missing symbols issues with interdependencies // between static libraries, but it is generally better to order them correctly instead. Group_static_libs *bool `android:"arch_variant"` + + Target struct { + Vendor struct { + // list of shared libs that should not be used to build + // the vendor variant of the C/C++ module. + Exclude_shared_libs []string + } + } } func NewBaseLinker() *baseLinker { @@ -123,6 +131,10 @@ func (linker *baseLinker) linkerDeps(ctx BaseModuleContext, deps Deps) Deps { deps.StaticLibs = append(deps.StaticLibs, linker.Properties.Static_libs...) deps.SharedLibs = append(deps.SharedLibs, linker.Properties.Shared_libs...) + if ctx.vndk() { + deps.SharedLibs = removeListFromList(deps.SharedLibs, linker.Properties.Target.Vendor.Exclude_shared_libs) + } + deps.ReexportHeaderLibHeaders = append(deps.ReexportHeaderLibHeaders, linker.Properties.Export_header_lib_headers...) deps.ReexportStaticLibHeaders = append(deps.ReexportStaticLibHeaders, linker.Properties.Export_static_lib_headers...) deps.ReexportSharedLibHeaders = append(deps.ReexportSharedLibHeaders, linker.Properties.Export_shared_lib_headers...) @@ -66,6 +66,16 @@ func filterList(list []string, filter []string) (remainder []string, filtered [] return } +func removeListFromList(list []string, filter_out []string) (result []string) { + result = make([]string, 0, len(list)) + for _, l := range list { + if !inList(l, filter_out) { + result = append(result, l) + } + } + return +} + func removeFromList(s string, list []string) (bool, []string) { i := indexList(s, list) if i != -1 { |