Skip to content

Commit e0502c4

Browse files
nglevinluispadron
authored andcommitted
Remove codesigning_dossier_partial from Apple framework rules.
The dossier signing script signs embedded frameworks and dylibs in the "Frameworks" folder when found. Frameworks themselves do not define entitlements and the optional "provisioning_profile" attribute was only done way back when as a means of "pre-signing" frameworks for app bundles without dossiers. Cherry-pick: d8ecc9b
1 parent 43b5223 commit e0502c4

3 files changed

Lines changed: 0 additions & 90 deletions

File tree

apple/internal/ios_rules.bzl

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -1007,21 +1007,6 @@ def _ios_framework_impl(ctx):
10071007
executable_name = executable_name,
10081008
label_name = label.name,
10091009
),
1010-
partials.codesigning_dossier_partial(
1011-
actions = actions,
1012-
apple_mac_toolchain_info = apple_mac_toolchain_info,
1013-
apple_xplat_toolchain_info = apple_xplat_toolchain_info,
1014-
bundle_extension = bundle_extension,
1015-
bundle_location = processor.location.framework,
1016-
bundle_name = bundle_name,
1017-
embed_target_dossiers = False,
1018-
embedded_targets = ctx.attr.frameworks,
1019-
label_name = label.name,
1020-
platform_prerequisites = platform_prerequisites,
1021-
predeclared_outputs = predeclared_outputs,
1022-
provisioning_profile = provisioning_profile,
1023-
rule_descriptor = rule_descriptor,
1024-
),
10251010
# TODO(kaipi): Check if clang_rt dylibs are needed in Frameworks, or if
10261011
# the can be skipped.
10271012
partials.clang_rt_dylibs_partial(
@@ -1601,21 +1586,6 @@ def _ios_dynamic_framework_impl(ctx):
16011586
executable_name = executable_name,
16021587
label_name = label.name,
16031588
),
1604-
partials.codesigning_dossier_partial(
1605-
actions = actions,
1606-
apple_mac_toolchain_info = apple_mac_toolchain_info,
1607-
apple_xplat_toolchain_info = apple_xplat_toolchain_info,
1608-
bundle_extension = bundle_extension,
1609-
bundle_location = processor.location.framework,
1610-
bundle_name = bundle_name,
1611-
embed_target_dossiers = False,
1612-
embedded_targets = ctx.attr.frameworks,
1613-
label_name = label.name,
1614-
platform_prerequisites = platform_prerequisites,
1615-
predeclared_outputs = predeclared_outputs,
1616-
provisioning_profile = provisioning_profile,
1617-
rule_descriptor = rule_descriptor,
1618-
),
16191589
partials.clang_rt_dylibs_partial(
16201590
actions = actions,
16211591
apple_mac_toolchain_info = apple_mac_toolchain_info,

apple/internal/tvos_rules.bzl

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -614,21 +614,6 @@ def _tvos_dynamic_framework_impl(ctx):
614614
executable_name = executable_name,
615615
label_name = label.name,
616616
),
617-
partials.codesigning_dossier_partial(
618-
actions = actions,
619-
apple_mac_toolchain_info = apple_mac_toolchain_info,
620-
apple_xplat_toolchain_info = apple_xplat_toolchain_info,
621-
bundle_extension = bundle_extension,
622-
bundle_location = processor.location.framework,
623-
bundle_name = bundle_name,
624-
embed_target_dossiers = False,
625-
embedded_targets = ctx.attr.frameworks,
626-
label_name = label.name,
627-
platform_prerequisites = platform_prerequisites,
628-
predeclared_outputs = predeclared_outputs,
629-
provisioning_profile = provisioning_profile,
630-
rule_descriptor = rule_descriptor,
631-
),
632617
partials.clang_rt_dylibs_partial(
633618
actions = actions,
634619
apple_mac_toolchain_info = apple_mac_toolchain_info,
@@ -917,21 +902,6 @@ def _tvos_framework_impl(ctx):
917902
platform_prerequisites = platform_prerequisites,
918903
dylibs = main_thread_checker_dylibs.get_from_toolchain(ctx),
919904
),
920-
partials.codesigning_dossier_partial(
921-
actions = actions,
922-
apple_mac_toolchain_info = apple_mac_toolchain_info,
923-
apple_xplat_toolchain_info = apple_xplat_toolchain_info,
924-
bundle_extension = bundle_extension,
925-
bundle_location = processor.location.framework,
926-
bundle_name = bundle_name,
927-
embed_target_dossiers = False,
928-
embedded_targets = ctx.attr.frameworks,
929-
label_name = label.name,
930-
platform_prerequisites = platform_prerequisites,
931-
predeclared_outputs = predeclared_outputs,
932-
provisioning_profile = provisioning_profile,
933-
rule_descriptor = rule_descriptor,
934-
),
935905
partials.debug_symbols_partial(
936906
actions = actions,
937907
bundle_extension = bundle_extension,

apple/internal/watchos_rules.bzl

Lines changed: 0 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -275,21 +275,6 @@ def _watchos_framework_impl(ctx):
275275
executable_name = executable_name,
276276
label_name = label.name,
277277
),
278-
partials.codesigning_dossier_partial(
279-
actions = actions,
280-
apple_mac_toolchain_info = apple_mac_toolchain_info,
281-
apple_xplat_toolchain_info = apple_xplat_toolchain_info,
282-
bundle_extension = bundle_extension,
283-
bundle_location = processor.location.framework,
284-
bundle_name = bundle_name,
285-
embed_target_dossiers = False,
286-
embedded_targets = ctx.attr.frameworks,
287-
label_name = label.name,
288-
platform_prerequisites = platform_prerequisites,
289-
predeclared_outputs = predeclared_outputs,
290-
provisioning_profile = provisioning_profile,
291-
rule_descriptor = rule_descriptor,
292-
),
293278
partials.clang_rt_dylibs_partial(
294279
actions = actions,
295280
apple_mac_toolchain_info = apple_mac_toolchain_info,
@@ -562,21 +547,6 @@ def _watchos_dynamic_framework_impl(ctx):
562547
executable_name = executable_name,
563548
label_name = label.name,
564549
),
565-
partials.codesigning_dossier_partial(
566-
actions = actions,
567-
apple_mac_toolchain_info = apple_mac_toolchain_info,
568-
apple_xplat_toolchain_info = apple_xplat_toolchain_info,
569-
bundle_extension = bundle_extension,
570-
bundle_location = processor.location.framework,
571-
bundle_name = bundle_name,
572-
embed_target_dossiers = False,
573-
embedded_targets = ctx.attr.frameworks,
574-
label_name = label.name,
575-
platform_prerequisites = platform_prerequisites,
576-
predeclared_outputs = predeclared_outputs,
577-
provisioning_profile = provisioning_profile,
578-
rule_descriptor = rule_descriptor,
579-
),
580550
partials.clang_rt_dylibs_partial(
581551
actions = actions,
582552
apple_mac_toolchain_info = apple_mac_toolchain_info,

0 commit comments

Comments
 (0)