Skip to content

Conversation

maxwellE
Copy link
Contributor

Adds logic to support the codesignopts property on rules
as part of Support macos_command_line_application #99

@maxwellE maxwellE force-pushed the me/support_codesignopts branch from 6a5a65e to a08e52d Compare June 27, 2022 17:52
@maxwellE maxwellE requested a review from brentleyjones June 27, 2022 17:55
@maxwellE maxwellE force-pushed the me/support_codesignopts branch from 306d6e7 to 235bdf2 Compare June 28, 2022 01:38
Comment on lines 54 to 59
bundle_id = None
codesignopts = None
entitlements = None
hdrs = ()
infoplists = ()
non_arc_srcs = ()
pch = None
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorted these

@@ -88,6 +89,10 @@ def _default_automatic_target_processing_aspect_impl(target, ctx):
infoplists = ("infoplists")
should_generate_target = False
elif AppleBundleInfo in target:
codesignopts = "codesignopts"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorted these values in both locations

@maxwellE maxwellE requested a review from brentleyjones June 28, 2022 01:40
@maxwellE maxwellE requested a review from brentleyjones June 28, 2022 01:58
maxwellE and others added 8 commits June 28, 2022 10:03
Adds logic to support the `codesignopts` property on rules
as part of Support `macos_command_line_application` MobileNativeFoundation#99
Use `XcodeProjAutomaticTargetProcessingInfo` to fetch codesign flags
@maxwellE maxwellE force-pushed the me/support_codesignopts branch from ac68499 to 1b53f8e Compare June 28, 2022 14:07
@brentleyjones brentleyjones merged commit d86b6f6 into MobileNativeFoundation:main Jun 28, 2022
@maxwellE maxwellE deleted the me/support_codesignopts branch June 28, 2022 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants