Merge "Don't run SoongBuildInvocation for bp2build + dist"
This commit is contained in:
@@ -721,10 +721,6 @@ func (c *configImpl) Arguments() []string {
|
||||
}
|
||||
|
||||
func (c *configImpl) SoongBuildInvocationNeeded() bool {
|
||||
if c.Dist() {
|
||||
return true
|
||||
}
|
||||
|
||||
if len(c.Arguments()) > 0 {
|
||||
// Explicit targets requested that are not special targets like b2pbuild
|
||||
// or the JSON module graph
|
||||
@@ -736,6 +732,11 @@ func (c *configImpl) SoongBuildInvocationNeeded() bool {
|
||||
return true
|
||||
}
|
||||
|
||||
// bp2build + dist may be used to dist bp2build logs but does not require SoongBuildInvocation
|
||||
if c.Dist() && !c.Bp2Build() {
|
||||
return true
|
||||
}
|
||||
|
||||
// build.ninja doesn't need to be generated
|
||||
return false
|
||||
}
|
||||
|
Reference in New Issue
Block a user