On Thu, 20 Oct 2016, Daniel Stenberg wrote:

> On Thu, 20 Oct 2016, Martin Storsjö wrote:
>
>> So to expand the fix to "just work" in most cases, perhaps we should add
>> -miphoneos-version-min=6.0, which should be ancient enough for any current
>> usage.
>
> Sound logic, but it seems we need to pick one of them because if we use both
> I get:
>
> clang: error: invalid argument '-mmacosx-version-min=10.5' not allowed with
> '-miphoneos-version-min=6.0'

Hmm, ok. I guess it's better to leave it as is then. If the
-mmacosx-version-min parameter is set while building with an iOS SDK, you
get a warning like this:

clang: warning: using sysroot for 'iPhoneOS' but targeting 'MacOSX'

So that will at least give another clue to iOS builders that they might
need to set something if they didn't already.

What might need to be adjusted though, is the fact that one also can use
-mios-version-min instead of -miphoneos-version-min. (And for the other
newlings, there's also -mtvos-version-min and -mwatchos-version-min.)

What about the attached patch for that?

// Martin

-------------------------------------------------------------------
List admin: https://cool.haxx.se/list/listinfo/curl-library
Etiquette: https://curl.haxx.se/mail/etiquette.html