Feature/update podfile

[sunmou99]

Update integration testapps' Podfiles to match the SDK's Podfile.

Script reference: https://github.com/firebase/firebase-ios-sdk/blob/master/scripts/localize_podfile.swift

[anonymous-akorn]

I'm curious about the choice to use swift for this script. It looks like the logic in the script boils down to this:

• Read a file.
• Look for lines of the form pod '<api>', '<version>' and store a mapping of api -> version.
• Read another file.
• Look for lines of the same form, and replace their version with the version in the mapping.

Given that we're defaulting to Python for our scripts, would it make sense to write this logic in Python instead? Or is there a benefit to leaving it in swift?

[sunmou99]

Given that we're defaulting to Python for our scripts, would it make sense to write this logic in Python instead? Or is there a benefit to leaving it in swift?

I was thinking about Python in the first place, but @granluo told me that iCore team uses swift script to update podfile.
So I use swift for the sake of consistency. And Gran probably gonna have a similar swift script in his 'cpp ios sdk unrelease testing project'.

[anonymous-akorn]

I was thinking about Python in the first place, but @granluo told me that iCore team uses swift script to update podfile.
So I use swift for the sake of consistency. And Gran probably gonna have a similar swift script in his 'cpp ios sdk unrelease testing project'.

The Games team will be the ones maintaining this, though - leaving it in swift means another language they need to use. Though the script is unlikely to change in the future, so it's probably not a big deal. I'd suggest getting someone on the Games team to review the PR, since their approval is more relevant than mine for this.

Beyond that, LGTM.

[granluo]

I think using Python could be an option for the script. The iOS team use Swift because Swift is their preferred language for command line tools. I would recommend Python since 1) Python is the major language for infra tools in Games team, and 2) it would be easier if other Python scripts want to reuse this tool. Unreleased testing could apply this tool and reviews for updates of this tool might be required.

[sunmou99]

@granluo Updated to python script.

[DellaBitta]

LGTM