Skip to content

Commit

Permalink
Revert "Forward package_name to swift_library part of apple_library (b…
Browse files Browse the repository at this point in the history
…azel-ios#853)"

This reverts commit 3b0ca4c.
  • Loading branch information
Leonid Melnyk committed Apr 5, 2024
1 parent 516adfe commit b25f514
Showing 1 changed file with 13 additions and 13 deletions.
26 changes: 13 additions & 13 deletions rules/library.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -27,12 +27,6 @@ GLOBAL_INDEX_STORE_PATH = "bazel-out/_global_index_store"

_MANUAL = ["manual"]

_SWIFT_LIBRARY_KWARGS = [
"always_include_developer_search_paths",
"package_name",
"plugins",
]

def _private_headers_impl(ctx):
return [
PrivateHeadersInfo(
Expand Down Expand Up @@ -584,10 +578,8 @@ def apple_library(
defines = kwargs.pop("defines", [])
testonly = kwargs.pop("testonly", False)
features = kwargs.pop("features", [])

# Collect the swift_library related kwargs, these are typically only set when provided to allow
# for wider compatibility with rule_swift versions.
swift_kwargs = {arg: kwargs.pop(arg) for arg in _SWIFT_LIBRARY_KWARGS if arg in kwargs}
plugins = kwargs.pop("plugins", None)
always_include_developer_search_paths = kwargs.pop("always_include_developer_search_paths", None)

for (k, v) in {"momc_copts": momc_copts, "mapc_copts": mapc_copts, "ibtool_copts": ibtool_copts}.items():
if v:
Expand Down Expand Up @@ -966,8 +958,16 @@ def apple_library(
)

if swift_sources:
# Forward the kwargs and the swift specific kwargs to the swift_library
swift_library_kwargs = dicts.add(kwargs, swift_kwargs)
swift_kwargs = kwargs

# To be backward compatible with rules_apple 2.x
if plugins:
swift_kwargs = dicts.add(swift_kwargs, {"plugins": plugins})

# Allow consumers to opt-in to https://github.com/bazelbuild/rules_swift/pull/1162
if always_include_developer_search_paths:
swift_kwargs = dicts.add(swift_kwargs, {"always_include_developer_search_paths": always_include_developer_search_paths})

swift_library(
name = swift_libname,
module_name = module_name,
Expand All @@ -993,7 +993,7 @@ def apple_library(
tags = tags_manual,
defines = defines + swift_defines,
testonly = testonly,
**swift_library_kwargs
**swift_kwargs
)
lib_names.append(swift_libname)

Expand Down

0 comments on commit b25f514

Please sign in to comment.