Remove platforms declaration to avoid deployment target warnings

[CemYil03]

This is required to fix the following warning in Xcode 13.1: "The iOS Simulator deployment target 'IPHONEOS_DEPLOYMENT_TARGET' is set to 8.0, but the range of supported deployment target versions is 9.0 to 15.0.99."

[Oliver-Binns]

@CemYil03 since the deployment target specified is actually the base required for Swift Package Manager, I'd suggest removing the platforms declaration entirely. This will remove the build warning that appears in lower versions of Xcode too.

I recommend taking a look at these two links if you want further information: https://forums.swift.org/t/xcode12-minimum-deployment-target-and-spm/40467 https://github.com/nanopb/nanopb/pull/585/files

[simonseyer]

I would second @Oliver-Binns comment. Would be great if you could apply the change by removing the platforms definition altogether.

[CemYil03]

First I thought this could result in compile errors, but it seems to work. The change is now in the latest commit

[CemYil03]

Would really like to merge it in asap. Any idea who to contact?

[Oliver-Binns]

@WilliamDenniss looks like you've merged a few recently? 🙂 please can you take a look here?

[CemYil03]

Someone like @WilliamDenniss or @julienbodet do you have the rights to perform a merge?

[khalideali]

@marcoceppi @iainmcgin @panva Sorry to bother you, but can anyone from openid review and merge this pull request?

[marcoceppi]

I unfortunately can only review the PHP OpenID project. Asking here may help: http://lists.openid.net/mailman/listinfo/openid-code

[julien-baillon]

Please, could anyone merge this MR ? 🙏🏻

[marcoceppi]

@petea Is this something you can help review and merge?

[petea]

This has been addressed with #690

[nh7a]

@petea did you see Oliver-Binns's https://github.com/openid/AppAuth-iOS/pull/661#issuecomment-977207559? Surely, #690 is better than nothing, but it'll only work until it'll break soon.

[petea]

@nh7a I did, and will consider taking that step next.