Bug 184225 – [Xcode] Adopt WK_PLATFORM_NAME for SUPPORTS_TEXT_BASED_API

• Component: WebKit Misc.
• Status: RESOLVED CONFIGURATION CHANGED

<rdar://problem/39079659>

Created attachment 336956 [details]
Base SUPPORTS_TEXT_BASED_API on WK_PLATFORM_NAME

[Alexey Proskuryakov]

Comment on attachment 336956 [details]
Base SUPPORTS_TEXT_BASED_API on WK_PLATFORM_NAME

View in context: https://bugs.webkit.org/attachment.cgi?id=336956&action=review

> Source/WebKitLegacy/mac/ChangeLog:3
> +        [Xcode] Adopt WK_PLATFORM_NAME for SUPPORTS_TEXT_BASED_API

I’m very much not sure if we want to do this. Will start an e-mail thread to discuss.

> Source/WebKitLegacy/mac/Configurations/WebKitLegacy.xcconfig:137
> +SUPPORTS_TEXT_BASED_API_iphoneos = $(USE_INTERNAL_SDK);

This aspect of existing behavior is not right, we should have it enabled in open source builds too, to have EWS catch mistakes.

(In reply to Alexey Proskuryakov from comment #2)
> Comment on attachment 336956 [details]
> Base SUPPORTS_TEXT_BASED_API on WK_PLATFORM_NAME
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=336956&action=review
> 
> > Source/WebKitLegacy/mac/ChangeLog:3
> > +        [Xcode] Adopt WK_PLATFORM_NAME for SUPPORTS_TEXT_BASED_API
> 
> I’m very much not sure if we want to do this. Will start an e-mail thread to
> discuss.
> 
> > Source/WebKitLegacy/mac/Configurations/WebKitLegacy.xcconfig:137
> > +SUPPORTS_TEXT_BASED_API_iphoneos = $(USE_INTERNAL_SDK);
> 
> This aspect of existing behavior is not right, we should have it enabled in
> open source builds too, to have EWS catch mistakes.

I agree. The Apple bug referenced in relation to this is <rdar://problem/24582471>, which essentially suggests that we should do this when all versions of Xcode that are supported for the Open Source project include installapi. Do you know if this is now the case? If not, I can also enable it for a subset of Xcode versions, if I get the list of Xcode versions that are supported for Open Source development yet lack installapi.

[Alexey Proskuryakov]

InstallAPI should be already supported in all Xcode versions we use with iOS. I did not verify that.

Created attachment 336962 [details]
For EWS

Created attachment 336963 [details]
For EWS

Created attachment 336964 [details]
For EWS

Created attachment 336965 [details]
For EWS

Created attachment 336967 [details]
For EWS

(In reply to mitz from comment #9)
> Created attachment 336967 [details]

This demonstrates that installapi can be enabled when building with the public SDK with only minor changes. However, doing so is separate from what this bug is about, and probably warrants a separate bug with a separate patch.

[Alexey Proskuryakov]

This patch is obsolete, we already enable InstallAPI pretty unconditionally. I'll post a separate PR to remove dead code for conditions in a few moments.