build-script: Add --install-wasmkit option (#82871)

This allows skipping the installation for WasmKit when building it, which is consistent to how other products are handled by `build-script`, where they have separate `--install` options. Existing behavior is preserved: passing `--wasmkit` alone will install it, but now `--wasmkit --install-wasmkit false` will skip its installation step.

Author: MaxDesiatov

utils/build-presets.ini

@@ -862,6 +862,7 @@ swift-testing-macros
 xctest
 swiftdocc
 swiftformat
+wasmkit
 
 build-ninja
 install-llvm
@@ -879,12 +880,11 @@ install-prefix=/usr
 install-sourcekit-lsp
 install-swiftformat
 install-swiftdocc
+install-wasmkit
 build-swift-static-stdlib
 build-swift-static-sdk-overlay
 build-swift-stdlib-unittest-extra
 build-embedded-stdlib-cross-compiling
-
-wasmkit
 build-wasm-stdlib
 
 # Executes the lit tests for the installable package that is created
@@ -958,6 +958,7 @@ mixin-preset=buildbot_linux
 
 llvm-targets-to-build=X86;ARM;AArch64;WebAssembly
 wasmkit
+install-wasmkit
 build-wasm-stdlib
 # FIXME(katei): Test fails on a specific CI node.
 # https://github.com/apple/swift/issues/70980
@@ -1416,6 +1417,7 @@ install-playgroundsupport
 install-sourcekit-lsp
 install-swiftformat
 install-swiftdocc
+install-wasmkit
 
 install-destdir=%(install_destdir)s
 

utils/build_swift/build_swift/driver_arguments.py

@@ -823,11 +823,18 @@ def create_argument_parser():
     option(['--build-minimal-stdlib'], toggle_true('build_minimalstdlib'),
            help='build the \'minimal\' freestanding stdlib variant into a '
                 'separate build directory ')
+
+    # Wasm options
+
     option(['--build-wasm-stdlib'], toggle_true('build_wasmstdlib'),
            help='build the stdlib for WebAssembly target into a'
                 'separate build directory ')
     option(['--wasmkit'], toggle_true('build_wasmkit'),
            help='build WasmKit')
+    option(['--install-wasmkit'], toggle_true('install_wasmkit'),
+           help='install SourceKitLSP')
+
+    # Swift Testing options
 
     option('--swift-testing', toggle_true('build_swift_testing'),
            help='build Swift Testing')

utils/build_swift/tests/expected_options.py

@@ -119,6 +119,7 @@
     'install_swift_testing_macros': False,
     'install_swift_driver': False,
     'install_swiftdocc': False,
+    'install_wasmkit': False,
     'swiftsyntax_verify_generated_files': False,
     'swiftsyntax_enable_rawsyntax_validation': False,
     'swiftsyntax_enable_test_fuzzing': False,
@@ -684,6 +685,7 @@ class BuildScriptImplOption(_BaseOption):
     EnableOption('--install-swiftformat', dest='install_swiftformat'),
     EnableOption('--install-skstresstester', dest='install_skstresstester'),
     EnableOption('--install-swiftdocc', dest='install_swiftdocc'),
+    EnableOption('--install-wasmkit', dest='install_wasmkit'),
     EnableOption('--toolchain-benchmarks', dest='build_toolchainbenchmarks'),
     EnableOption('--swift-inspect', dest='build_swift_inspect'),
     EnableOption('--tsan-libdispatch-test'),

utils/swift_build_support/swift_build_support/products/wasmkit.py

@@ -47,8 +47,7 @@ def should_test(self, host_target):
         return False
 
     def should_install(self, host_target):
-        # Currently, it's only used for testing stdlib.
-        return True
+        return self.args.install_wasmkit
 
     def install(self, host_target):
         """
@@ -61,10 +60,10 @@ def install(self, host_target):
 
     def build(self, host_target):
         bin_path = run_swift_build(host_target, self, 'wasmkit-cli')
-        print("Built wasmkit-cli at: " + bin_path)
+        print("Built wasmkit-cli at: " + bin_path, flush=True)
         # Copy the built binary to ./bin
         dest_bin_path = self.__class__.cli_file_path(self.build_dir)
-        print("Copying wasmkit-cli to: " + dest_bin_path)
+        print("Copying wasmkit-cli to: " + dest_bin_path, flush=True)
         os.makedirs(os.path.dirname(dest_bin_path), exist_ok=True)
         shutil.copy(bin_path, dest_bin_path)