Skip to content

Commit 2024a70

Browse files
committed
WIP: Don't link plugin dependencies to products
This doesn't look right, but a comment was changed when the code change was made, suggesting it was intentional.
1 parent 979dabc commit 2024a70

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Sources/Build/BuildPlan.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -705,16 +705,16 @@ public class BuildPlan: SPMBuildCore.BuildPlan {
705705
return target.dependencies.filter { $0.satisfies(self.buildEnvironment) }
706706

707707
// For a product dependency, we only include its content only if we
708-
// need to statically link it or if it's a plugin.
708+
// need to statically link it.
709709
case .product(let product, _):
710710
guard dependency.satisfies(self.buildEnvironment) else {
711711
return []
712712
}
713713

714714
switch product.type {
715-
case .library(.automatic), .library(.static), .plugin:
715+
case .library(.automatic), .library(.static):
716716
return product.targets.map { .target($0, conditions: []) }
717-
case .library(.dynamic), .test, .executable, .snippet, .macro:
717+
case .library(.dynamic), .test, .executable, .snippet, .macro, .plugin:
718718
return []
719719
}
720720
}

0 commit comments

Comments
 (0)