StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

genrule: let Android.bp file specify exported header dirs

Browse Source 
Instead of exporting the generated sources dir as headers, let
the Android.bp file specify subdirectories as exported.

Test: m -j checkbuild
Change-Id: I18dd900d63ce7485c8fbfcc39dc77abad6f733d7
This commit is contained in:
Colin Cross
2016-11-22 12:55:55 -08:00
parent 33bfb0a36a
commit 5ed99c6472
2 changed files with 17 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cc/cc.go
View File

@@ -786,7 +786,7 @@ func (c *Module) depsToPaths(ctx android.ModuleContext) PathDeps {
if genRule, ok := m.(genrule.SourceFileGenerator); ok { if genRule, ok := m.(genrule.SourceFileGenerator); ok {
depPaths.GeneratedHeaders = append(depPaths.GeneratedHeaders, depPaths.GeneratedHeaders = append(depPaths.GeneratedHeaders,
genRule.GeneratedSourceFiles()...) genRule.GeneratedSourceFiles()...)
flags := includeDirsToFlags(android.Paths{genRule.GeneratedHeaderDir()}) flags := includeDirsToFlags(genRule.GeneratedHeaderDirs())
depPaths.Flags = append(depPaths.Flags, flags) depPaths.Flags = append(depPaths.Flags, flags)
if tag == genHeaderExportDepTag { if tag == genHeaderExportDepTag {
depPaths.ReexportedFlags = append(depPaths.ReexportedFlags, flags) depPaths.ReexportedFlags = append(depPaths.ReexportedFlags, flags)

22
genrule/genrule.go
View File

@@ -34,7 +34,7 @@ var (
type SourceFileGenerator interface { type SourceFileGenerator interface {
GeneratedSourceFiles() android.Paths GeneratedSourceFiles() android.Paths
GeneratedHeaderDir() android.Path GeneratedHeaderDirs() android.Paths
} }
type HostToolProvider interface { type HostToolProvider interface {
@@ -65,6 +65,9 @@ type generatorProperties struct {
// Local file that is used as the tool // Local file that is used as the tool
Tool_files []string Tool_files []string
// List of directories to export generated headers from
Export_include_dirs []string
} }
type generator struct { type generator struct {
@@ -77,7 +80,7 @@ type generator struct {
deps android.Paths deps android.Paths
rule blueprint.Rule rule blueprint.Rule
genPath android.Path exportedIncludeDirs android.Paths
outputFiles android.Paths outputFiles android.Paths
} }
@@ -93,8 +96,8 @@ func (g *generator) GeneratedSourceFiles() android.Paths {
return g.outputFiles return g.outputFiles
} }
func (g *generator) GeneratedHeaderDir() android.Path { func (g *generator) GeneratedHeaderDirs() android.Paths {
return g.genPath return g.exportedIncludeDirs
} }
func (g *generator) DepsMutator(ctx android.BottomUpMutatorContext) { func (g *generator) DepsMutator(ctx android.BottomUpMutatorContext) {
@@ -113,7 +116,14 @@ func (g *generator) GenerateAndroidBuildActions(ctx android.ModuleContext) {
return return
} }
g.genPath = android.PathForModuleGen(ctx, "") if len(g.properties.Export_include_dirs) > 0 {
for _, dir := range g.properties.Export_include_dirs {
g.exportedIncludeDirs = append(g.exportedIncludeDirs,
android.PathForModuleGen(ctx, ctx.ModuleDir(), dir))
}
} else {
g.exportedIncludeDirs = append(g.exportedIncludeDirs, android.PathForModuleGen(ctx, ""))
}
tools := map[string]android.Path{} tools := map[string]android.Path{}
@@ -166,7 +176,7 @@ func (g *generator) GenerateAndroidBuildActions(ctx android.ModuleContext) {
} }
return "${depfile}", nil return "${depfile}", nil
case "genDir": case "genDir":
return g.genPath.String(), nil return android.PathForModuleGen(ctx, "").String(), nil
default: default:
if strings.HasPrefix(name, "location ") { if strings.HasPrefix(name, "location ") {
label := strings.TrimSpace(strings.TrimPrefix(name, "location ")) label := strings.TrimSpace(strings.TrimPrefix(name, "location "))