Bug 217649 - Remove leftover MACOSX_DEPLOYMENT_TARGET_macosx support
Summary: Remove leftover MACOSX_DEPLOYMENT_TARGET_macosx support
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Keith Rollin
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-10-12 20:34 PDT by Keith Rollin
Modified: 2020-10-14 14:17 PDT (History)
18 users (show)

See Also:


Attachments
Patch (43.99 KB, patch)
2020-10-12 20:44 PDT, Keith Rollin
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Keith Rollin 2020-10-12 20:34:42 PDT
Bug 42796 introduced MACOSX_DEPLOYMENT_TARGET_<PLATFORM> as "support for compiling WebKit against iOS SDKs". Support for the iOS part of this feature was later removed in several changes, including Bug 139212, Bug 139463 and Bug 144762. However, vestiges have remained for five or six years in the form of MACOSX_DEPLOYMENT_TARGET_macosx. The inclusion of the platform in MACOSX_DEPLOYMENT_TARGET is no longer needed and can be removed.

This changes brings most projects in conformance with other projects that don't support including the platform in MACOSX_DEPLOYMENT_TARGET, including WebEditingTester, gtest, WebKitTestRunner, MiniBrowser, and TestWebKitAPI.

Along the way, remove a couple of left-over references to macOS 10.16, and a couple of places where [sdk=macosx*] was still being used.

With this change, initialization of MACOSX_DEPLOYMENT_TARGET should be consistent across all projects, with two exceptions: WebKitLauncher (which hardcodes it to 10.12) and libwebrtc's copy of googletest (which hardcodes it to 10.4). The reasons for these hard-coded values is not apparent, so leave them be.
Comment 1 Radar WebKit Bug Importer 2020-10-12 20:34:54 PDT
<rdar://problem/70236877>
Comment 2 Keith Rollin 2020-10-12 20:44:20 PDT
Created attachment 411193 [details]
Patch
Comment 3 EWS Watchlist 2020-10-12 20:45:20 PDT
Note that there are important steps to take when updating ANGLE. See https://trac.webkit.org/wiki/UpdatingANGLE
Comment 4 EWS 2020-10-13 14:25:52 PDT
Committed r268420: <https://trac.webkit.org/changeset/268420>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 411193 [details].
Comment 5 Alexey Proskuryakov 2020-10-14 14:17:03 PDT
Comment on attachment 411193 [details]
Patch

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

> Source/JavaScriptCore/ChangeLog:26
> +        consistent across all projects, with two exceptions: WebKitLauncher

I didn't realize that we still had it the tree!

Bug 217729 Remove WebKitLauncher.