Merge "Support filegroups for version_script and related properties" am: ca59511e21
am: 4009b12ead
am: bfbda64f6a
Change-Id: Iace6d60c3d05af33825a5066caacf169bf7b541c
This commit is contained in:
commit
64ab8062ec
|
@ -110,6 +110,7 @@ type ModuleContext interface {
|
|||
|
||||
ExpandSources(srcFiles, excludes []string) Paths
|
||||
ExpandSource(srcFile, prop string) Path
|
||||
ExpandOptionalSource(srcFile *string, prop string) OptionalPath
|
||||
ExpandSourcesSubDir(srcFiles, excludes []string, subDir string) Paths
|
||||
Glob(globPattern string, excludes []string) Paths
|
||||
|
||||
|
@ -1162,6 +1163,16 @@ func (ctx *androidModuleContext) ExpandSource(srcFile, prop string) Path {
|
|||
}
|
||||
}
|
||||
|
||||
// Returns an optional single path expanded from globs and modules referenced using ":module" syntax if
|
||||
// the srcFile is non-nil.
|
||||
// ExtractSourceDeps must have already been called during the dependency resolution phase.
|
||||
func (ctx *androidModuleContext) ExpandOptionalSource(srcFile *string, prop string) OptionalPath {
|
||||
if srcFile != nil {
|
||||
return OptionalPathForPath(ctx.ExpandSource(*srcFile, prop))
|
||||
}
|
||||
return OptionalPath{}
|
||||
}
|
||||
|
||||
func (ctx *androidModuleContext) ExpandSourcesSubDir(srcFiles, excludes []string, subDir string) Paths {
|
||||
prefix := PathForModuleSrc(ctx).String()
|
||||
|
||||
|
|
|
@ -158,6 +158,9 @@ func (binary *binaryDecorator) linkerDeps(ctx DepsContext, deps Deps) Deps {
|
|||
ctx.ModuleErrorf("statically linking libc to dynamic executable, please remove libc\n" +
|
||||
"from static libs or set static_executable: true")
|
||||
}
|
||||
|
||||
android.ExtractSourceDeps(ctx, binary.Properties.Version_script)
|
||||
|
||||
return deps
|
||||
}
|
||||
|
||||
|
@ -277,7 +280,7 @@ func (binary *binaryDecorator) linkerFlags(ctx ModuleContext, flags Flags) Flags
|
|||
func (binary *binaryDecorator) link(ctx ModuleContext,
|
||||
flags Flags, deps PathDeps, objs Objects) android.Path {
|
||||
|
||||
versionScript := android.OptionalPathForModuleSrc(ctx, binary.Properties.Version_script)
|
||||
versionScript := ctx.ExpandOptionalSource(binary.Properties.Version_script, "version_script")
|
||||
fileName := binary.getStem(ctx) + flags.Toolchain.ExecutableSuffix()
|
||||
outputFile := android.PathForModuleOut(ctx, fileName)
|
||||
ret := outputFile
|
||||
|
|
|
@ -476,6 +476,13 @@ func (library *libraryDecorator) linkerDeps(ctx DepsContext, deps Deps) Deps {
|
|||
deps.SharedLibs = removeListFromList(deps.SharedLibs, library.baseLinker.Properties.Target.Vendor.Exclude_shared_libs)
|
||||
deps.StaticLibs = removeListFromList(deps.StaticLibs, library.baseLinker.Properties.Target.Vendor.Exclude_static_libs)
|
||||
}
|
||||
|
||||
android.ExtractSourceDeps(ctx, library.Properties.Version_script)
|
||||
android.ExtractSourceDeps(ctx, library.Properties.Unexported_symbols_list)
|
||||
android.ExtractSourceDeps(ctx, library.Properties.Force_symbols_not_weak_list)
|
||||
android.ExtractSourceDeps(ctx, library.Properties.Force_symbols_weak_list)
|
||||
android.ExtractSourceDeps(ctx, library.Properties.Target.Vendor.Version_script)
|
||||
|
||||
return deps
|
||||
}
|
||||
|
||||
|
@ -507,12 +514,12 @@ func (library *libraryDecorator) linkShared(ctx ModuleContext,
|
|||
var linkerDeps android.Paths
|
||||
linkerDeps = append(linkerDeps, flags.LdFlagsDeps...)
|
||||
|
||||
versionScript := android.OptionalPathForModuleSrc(ctx, library.Properties.Version_script)
|
||||
unexportedSymbols := android.OptionalPathForModuleSrc(ctx, library.Properties.Unexported_symbols_list)
|
||||
forceNotWeakSymbols := android.OptionalPathForModuleSrc(ctx, library.Properties.Force_symbols_not_weak_list)
|
||||
forceWeakSymbols := android.OptionalPathForModuleSrc(ctx, library.Properties.Force_symbols_weak_list)
|
||||
versionScript := ctx.ExpandOptionalSource(library.Properties.Version_script, "version_script")
|
||||
unexportedSymbols := ctx.ExpandOptionalSource(library.Properties.Unexported_symbols_list, "unexported_symbols_list")
|
||||
forceNotWeakSymbols := ctx.ExpandOptionalSource(library.Properties.Force_symbols_not_weak_list, "force_symbols_not_weak_list")
|
||||
forceWeakSymbols := ctx.ExpandOptionalSource(library.Properties.Force_symbols_weak_list, "force_symbols_weak_list")
|
||||
if ctx.useVndk() && library.Properties.Target.Vendor.Version_script != nil {
|
||||
versionScript = android.OptionalPathForModuleSrc(ctx, library.Properties.Target.Vendor.Version_script)
|
||||
versionScript = ctx.ExpandOptionalSource(library.Properties.Target.Vendor.Version_script, "target.vendor.version_script")
|
||||
}
|
||||
if !ctx.Darwin() {
|
||||
if versionScript.Valid() {
|
||||
|
|
Loading…
Reference in New Issue