search

xamarin / xamarin-macios

.NET for iOS, Mac Catalyst, macOS, and tvOS provide open-source bindings of the Apple SDKs for use with .NET managed languages such as C#
Other
2.49k stars 515 forks source link

Bump min iOS version to 7.0 #6213

Closed rolfbjarne closed 5 years ago

rolfbjarne commented 5 years ago

It seems Xcode 11 drops support for iOS versions < 8.0

So we'll have to do the same.

spouliot commented 5 years ago

Release notes are not matching the behaviour you described (I have more trust in the later)

Xcode 11 supports on-device debugging for iOS 8 and later, tvOS 9 and later, and watchOS 2 and later. https://developer.apple.com/documentation/xcode_release_notes/xcode_11_beta_release_notes?language=objc

Simulator support is, for beta1, much more restrictive. In fact the above looks more like what Xcode 10.x had.

spouliot commented 5 years ago

for reference

/Users/poupou/git/d16-1/xamarin-macios/tests/monotouch-test/ld: Warning: OS version (6.0.0) too small, changing to 8.0.0 (monotouch-test)
spouliot commented 5 years ago

Test results from https://github.com/xamarin/xamarin-macios/issues/6257 suggest that 8.0.0 is set as minimum for simulator but 7.0.0 is used for device builds (so what can be submitted) The later is likely what we should target as an absolute minimum

rolfbjarne commented 5 years ago

Yes, it seems 7.0.0 is the minimum for device builds, and 8.0.0 for simulator builds:

$ /Applications/Xcode11-beta7.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang test.m -arch arm64 -isysroot /Applications/Xcode11-beta7.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS.sdk -miphoneos-version-min=6.0 -framework Foundation
ld: warning: OS version (6.0.0) too small, changing to 7.0.0

$ /Applications/Xcode11-beta7.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang test.m -arch x86_64 -isysroot /Applications/Xcode11-beta7.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator.sdk -miphoneos-version-min=6.0 -framework Foundation
ld: warning: OS version (6.0.0) too small, changing to 8.0.0
rolfbjarne commented 5 years ago

Fixed in https://github.com/xamarin/xamarin-macios/commit/344dadb212dc34db398f765af1d0ec79876b6323.