Merge "Small readability refactor for bp2buildDepsMutator." into main am: d9cc9ec2ea
am: 7c738120f9
am: f14e69b82a
am: dfbbe6d69e
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2774747 Change-Id: I40ed291fc27b994865af2dd05cd4691c2beb0f2a Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
committed by
Automerger Merge Worker
commit
585baad876
@@ -693,15 +693,27 @@ func bp2buildDepsMutator(ctx BottomUpMutatorContext) {
|
|||||||
|
|
||||||
if len(ctx.Module().GetMissingBp2buildDeps()) > 0 {
|
if len(ctx.Module().GetMissingBp2buildDeps()) > 0 {
|
||||||
exampleDep := ctx.Module().GetMissingBp2buildDeps()[0]
|
exampleDep := ctx.Module().GetMissingBp2buildDeps()[0]
|
||||||
ctx.MarkBp2buildUnconvertible(bp2build_metrics_proto.UnconvertedReasonType_UNCONVERTED_DEP, exampleDep)
|
ctx.MarkBp2buildUnconvertible(
|
||||||
|
bp2build_metrics_proto.UnconvertedReasonType_UNCONVERTED_DEP, exampleDep)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Transitively mark modules unconvertible with the following set of conditions.
|
||||||
ctx.VisitDirectDeps(func(dep Module) {
|
ctx.VisitDirectDeps(func(dep Module) {
|
||||||
if dep.base().GetUnconvertedReason() != nil &&
|
if dep.base().GetUnconvertedReason() == nil {
|
||||||
dep.base().GetUnconvertedReason().ReasonType != int(bp2build_metrics_proto.UnconvertedReasonType_DEFINED_IN_BUILD_FILE) &&
|
return
|
||||||
ctx.OtherModuleDependencyTag(dep) == Bp2buildDepTag {
|
|
||||||
ctx.MarkBp2buildUnconvertible(bp2build_metrics_proto.UnconvertedReasonType_UNCONVERTED_DEP, dep.Name())
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if dep.base().GetUnconvertedReason().ReasonType ==
|
||||||
|
int(bp2build_metrics_proto.UnconvertedReasonType_DEFINED_IN_BUILD_FILE) {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
if ctx.OtherModuleDependencyTag(dep) != Bp2buildDepTag {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
|
||||||
|
ctx.MarkBp2buildUnconvertible(
|
||||||
|
bp2build_metrics_proto.UnconvertedReasonType_UNCONVERTED_DEP, dep.Name())
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user