build: remove PythonKit integration

Author: compnerd

utils/build-script

@@ -219,7 +219,6 @@ def validate_arguments(toolchain, args):
     targets_needing_toolchain = [
         'build_indexstoredb',
         'build_playgroundsupport',
-        'build_pythonkit',
         'build_sourcekitlsp',
         'build_tensorflow_swift_apis',
         'build_toolchainbenchmarks',
@@ -273,7 +272,6 @@ def apply_default_arguments(toolchain, args):
     ninja_required = (
         args.build_foundation or
         args.build_indexstoredb or
-        args.build_pythonkit or
         args.build_sourcekitlsp or
         args.build_tensorflow_swift_apis or
         args.cmake_generator == 'Ninja'
@@ -891,8 +889,6 @@ class BuildScriptInvocation(object):
             product_classes.append(products.IndexStoreDB)
         if self.args.build_playgroundsupport:
             product_classes.append(products.PlaygroundSupport)
-        if self.args.build_pythonkit:
-            product_classes.append(products.PythonKit)
         if self.args.build_tensorflow_swift_apis:
             product_classes.append(products.TensorFlowSwiftAPIs)
         if self.args.build_sourcekitlsp:

utils/build_swift/build_swift/driver_arguments.py

@@ -598,8 +598,6 @@ def create_argument_parser():
            toggle_true('swiftsyntax_verify_generated_files'),
            help='set to verify that the generated files in the source tree '
                 'match the ones that would be generated from current master')
-    option(['--install-pythonkit'], toggle_true('install_pythonkit'),
-           help='install PythonKit')
     option(['--install-sourcekit-lsp'], toggle_true('install_sourcekitlsp'),
            help='install SourceKitLSP')
     option(['--install-skstresstester'], toggle_true('install_skstresstester'),
@@ -635,9 +633,6 @@ def create_argument_parser():
            store_true('install_playgroundsupport'),
            help='install playground support')
 
-    option('--pythonkit', store_true('build_pythonkit'),
-           help='build PythonKit')
-
     option('--tensorflow-swift-apis', store_true('build_tensorflow_swift_apis'),
            help='build TensorFlow Swift APIs')
     option('--install-tensorflow-swift-apis',
@@ -895,9 +890,6 @@ def create_argument_parser():
     option('--skip-test-cygwin', toggle_false('test_cygwin'),
            help='skip testing Swift stdlibs for Cygwin')
 
-    option('--test-pythonkit', toggle_true('test_pythonkit'),
-           help='skip testing PythonKit')
-
     # -------------------------------------------------------------------------
     in_group('Run build')
 

utils/build_swift/tests/expected_options.py

@@ -78,7 +78,6 @@
     'build_ninja': False,
     'build_osx': True,
     'build_playgroundsupport': False,
-    'build_pythonkit': False,
     'build_runtime_with_host_compiler': False,
     'build_stdlib_deployment_targets': ['all'],
     'build_subdir': None,
@@ -105,7 +104,6 @@
     'install_swiftsyntax': False,
     'swiftsyntax_verify_generated_files': False,
     'install_playgroundsupport': False,
-    'install_pythonkit': False,
     'install_sourcekitlsp': False,
     'install_skstresstester': False,
     'install_swiftevolve': False,
@@ -223,7 +221,6 @@
     'test_optimized': None,
     'test_osx': False,
     'test_paths': [],
-    'test_pythonkit': False,
     'test_swift_inspect': True,
     'test_tvos': False,
     'test_tvos_host': False,
@@ -468,11 +465,8 @@ class BuildScriptImplOption(_BaseOption):
     SetTrueOption('--maccatalyst', dest='maccatalyst'),
     SetTrueOption('--maccatalyst-ios-tests', dest='maccatalyst_ios_tests'),
     SetTrueOption('--playgroundsupport', dest='build_playgroundsupport'),
-    SetTrueOption('--pythonkit', dest='build_pythonkit'),
     SetTrueOption('--install-playgroundsupport',
                   dest='install_playgroundsupport'),
-    SetTrueOption('--install-pythonkit', dest='install_pythonkit'),
-    SetTrueOption('--test-pythonkit', dest='test_pythonkit'),
     SetTrueOption('--tensorflow-swift-apis', dest='build_tensorflow_swift_apis'),
     SetTrueOption('--install-tensorflow-swift-apis',
                   dest='install_tensorflow_swift_apis'),

utils/swift_build_support/swift_build_support/products/__init__.py

@@ -22,7 +22,6 @@
 from .llvm import LLVM
 from .ninja import Ninja
 from .playgroundsupport import PlaygroundSupport
-from .pythonkit import PythonKit
 from .skstresstester import SKStressTester
 from .sourcekitlsp import SourceKitLSP
 from .swift import Swift
@@ -48,7 +47,6 @@
     'LLVM',
     'Ninja',
     'PlaygroundSupport',
-    'PythonKit',
     'Swift',
     'SwiftFormat',
     'SwiftInspect',

utils/swift_build_support/swift_build_support/products/pythonkit.py

@@ -43,9 +43,6 @@
             "remote": { "id": "KitWare/CMake" },
             "platforms": [ "Linux" ]
         },
-        "pythonkit": {
-            "remote": { "id": "pvieito/PythonKit" }
-        },
         "tensorflow-swift-apis": {
             "remote": { "id": "tensorflow/swift-apis" }
         },
@@ -85,7 +82,6 @@
                 "indexstore-db": "master",
                 "sourcekit-lsp": "master",
                 "swift-format": "master",
-                "pythonkit": "master",
                 "tensorflow-swift-apis": "master"
             }
         },
@@ -117,7 +113,6 @@
                 "indexstore-db": "master",
                 "sourcekit-lsp": "master",
                 "swift-format": "master",
-                "pythonkit": "master",
                 "tensorflow-swift-apis": "master"
             }
         },
@@ -315,7 +310,6 @@
                 "indexstore-db": "release/5.3",
                 "sourcekit-lsp": "release/5.3",
                 "swift-format": "master",
-                "pythonkit": "master",
                 "tensorflow-swift-apis": "master"
             }
         },