Skip to content

Commit bcb57da

Browse files
committed
Replace apple_common.compilation_support.get_*_library_for_linking
1 parent 43b9591 commit bcb57da

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

apple/internal/compilation_support.bzl

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,14 @@ def _build_fully_linked_variable_extensions(archive, libs):
5959
extensions["imported_library_exec_paths"] = []
6060
return extensions
6161

62+
def _get_static_library_for_linking(library_to_link):
63+
if library_to_link.static_library:
64+
return library_to_link.static_library
65+
elif library_to_link.pic_static_library:
66+
return library_to_link.pic_static_library
67+
else:
68+
return None
69+
6270
def _get_library_for_linking(library_to_link):
6371
if library_to_link.static_library:
6472
return library_to_link.static_library
@@ -566,6 +574,8 @@ def _register_configuration_specific_link_actions_with_objc_variables(
566574

567575
compilation_support = struct(
568576
# TODO(b/331163513): Move apple_common.compliation_support.build_common_variables here, too.
577+
get_library_for_linking = _get_library_for_linking,
578+
get_static_library_for_linking = _get_static_library_for_linking,
569579
register_fully_link_action = _register_fully_link_action,
570580
register_configuration_specific_link_actions = _register_configuration_specific_link_actions,
571581
)

apple/internal/multi_arch_binary_support.bzl

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
"""Support utility for creating multi-arch Apple binaries."""
1616

1717
load("@rules_cc//cc/common:cc_common.bzl", "cc_common")
18+
load("//apple/internal:compilation_support.bzl", "compilation_support")
1819

1920
visibility([
2021
"//apple/...",
@@ -26,7 +27,7 @@ def _build_avoid_library_set(avoid_dep_linking_contexts):
2627
for linking_context in avoid_dep_linking_contexts:
2728
for linker_input in linking_context.linker_inputs.to_list():
2829
for library_to_link in linker_input.libraries:
29-
library_artifact = apple_common.compilation_support.get_static_library_for_linking(library_to_link)
30+
library_artifact = compilation_support.get_static_library_for_linking(library_to_link)
3031
if library_artifact:
3132
avoid_library_set[library_artifact.short_path] = True
3233

@@ -58,7 +59,7 @@ def subtract_linking_contexts(owner, linking_contexts, avoid_dep_linking_context
5859
linker_inputs_seen[linker_input] = True
5960

6061
for library_to_link in linker_input.libraries:
61-
library_artifact = apple_common.compilation_support.get_library_for_linking(library_to_link)
62+
library_artifact = compilation_support.get_library_for_linking(library_to_link)
6263

6364
if library_artifact.short_path not in avoid_library_set:
6465
libraries.append(library_to_link)

0 commit comments

Comments
 (0)