Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calculate ExtensionPointIdentifier without compiling XCAssets #797

Merged

Conversation

brentleyjones
Copy link
Contributor

Using the Bazel generated Info.plist to calculate ExtensionPointIdentifier adds an implicit dependency on AssetCatalogCompile. We only need to "merge" the hand provided Info.plists, which we do by processing multiple Info.plists for a target, in order.

@brentleyjones brentleyjones force-pushed the bj/more-efficient-parse_extensionpointidentifiers branch from 1c36cc7 to 1e1fd0e Compare July 21, 2022 21:02
@brentleyjones brentleyjones force-pushed the bj/more-efficient-parse_extensionpointidentifiers branch from 1e1fd0e to 11b0c14 Compare July 21, 2022 21:24
@brentleyjones brentleyjones force-pushed the bj/more-efficient-parse_extensionpointidentifiers branch from 11b0c14 to eac75a2 Compare July 21, 2022 21:46
@brentleyjones brentleyjones force-pushed the bj/more-efficient-parse_extensionpointidentifiers branch from eac75a2 to 78360b4 Compare July 21, 2022 21:47
Base automatically changed from bj/selective-generated-files to main July 21, 2022 21:53
Using the Bazel generated Info.plist to calculate `ExtensionPointIdentifier` adds an implicit dependency on `AssetCatalogCompile`. We only need to "merge" the hand provided Info.plists, which we do by processing multiple Info.plists for a target, in order.
@brentleyjones brentleyjones force-pushed the bj/more-efficient-parse_extensionpointidentifiers branch from 78360b4 to 5a0f598 Compare July 21, 2022 21:55
@brentleyjones brentleyjones marked this pull request as ready for review July 21, 2022 21:55
@brentleyjones brentleyjones enabled auto-merge (squash) July 21, 2022 21:55
@brentleyjones brentleyjones merged commit c33fac6 into main Jul 21, 2022
@brentleyjones brentleyjones deleted the bj/more-efficient-parse_extensionpointidentifiers branch July 21, 2022 21:57
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.

None yet

2 participants