Merge "Ensure that genrule's host tool deps are for host" into main am: 74ff1f80bc

Original change: https://android-review.googlesource.com/c/platform/build/soong/+/3000611

Change-Id: I72e402e56ab8182131b613b9dab7bc42712e33bd
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Jiyong Park
2024-03-22 23:53:33 +00:00
committed by Automerger Merge Worker

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 {