Update for rules_swift 3.0

Author: luispadron

MODULE.bazel

@@ -22,7 +22,7 @@ bazel_dep(
 )
 bazel_dep(
     name = "rules_apple",
-    version = "3.15.0",
+    version = "4.0.1",
     repo_name = "build_bazel_rules_apple",
 )
 bazel_dep(
@@ -31,8 +31,8 @@ bazel_dep(
 )
 bazel_dep(
     name = "rules_swift",
-    version = "1.18.0",
-    max_compatibility_level = 2,
+    version = "3.0.2",
+    max_compatibility_level = 3,
     repo_name = "build_bazel_rules_swift",
 )
 bazel_dep(

rules/framework.bzl

@@ -25,7 +25,8 @@ load("@build_bazel_rules_apple//apple/internal:apple_toolchains.bzl", "AppleMacT
 load("@build_bazel_rules_apple//apple/internal:swift_support.bzl", "swift_support")
 load("@build_bazel_rules_apple//apple/internal/utils:clang_rt_dylibs.bzl", "clang_rt_dylibs")
 load("@build_bazel_rules_apple//apple/internal:providers.bzl", "AppleBundleInfo", "ApplePlatformInfo", "IosFrameworkBundleInfo", "new_applebundleinfo", "new_iosframeworkbundleinfo")
-load("@build_bazel_rules_swift//swift:swift.bzl", "SwiftInfo", "swift_clang_module_aspect", "swift_common")
+load("@build_bazel_rules_swift//swift:providers.bzl", "SwiftInfo", "create_clang_module_inputs", "create_swift_module_context", "create_swift_module_inputs")
+load("@build_bazel_rules_swift//swift:swift.bzl", "swift_clang_module_aspect", "swift_common")
 load(
     "@build_bazel_rules_apple//apple/internal/aspects:resource_aspect.bzl",
     "apple_resource_aspect",
@@ -558,7 +559,7 @@ def _create_swiftmodule(attrs):
     if attrs.symbol_graph:
         kwargs["symbol_graph"] = attrs.symbol_graph
 
-    return swift_common.create_swift_module(
+    return create_swift_module_inputs(
         swiftdoc = attrs.swiftdoc,
         swiftmodule = attrs.swiftmodule,
         swiftinterface = attrs.swiftinterface,
@@ -585,7 +586,7 @@ def _copy_swiftmodule(ctx, framework_files, clang_module):
     return [
         # only add the swift module, the objc modulemap is already listed as a header,
         # and it will be discovered via the framework search path
-        swift_common.create_module(
+        create_swift_module_context(
             name = swiftmodule_name,
             clang = clang_module,
             swift = swift_module,
@@ -600,13 +601,13 @@ def _get_merged_swift_info(ctx, framework_files, transitive_deps, clang_module):
         swift_info_fields["modules"] = _copy_swiftmodule(ctx, framework_files, clang_module)
     else:
         swift_info_fields["modules"] = [
-            swift_common.create_module(
+            create_swift_module_context(
                 name = ctx.attr.framework_name,
                 clang = clang_module,
             ),
         ]
 
-    return swift_common.create_swift_info(**swift_info_fields)
+    return SwiftInfo(**swift_info_fields)
 
 def _merge_root_infoplists(ctx):
     if ctx.attr.infoplists == None or len(ctx.attr.infoplists) == 0:
@@ -1104,15 +1105,15 @@ def _apple_framework_packaging_impl(ctx):
 
     # rules_swift 2.x no longers takes compilation_context from CcInfo, need to pass it in via SwiftInfo's clang_module
     if virtualize_frameworks:
-        clang_module = swift_common.create_clang_module(
+        clang_module = create_clang_module_inputs(
             module_map = None,
             compilation_context = compilation_context,
         )
     else:
         # Setup the `clang` attr of the Swift module for non-vfs case this is required to have it locate the modulemap
         # and headers correctly.
         module_map = outputs.modulemaps[0] if outputs.modulemaps else None
-        clang_module = swift_common.create_clang_module(
+        clang_module = create_clang_module_inputs(
             module_map = module_map,
             compilation_context = cc_common.create_compilation_context(
                 headers = depset(_compact(

rules/framework/vfs_overlay.bzl

@@ -10,7 +10,7 @@ to each of the entries.
 """
 
 load("@bazel_tools//tools/cpp:toolchain_utils.bzl", "find_cpp_toolchain", "use_cpp_toolchain")
-load("@build_bazel_rules_swift//swift:swift.bzl", "swift_common")
+load("@build_bazel_rules_swift//swift:providers.bzl", "SwiftInfo")
 load("//rules:features.bzl", "feature_names")
 load("//rules:providers.bzl", "FrameworkInfo")
 
@@ -298,7 +298,7 @@ def _framework_vfs_overlay_impl(ctx):
             files = depset([vfs.vfsoverlay_file]),
             vfs_info = vfs.vfs_info,
         ),
-        swift_common.create_swift_info(),
+        SwiftInfo(),
     ]
 
 def _merge_vfs_infos(base, vfs_infos):

rules/transition_support.bzl

@@ -252,7 +252,6 @@ def _is_arch_supported_for_target_tuple(*, environment_arch, minimum_os_version,
 def _command_line_options(
         *,
         apple_platforms = [],
-        emit_swiftinterface = False,
         environment_arch = None,
         force_bundle_outputs = False,
         minimum_os_version,
@@ -267,8 +266,6 @@ def _command_line_options(
             first element will be applied to `platforms` as that will be what is resolved by the
             underlying rule. Defaults to an empty list, which will signal to Bazel that platform
             mapping can take place as a fallback measure.
-        emit_swiftinterface: Wheither to emit swift interfaces for the given target. Defaults to
-            `False`.
         environment_arch: A valid Apple environment when applicable with its architecture as a
             string (for example `sim_arm64` from `ios_sim_arm64`, or `arm64` from `ios_arm64`), or
             None to infer a value from command line options passed through settings.
@@ -339,7 +336,6 @@ def _command_line_options(
             platform = "watchos",
             platform_type = platform_type,
         ),
-        "@build_bazel_rules_swift//swift:emit_swiftinterface": emit_swiftinterface,
     }
 
 def _apple_rule_base_transition_impl(settings, attr):
@@ -351,7 +347,6 @@ def _apple_rule_base_transition_impl(settings, attr):
     # PATCH: end
 
     return _command_line_options(
-        emit_swiftinterface = hasattr(attr, "_emitswiftinterface"),
         environment_arch = _environment_archs(platform_type, settings)[0],
         minimum_os_version = minimum_os_version,
         platform_type = platform_type,
@@ -401,7 +396,6 @@ _apple_rule_base_transition_outputs = [
     "//command_line_option:platforms",
     "//command_line_option:tvos_minimum_os",
     "//command_line_option:watchos_minimum_os",
-    "@build_bazel_rules_swift//swift:emit_swiftinterface",
 ]
 
 _apple_rule_base_transition = transition(
@@ -420,11 +414,6 @@ def _apple_platform_split_transition_impl(settings, attr):
     minimum_os_version = _rules_ios_minimum_os_version(platform_type, attr)
     # PATCH: end
 
-    # iOS and tvOS static frameworks require underlying swift_library targets generate a Swift
-    # interface file. These rules define a private attribute called `_emitswiftinterface` that
-    # let's this transition flip rules_swift config down the build graph.
-    emit_swiftinterface = hasattr(attr, "_emitswiftinterface")
-
     if settings["//command_line_option:incompatible_enable_apple_toolchain_resolution"]:
         platforms = (
             settings["//command_line_option:apple_platforms"] or
@@ -445,7 +434,6 @@ def _apple_platform_split_transition_impl(settings, attr):
             if str(platform) not in output_dictionary:
                 output_dictionary[str(platform)] = _command_line_options(
                     apple_platforms = apple_platforms,
-                    emit_swiftinterface = emit_swiftinterface,
                     minimum_os_version = minimum_os_version,
                     platform_type = platform_type,
                     settings = settings,
@@ -492,7 +480,6 @@ def _apple_platform_split_transition_impl(settings, attr):
                 continue
 
             output_dictionary[found_cpu] = _command_line_options(
-                emit_swiftinterface = emit_swiftinterface,
                 environment_arch = environment_arch,
                 minimum_os_version = minimum_os_version,
                 platform_type = platform_type,