Skip to content

Commit d06deee

Browse files
committed
Remove dependency on swift-system
This dependency is unused and just unnecessarily complicates things.
1 parent 688a840 commit d06deee

File tree

3 files changed

+2
-32
lines changed

3 files changed

+2
-32
lines changed

CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ find_package(dispatch QUIET)
6262
find_package(Foundation QUIET)
6363
find_package(Yams CONFIG REQUIRED)
6464
find_package(ArgumentParser CONFIG REQUIRED)
65-
find_package(SwiftSystem CONFIG REQUIRED)
6665

6766
add_subdirectory(Sources)
6867
add_subdirectory(cmake/modules)

README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,6 @@ all with CMake:
6464
```
6565
cmake -B <llbuild-build-dir> -G Ninja <llbuild-source-dir> -DLLBUILD_SUPPORT_BINDINGS="Swift" -DCMAKE_OSX_ARCHITECTURES=x86_64
6666
```
67-
* [swift-system](https://github.com/apple/swift-system)
6867
* [swift-argument-parser](https://github.com/apple/swift-argument-parser)
6968
* [Yams](https://github.com/jpsim/Yams)
7069

Utilities/build-script-helper.py

Lines changed: 2 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,6 @@ def install_swiftdriver(args, build_dir, prefix, targets) :
247247
# swift-driver: SwiftDriver.swiftmodule, SwiftOptions.swiftmodule
248248
# swift-tools-support-core: TSCUtility.swiftmodule, TSCBasic.swiftmodule
249249
# swift-argument-parser: ArgumentParser.swiftmodule (disabled until needed)
250-
# swift-system: SystemPackage.swiftmodule
251250
install_binary_swift_modules(args, build_dir, install_lib, targets)
252251

253252
# Modulemaps for C Modules:
@@ -324,11 +323,6 @@ def install_libraries(args, build_dir, universal_lib_dir, toolchain_lib_dir, tar
324323
install_library(args, build_dir, package_subpath, lib, shared_lib_ext,
325324
universal_lib_dir, toolchain_lib_dir, 'swift-tools-support-core', targets)
326325

327-
# Install the swift-system shared library into the toolchain lib
328-
package_subpath = os.path.join(args.configuration, 'dependencies', 'swift-system')
329-
install_library(args, build_dir, package_subpath, 'libSystemPackage', shared_lib_ext,
330-
universal_lib_dir, toolchain_lib_dir, 'swift-system', targets)
331-
332326
# Install the swift-argument-parser shared libraries into the toolchain lib
333327
package_subpath = os.path.join(args.configuration, 'dependencies', 'swift-argument-parser')
334328
for (lib, ext) in [('libArgumentParser', shared_lib_ext), ('libArgumentParserToolInfo', static_lib_ext)]:
@@ -376,11 +370,6 @@ def install_binary_swift_modules(args, build_dir, toolchain_lib_dir, targets):
376370
product_subpath)
377371
install_module(args, build_dir, package_subpath, toolchain_lib_dir, 'ArgumentParser', targets)
378372

379-
# swift-system
380-
package_subpath = os.path.join(args.configuration, 'dependencies', 'swift-system',
381-
product_subpath)
382-
install_module(args, build_dir, package_subpath, toolchain_lib_dir, 'SystemPackage', targets)
383-
384373

385374
# Install the modulemaps and headers of the driver's C module dependencies into the toolchain
386375
# include directory
@@ -442,9 +431,6 @@ def build_using_cmake(args, toolchain_bin, build_dir, targets):
442431
# LLBuild
443432
build_llbuild_using_cmake(args, target, swiftc_exec, dependencies_dir,
444433
base_cmake_flags, swift_flags)
445-
# SwiftSystem
446-
build_system_using_cmake(args, target, swiftc_exec, dependencies_dir,
447-
base_cmake_flags, swift_flags)
448434

449435
# TSC
450436
build_tsc_using_cmake(args, target, swiftc_exec, dependencies_dir,
@@ -485,24 +471,11 @@ def build_llbuild_using_cmake(args, target, swiftc_exec, build_dir, base_cmake_f
485471
cmake_build(args, swiftc_exec, llbuild_cmake_flags, llbuild_swift_flags,
486472
llbuild_source_dir, llbuild_build_dir, 'products/all')
487473

488-
def build_system_using_cmake(args, target, swiftc_exec, build_dir, base_cmake_flags, swift_flags):
489-
print('Building Swift Driver dependency: Swift System')
490-
system_source_dir = os.path.join(os.path.dirname(args.package_path), 'swift-system')
491-
system_build_dir = os.path.join(build_dir, 'swift-system')
492-
flags = [
493-
# required due to swift-autolink-extract bug ("The file was not recognized as a valid object file")
494-
"-DBUILD_SHARED_LIBS=YES"]
495-
system_cmake_flags = base_cmake_flags + flags
496-
system_swift_flags = swift_flags[:]
497-
cmake_build(args, swiftc_exec, system_cmake_flags, system_swift_flags,
498-
system_source_dir, system_build_dir)
499-
500474
def build_tsc_using_cmake(args, target, swiftc_exec, build_dir, base_cmake_flags, swift_flags):
501475
print('Building Swift Driver dependency: TSC')
502476
tsc_source_dir = os.path.join(os.path.dirname(args.package_path), 'swift-tools-support-core')
503477
tsc_build_dir = os.path.join(build_dir, 'swift-tools-support-core')
504-
flags = [
505-
'-DSwiftSystem_DIR=' + os.path.join(os.path.join(build_dir, 'swift-system'), 'cmake/modules')]
478+
flags = []
506479
tsc_cmake_flags = base_cmake_flags + flags
507480

508481
tsc_swift_flags = swift_flags[:]
@@ -550,8 +523,7 @@ def build_swift_driver_using_cmake(args, target, swiftc_exec, build_dir, base_cm
550523
'-DLLBuild_DIR=' + os.path.join(os.path.join(dependencies_dir, 'llbuild'), 'cmake/modules'),
551524
'-DTSC_DIR=' + os.path.join(os.path.join(dependencies_dir, 'swift-tools-support-core'), 'cmake/modules'),
552525
'-DYams_DIR=' + os.path.join(os.path.join(dependencies_dir, 'yams'), 'cmake/modules'),
553-
'-DArgumentParser_DIR=' + os.path.join(os.path.join(dependencies_dir, 'swift-argument-parser'), 'cmake/modules'),
554-
'-DSwiftSystem_DIR=' + os.path.join(os.path.join(dependencies_dir, 'swift-system'), 'cmake/modules')]
526+
'-DArgumentParser_DIR=' + os.path.join(os.path.join(dependencies_dir, 'swift-argument-parser'), 'cmake/modules')]
555527
driver_cmake_flags = base_cmake_flags + flags
556528
cmake_build(args, swiftc_exec, driver_cmake_flags, driver_swift_flags,
557529
driver_source_dir, driver_build_dir)

0 commit comments

Comments
 (0)