Skip to content

Commit fb7a5b8

Browse files
committed
[BuildScript] Don't install SwiftSyntax
Instead, link all binaries in the toolchain using SwiftSyntax (currently only `sk-stress-test`) should link to the binary statically. This simplifies the way SwiftSyntax is built and at the same time slightly reduces the toolchain size.
1 parent 1202d90 commit fb7a5b8

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

utils/swift_build_support/swift_build_support/products/swiftsyntax.py

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -94,17 +94,10 @@ def should_install(self, host_target):
9494
return self.args.install_swiftsyntax
9595

9696
def install(self, target_name):
97-
install_prefix = self.args.install_destdir + self.args.install_prefix
98-
99-
dylib_dir = os.path.join(install_prefix, 'lib')
100-
101-
additional_params = [
102-
'--dylib-dir', dylib_dir,
103-
'--install'
104-
]
105-
106-
self.run_swiftsyntax_build_script(target=target_name,
107-
additional_params=additional_params)
97+
# SwiftSyntax doesn't produce any products thate should be installed
98+
# into the toolchein. All tools using it link against SwiftSyntax
99+
# statically.
100+
pass
108101

109102
@classmethod
110103
def get_dependencies(cls):

0 commit comments

Comments
 (0)