Skip to content

Commit 69e9a5c

Browse files
committed
Don’t log “Did not compute dependents for target”
Leaf targets don’t have dependents and that’s expected. We shouldn’t log a fault for them.
1 parent 87b9285 commit 69e9a5c

File tree

1 file changed

+5
-8
lines changed

1 file changed

+5
-8
lines changed

Sources/BuildSystemIntegration/BuildSystemManager.swift

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -960,14 +960,11 @@ package actor BuildSystemManager: QueueBasedMessageHandler {
960960
logger.fault("Did not compute depth for target \(buildTarget.id)")
961961
depth = 0
962962
}
963-
let targetDependents: Set<BuildTargetIdentifier>
964-
if let d = dependents[buildTarget.id] {
965-
targetDependents = d
966-
} else {
967-
logger.fault("Did not compute dependents for target \(buildTarget.id)")
968-
targetDependents = []
969-
}
970-
result[buildTarget.id] = BuildTargetInfo(target: buildTarget, depth: depth, dependents: targetDependents)
963+
result[buildTarget.id] = BuildTargetInfo(
964+
target: buildTarget,
965+
depth: depth,
966+
dependents: dependents[buildTarget.id] ?? []
967+
)
971968
}
972969
return result
973970
}

0 commit comments

Comments
 (0)