Merge "Use OutputFilesProvider on buildinfo_prop" into main

This commit is contained in:
Zi Wang
2024-07-01 21:49:21 +00:00
committed by Gerrit Code Review

View File

@@ -15,8 +15,6 @@
package android package android
import ( import (
"fmt"
"github.com/google/blueprint/proptools" "github.com/google/blueprint/proptools"
) )
@@ -41,20 +39,10 @@ type buildinfoPropModule struct {
installPath InstallPath installPath InstallPath
} }
var _ OutputFileProducer = (*buildinfoPropModule)(nil)
func (p *buildinfoPropModule) installable() bool { func (p *buildinfoPropModule) installable() bool {
return proptools.BoolDefault(p.properties.Installable, true) return proptools.BoolDefault(p.properties.Installable, true)
} }
// OutputFileProducer
func (p *buildinfoPropModule) OutputFiles(tag string) (Paths, error) {
if tag != "" {
return nil, fmt.Errorf("unsupported tag %q", tag)
}
return Paths{p.outputFilePath}, nil
}
func shouldAddBuildThumbprint(config Config) bool { func shouldAddBuildThumbprint(config Config) bool {
knownOemProperties := []string{ knownOemProperties := []string{
"ro.product.brand", "ro.product.brand",
@@ -76,6 +64,8 @@ func (p *buildinfoPropModule) GenerateAndroidBuildActions(ctx ModuleContext) {
return return
} }
p.outputFilePath = PathForModuleOut(ctx, p.Name()).OutputPath p.outputFilePath = PathForModuleOut(ctx, p.Name()).OutputPath
ctx.SetOutputFiles(Paths{p.outputFilePath}, "")
if !ctx.Config().KatiEnabled() { if !ctx.Config().KatiEnabled() {
WriteFileRule(ctx, p.outputFilePath, "# no buildinfo.prop if kati is disabled") WriteFileRule(ctx, p.outputFilePath, "# no buildinfo.prop if kati is disabled")
return return