search

MobileNativeFoundation / rules_xcodeproj

Bazel rules for generating Xcode projects.
MIT License
502 stars 76 forks source link

Fix the issue with missing associated extra files #3010

Closed CognitiveDisson closed 2 months ago

CognitiveDisson commented 2 months ago

Fix for #3009. While this approach of converting a depset to a list and back to a depset may not be optimal, it seems the only way. Dictionary comprehension is also an option, although it is less readable. As far as I know, defaultdict is not available in Starlark.

I don't insist on merging this implementation if there's a better way. Feel free to close this PR.

brentleyjones commented 2 months ago

Thanks for the PR. I'm going to fix it in a slightly different way soon.