Merge changes I6a5357e9,I199f070e

* changes:
  Install ko to /lib/modules
  android_filesystem is OutputFileProducer
This commit is contained in:
Treehugger Robot 2021-02-05 03:39:51 +00:00 committed by Gerrit Code Review
commit 364da36115
3 changed files with 17 additions and 7 deletions

View File

@ -271,6 +271,16 @@ func (f *filesystem) AndroidMkEntries() []android.AndroidMkEntries {
}}
}
var _ android.OutputFileProducer = (*filesystem)(nil)
// Implements android.OutputFileProducer
func (f *filesystem) OutputFiles(tag string) (android.Paths, error) {
if tag == "" {
return []android.Path{f.output}, nil
}
return nil, fmt.Errorf("unsupported module reference tag %q", tag)
}
// Filesystem is the public interface for the filesystem struct. Currently, it's only for the apex
// package to have access to the output file.
type Filesystem interface {

View File

@ -72,7 +72,7 @@ func (pkm *prebuiltKernelModules) GenerateAndroidBuildActions(ctx android.Module
depmodOut := runDepmod(ctx, modules)
strippedModules := stripDebugSymbols(ctx, modules)
installDir := android.PathForModuleInstall(ctx, "lib", "module")
installDir := android.PathForModuleInstall(ctx, "lib", "modules")
if pkm.KernelVersion() != "" {
installDir = installDir.Join(ctx, pkm.KernelVersion())
}

View File

@ -84,12 +84,12 @@ func TestKernelModulesFilelist(t *testing.T) {
})
expected := []string{
"lib/module/5.10/mod1.ko",
"lib/module/5.10/mod2.ko",
"lib/module/5.10/modules.load",
"lib/module/5.10/modules.dep",
"lib/module/5.10/modules.softdep",
"lib/module/5.10/modules.alias",
"lib/modules/5.10/mod1.ko",
"lib/modules/5.10/mod2.ko",
"lib/modules/5.10/modules.load",
"lib/modules/5.10/modules.dep",
"lib/modules/5.10/modules.softdep",
"lib/modules/5.10/modules.alias",
}
var actual []string