Module names are incorrect for indexing targets

[michaeleisel]

trafficstars

Indexing targets still need to have their normal module names or they won't build properly, but tulsi uses mangled module names with _idx at the start

[DavidGoldman]

Did you see any particular improvements? This came up before but we didn't notice any major differences when testing, perhaps we missed something. Would just need to do something similar to how we handle tests.

[michaeleisel]

yeah just setting the module names to be what they would be with bazel makes sense

[tinder-maxwellelliott]

@Keith Why was #23 closed? Is this no longer an issue?

[keith]

likely is but that PR was very stale was all

[DavidGoldman]

Looks like this is more relevant in Xcode 13 as the module name is used to resolve the auto-import functionality

[keith]

Thanks for submitting this issue but this tool is being deprecated. Please checkout rules_xcodeproj for a more complete and maintained Xcode integration.