Skip to content

Commit

Permalink
Remove id from direct_outputs (#3068)
Browse files Browse the repository at this point in the history
It’s not needed.

Signed-off-by: Brentley Jones <[email protected]>
  • Loading branch information
brentleyjones committed Aug 7, 2024
1 parent 8ba39d7 commit b3fe0fe
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions xcodeproj/internal/files/incremental_output_files.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ _COPYABLE_PRODUCT_TYPES = {

# Utility

def _get_outputs(*, debug_outputs, id, product, swift_info, output_group_info):
def _get_outputs(*, debug_outputs, product, swift_info, output_group_info):
"""Collects the output files for a given target.
The outputs are bucketed into two categories: build and index. The build
Expand All @@ -53,7 +53,6 @@ def _get_outputs(*, debug_outputs, id, product, swift_info, output_group_info):
Args:
debug_outputs: The `AppleDebugOutputs` provider for the target, or
`None`.
id: The unique identifier of the target.
output_group_info: The `OutputGroupInfo` provider for the target, or
`None`.
product: A value from `process_product`, or `None` if the
Expand Down Expand Up @@ -101,7 +100,6 @@ def _get_outputs(*, debug_outputs, id, product, swift_info, output_group_info):
product_path = None

return struct(
id = id,
is_framework = is_framework,
product = product_file,
product_path = product_path,
Expand Down Expand Up @@ -171,7 +169,6 @@ def _collect_incremental_output_files(
"""
direct_outputs = _get_outputs(
debug_outputs = debug_outputs,
id = id,
output_group_info = output_group_info,
product = product,
swift_info = swift_info,
Expand Down Expand Up @@ -244,7 +241,7 @@ def _collect_incremental_output_files(
],
)

products_output_group_name = "bp {}".format(direct_outputs.id)
products_output_group_name = "bp {}".format(id)

indexstores_filelist = indexstore_filelists.write(
actions = actions,
Expand All @@ -267,8 +264,8 @@ def _collect_incremental_output_files(
)

direct_group_list = [
("bc {}".format(direct_outputs.id), transitive_compile_params),
("bl {}".format(direct_outputs.id), transitive_link_params),
("bc {}".format(id), transitive_compile_params),
("bl {}".format(id), transitive_link_params),
(products_output_group_name, products_depset),
]

Expand Down

0 comments on commit b3fe0fe

Please sign in to comment.