Skip to content

Commit 9d43e33

Browse files
committed
[Build Script Helper] Remove dependency on llbuild in the install action
No longer needed with swiftlang#263
1 parent 959c9f7 commit 9d43e33

File tree

1 file changed

+6
-22
lines changed

1 file changed

+6
-22
lines changed

Utilities/build-script-helper.py

Lines changed: 6 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -210,12 +210,12 @@ def install(args, build_dir):
210210

211211
# Binary Swift Modules:
212212
# swift-driver: SwiftDriver.swiftmodule, SwiftOptions.swiftmodule
213-
# TODO: swift-argument-parser: ArgumentParser.swiftmodule
213+
# TODO: swift-argument-parser: ArgumentParser.swiftmodule (disabled until needed)
214214
# swift-tools-support-core: TSCUtility.swiftmodule, TSCLibc.swiftmodule, TSCBasic.swiftmodule
215215
install_binary_swift_modules(args, build_dir, toolchain_lib)
216216

217217
# Modulemaps for C Modules:
218-
# llbuild, TSCclibc
218+
# TSCclibc
219219
install_c_module_includes(args, build_dir, toolchain_include)
220220

221221
# Install universal binaries for swift-driver and swift-help into the toolchain bin
@@ -233,13 +233,7 @@ def install_executables(args, build_dir, universal_bin_dir, toolchain_bin_dir):
233233
delete_rpath(driver_lib_dir_path, exe_bin_path, args.verbose)
234234
delete_rpath(tsc_lib_dir_path, exe_bin_path, args.verbose)
235235

236-
# Only swift-driver relies libllbuild
237-
if exe == 'swift-driver':
238-
llbuild_lib_dir_path = os.path.join(build_dir, arch + '-apple-macos' + macos_deployment_target,
239-
'llbuild', 'lib')
240-
delete_rpath(llbuild_lib_dir_path, exe_bin_path, args.verbose)
241-
242-
# Point to the installed toolchain's lib
236+
# Point to the installation toolchain's lib directory
243237
add_rpath('@executable_path/../lib/swift/macosx', exe_bin_path, args.verbose)
244238

245239
# Merge the multiple architecture binaries into a universal binary and install
@@ -256,16 +250,13 @@ def install_executables(args, build_dir, universal_bin_dir, toolchain_bin_dir):
256250

257251
# Install shared libraries for the driver and its dependencies into the toolchain
258252
def install_libraries(args, build_dir, universal_lib_dir, toolchain_lib_dir):
259-
# Fixup the llbuild and SwiftDriver rpath for libSwiftDriver
253+
# Fixup the SwiftDriver rpath for libSwiftDriver
260254
for arch in macos_target_architectures:
261255
lib_path = os.path.join(build_dir, arch + '-apple-macos' + macos_deployment_target,
262256
'swift-driver', 'lib', 'libSwiftDriver' + shared_lib_ext)
263257
driver_lib_dir_path = os.path.join(build_dir, arch + '-apple-macos' + macos_deployment_target,
264258
'swift-driver', 'lib')
265-
llbuild_lib_dir_path = os.path.join(build_dir, arch + '-apple-macos' + macos_deployment_target,
266-
'llbuild', 'lib')
267259
delete_rpath(driver_lib_dir_path, lib_path, args.verbose)
268-
delete_rpath(llbuild_lib_dir_path, lib_path, args.verbose)
269260

270261
# Fixup the TSC rpath for libSwiftDriver and libSwiftOptions
271262
for lib in ['libSwiftDriver', 'libSwiftOptions']:
@@ -317,10 +308,6 @@ def install_binary_swift_modules(args, build_dir, toolchain_lib_dir):
317308
# TODO: Until the argument parser is used, avoid installing it into the toolchain
318309
#install_module(args, build_dir, product_subpath, toolchain_lib_dir, 'ArgumentParser', 'swift-argument-parser')
319310

320-
# llbuildSwift
321-
llbuild_product_subpath = os.path.join('products', 'llbuildSwift')
322-
install_module(args, build_dir, llbuild_product_subpath, toolchain_lib_dir, 'llbuildSwift', 'llbuild')
323-
324311
# Install the modulemaps and headers of the driver's C module dependencies into the toolchain
325312
# include directory
326313
def install_c_module_includes(args, build_dir, toolchain_include_dir):
@@ -329,11 +316,6 @@ def install_c_module_includes(args, build_dir, toolchain_include_dir):
329316
'TSCclibc', 'include')
330317
install_include_artifacts(args, toolchain_include_dir, tscc_include_dir, 'TSCclibc')
331318

332-
# llbuild C module's modulemap and header files
333-
llbuild_include_dir = os.path.join(os.path.dirname(args.package_path), 'llbuild', 'products',
334-
'libllbuild', 'include')
335-
install_include_artifacts(args, toolchain_include_dir, llbuild_include_dir, 'llbuild')
336-
337319
def install_module(args, build_dir, product_subpath, toolchain_lib, module_name, source_package):
338320
toolchain_module_dir = os.path.join(toolchain_lib, module_name + '.swiftmodule')
339321
mkdir_p(toolchain_module_dir)
@@ -372,6 +354,8 @@ def build_for_toolchain_install(args, toolchain_bin, build_dir):
372354
# Target directory to build distribution artifacts
373355
cmake_target_dir = os.path.join(build_dir, target)
374356
# LLBuild
357+
# We build llbuild for the sake of building the entirety of SwiftDriver project,
358+
# but it is not a dependency of any of the installed artifacts, nor is it itself installed.
375359
build_llbuild_using_cmake(args, target, swiftc_exec, cmake_target_dir, base_cmake_flags)
376360
# TSC
377361
build_tsc_using_cmake(args, target, swiftc_exec, cmake_target_dir, base_cmake_flags)

0 commit comments

Comments
 (0)