Skip to content

Commit 41a773f

Browse files
authored
Merge pull request #1804 from ahoppen/remove-unnecessary-fault
Don’t log “Did not compute dependents for target”
2 parents 5c7d495 + 69e9a5c commit 41a773f

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
@@ -987,14 +987,11 @@ package actor BuildSystemManager: QueueBasedMessageHandler {
987987
logger.fault("Did not compute depth for target \(buildTarget.id)")
988988
depth = 0
989989
}
990-
let targetDependents: Set<BuildTargetIdentifier>
991-
if let d = dependents[buildTarget.id] {
992-
targetDependents = d
993-
} else {
994-
logger.fault("Did not compute dependents for target \(buildTarget.id)")
995-
targetDependents = []
996-
}
997-
result[buildTarget.id] = BuildTargetInfo(target: buildTarget, depth: depth, dependents: targetDependents)
990+
result[buildTarget.id] = BuildTargetInfo(
991+
target: buildTarget,
992+
depth: depth,
993+
dependents: dependents[buildTarget.id] ?? []
994+
)
998995
}
999996
return result
1000997
}

0 commit comments

Comments
 (0)