build-script: ensure module group is used only when building SwiftSyntax library.

Author: nkcsgexi

build-script.py

@@ -163,7 +163,7 @@ def get_installed_dylib_name():
     return 'lib' + get_installed_name() + '.dylib'
 
 def get_swiftpm_invocation(spm_exec, build_dir, parser_header_dir,
-                           parser_lib_dir, release):
+                           parser_lib_dir, release, module_group=False):
     if spm_exec == 'swift build':
         swiftpm_call = ['swift', 'build']
     elif spm_exec == 'swift test':
@@ -192,41 +192,47 @@ def get_swiftpm_invocation(spm_exec, build_dir, parser_header_dir,
     swiftpm_call.extend(['-Xswiftc', '-enforce-exclusivity=unchecked'])
 
     # To build the group information into the module documentation file
-    # swiftpm_call.extend(['-Xswiftc', '-Xfrontend', '-Xswiftc', '-group-info-path'])
-    # swiftpm_call.extend(['-Xswiftc', '-Xfrontend', '-Xswiftc', GROUP_INFO_PATH])
+    if module_group:
+        swiftpm_call.extend(['-Xswiftc', '-Xfrontend', '-Xswiftc', '-group-info-path'])
+        swiftpm_call.extend(['-Xswiftc', '-Xfrontend', '-Xswiftc', GROUP_INFO_PATH])
 
     return swiftpm_call
 
-def build_swiftsyntax(swift_build_exec, swiftc_exec, build_dir,
-                      parser_header_dir, parser_lib_dir,
-                      build_test_util, release,
-                      verbose, disable_sandbox=False):
-    print('** Building SwiftSyntax **')
-
-    swiftpm_call = get_swiftpm_invocation(spm_exec=swift_build_exec,
-                                          build_dir=build_dir,
-                                          parser_header_dir=parser_header_dir,
-                                          parser_lib_dir=parser_lib_dir,
-                                          release=release)
-    swiftpm_call.extend(['--product', 'SwiftSyntax'])
-
-    if disable_sandbox:
-      swiftpm_call.append('--disable-sandbox')
-
-    # Only build lit-test-helper if we are planning to run tests
-    if build_test_util:
-        swiftpm_call.extend(['--product', 'lit-test-helper'])
-
-    if verbose:
-        swiftpm_call.extend(['--verbose'])
-    _environ = dict(os.environ)
-    _environ['SWIFT_EXEC'] = swiftc_exec
-    _environ['SWIFT_SYNTAX_BUILD_SCRIPT'] = ''
-    check_call(swiftpm_call, env=_environ, verbose=verbose)
+class Builder(object):
+  def __init__(self, swift_build_exec, swiftc_exec, build_dir,
+               parser_header_dir, parser_lib_dir,
+               release, verbose, disable_sandbox=False):
+      self.swift_build_exec = swift_build_exec
+      self.swiftc_exec = swiftc_exec
+      self.build_dir = build_dir
+      self.parser_header_dir = parser_header_dir
+      self.parser_lib_dir = parser_lib_dir
+      self.release = release
+      self.verbose = verbose
+      self.disable_sandbox = disable_sandbox
+
+  def build(self, product_name, module_group=False):
+      print('** Building ' + product_name + ' **')
+      swiftpm_call = get_swiftpm_invocation(spm_exec=self.swift_build_exec,
+                                            build_dir=self.build_dir,
+                                            parser_header_dir=self.parser_header_dir,
+                                            parser_lib_dir=self.parser_lib_dir,
+                                            release=self.release,
+                                            module_group=module_group)
+      swiftpm_call.extend(['--product', product_name])
+
+      if self.disable_sandbox:
+        swiftpm_call.append('--disable-sandbox')
+
+      if self.verbose:
+          swiftpm_call.extend(['--verbose'])
+      _environ = dict(os.environ)
+      _environ['SWIFT_EXEC'] = self.swiftc_exec
+      _environ['SWIFT_SYNTAX_BUILD_SCRIPT'] = ''
+      check_call(swiftpm_call, env=_environ, verbose=self.verbose)
 
 
 ## Testing
-
 def run_tests(swift_test_exec, build_dir, parser_header_dir, parser_lib_dir,
               release, swift_build_exec, filecheck_exec, swiftc_exec, verbose):
     print('** Running SwiftSyntax Tests **')
@@ -541,15 +547,19 @@ def main():
       sys.exit(0)
 
     try:
-        build_swiftsyntax(swift_build_exec=args.swift_build_exec,
+        builder = Builder(swift_build_exec=args.swift_build_exec,
                           swiftc_exec=args.swiftc_exec,
                           build_dir=args.build_dir,
                           parser_header_dir=args.syntax_parser_header_dir,
                           parser_lib_dir=args.syntax_parser_lib_dir,
-                          build_test_util=args.test,
                           release=args.release,
                           verbose=args.verbose,
                           disable_sandbox=args.disable_sandbox)
+        builder.build('SwiftSyntax', module_group=True)
+
+        # Only build lit-test-helper if we are planning to run tests
+        if args.test:
+          builder.build('lit-test-helper')
     except subprocess.CalledProcessError as e:
         printerr('Error: Building SwiftSyntax failed')
         printerr('Executing: %s' % ' '.join(e.cmd))