tequilaOS/platform_build_blueprint
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Rebootstrap for changes to any bootstrap go binaries

Browse source 
The final build.ninja can't contain a rules to build the primary
builder or it will create a circular dependency, so rules for all
bootstrap go packages and binaries are replaced with phony targets
that trigger a re-bootstrap to rebuild.  Add all bootstrap go
binaries to the dependencies that trigger a rebuild so that they
all get rebuilt.

Also fix the phony rule generation to include rules for the
intermediate files of bootstrap go binaries so that they don't
get deleted by the cleanup phase when the primary builder runs.

Change-Id: Id0cc287789f74cc4b21b062086723712b57ee051
This commit is contained in:
Colin Cross 2015-01-12 17:40:06 -08:00
parent 41c397af78
commit 3793062859
1 changed files with 28 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

43
blueprint/bootstrap/bootstrap.go
View file

@ -180,7 +180,7 @@ func (g *goPackage) GenerateBuildActions(ctx blueprint.ModuleContext) {
buildGoPackage(ctx, g.pkgRoot, g.properties.PkgPath, g.archiveFile,
g.properties.Srcs)
} else {
phonyGoTarget(ctx, g.archiveFile, g.properties.Srcs)
phonyGoTarget(ctx, g.archiveFile, g.properties.Srcs, nil)
}
}
@ -241,7 +241,8 @@ func (g *goBinary) GenerateBuildActions(ctx blueprint.ModuleContext) {
Inputs: []string{aoutFile},
})
} else {
phonyGoTarget(ctx, binaryFile, g.properties.Srcs)
intermediates := []string{aoutFile, archiveFile}
phonyGoTarget(ctx, binaryFile, g.properties.Srcs, intermediates)
}
}
@ -279,7 +280,9 @@ func buildGoPackage(ctx blueprint.ModuleContext, pkgRoot string,
})
}
func phonyGoTarget(ctx blueprint.ModuleContext, target string, srcs []string) {
func phonyGoTarget(ctx blueprint.ModuleContext, target string, srcs []string,
intermediates []string) {
var depTargets []string
ctx.VisitDepsDepthFirstIf(isGoPackageProducer,
func(module blueprint.Module) {
@ -309,6 +312,17 @@ func phonyGoTarget(ctx blueprint.ModuleContext, target string, srcs []string) {
Outputs: []string{src},
})
}
// If there is no rule to build the intermediate files of a bootstrap go package
// the cleanup phase of the primary builder will delete the intermediate files,
// forcing an unnecessary rebuild. Add phony rules for all of them.
for _, intermediate := range intermediates {
ctx.Build(pctx, blueprint.BuildParams{
Rule: blueprint.Phony,
Outputs: []string{intermediate},
})
}
}
type singleton struct{}
@ -448,23 +462,22 @@ func (s *singleton) GenerateBuildActions(ctx blueprint.SingletonContext) {
} else {
// We're generating a non-bootstrapper Ninja file, so we need to set it
// up to depend on the bootstrapper Ninja file. The build.ninja target
// also has an implicit dependency on the primary builder, which will
// have a phony dependency on all its sources. This will cause any
// changes to the primary builder's sources to trigger a re-bootstrap
// operation, which will rebuild the primary builder.
// also has an implicit dependency on the primary builder and all other
// bootstrap go binaries, which will have phony dependencies on all of
// their sources. This will cause any changes to a bootstrap binary's
// sources to trigger a re-bootstrap operation, which will rebuild the
// binary.
//
// On top of that we need to use the depfile generated by the bigbp
// rule. We do this by depending on that file and then setting up a
// phony rule to generate it that uses the depfile.
buildNinjaDeps := []string{"$bootstrapCmd", mainNinjaFile}
buildNinjaDeps = append(buildNinjaDeps, allGoBinaries...)
ctx.Build(pctx, blueprint.BuildParams{
Rule: rebootstrap,
Outputs: []string{"build.ninja"},
Inputs: []string{"$bootstrapManifest"},
Implicits: []string{
"$bootstrapCmd",
primaryBuilderFile,
mainNinjaFile,
},
Rule: rebootstrap,
Outputs: []string{"build.ninja"},
Inputs: []string{"$bootstrapManifest"},
Implicits: buildNinjaDeps,
})
ctx.Build(pctx, blueprint.BuildParams{