Thoughts on allowing to configure platform?

[davidmcraejr]

We use different pods for different platforms and some don't compile for both iOS and Watch as an example, doing something like this would allow us to make different merges for each platform as needed.

[biocross]

Hello @dmac81, thank you for your PR.

Can you please tell me a little more about what use case does this MR solve? Unfortunately, the plugin itself does not take into account platforms other than iOS at this time, so I'm curious if the changes in the MR enough to solve your use case?

[davidmcraejr]

@biocross So we have two platforms in our Podfile (watch and iOS) when we include pods that will only work on iOS and not Watch then those will fail to build because the script here will pickup and include both iOS and Watch in the "merged" podfile. This allows us to override the default behavior of finding all platforms in the main podfile and putting them in the merged podfile and will utilize the platform set for each group. I haven't looked deep enough into the code to determine what else would need to be done to support other platforms, we are starting with our iOS ones first, but would think this change would allow for a simple use case at least.

[biocross]

@dmac81 got it. Can you just please share an example on how you'd override this in the MergeFile / Podfile? That way I can also update the README with this.

[davidmcraejr]

@biocross let me know if that works for you

[biocross]

@dmac81 Thanks a lot for this PR. I'll release a new version of the plugin with these changes soon.