Merge "Ensure that genrule's host tool deps are for host" into main

This commit is contained in:
Jiyong Park
2024-03-22 23:33:48 +00:00
committed by Gerrit Code Review

View File

@@ -317,7 +317,17 @@ func (g *Module) generateCommonBuildActions(ctx android.ModuleContext) {
// required relative locations of the tool and its dependencies, use those
// instead. They will be copied to those relative locations in the sbox
// sandbox.
packagedTools = append(packagedTools, specs...)
// Care must be taken since TransitivePackagingSpec may return device-side
// paths via the required property. Filter them out.
for i, ps := range specs {
if ps.Partition() != "" {
if i == 0 {
panic("first PackagingSpec is assumed to be the host-side tool")
}
continue
}
packagedTools = append(packagedTools, ps)
}
// Assume that the first PackagingSpec of the module is the tool.
addLocationLabel(tag.label, packagedToolLocation{specs[0]})
} else {