[webkit-changes] [WebKit/WebKit] 7c98dd: [Xcode] Dead webkitdirs.pm code assumes all signin...

Elliott Williams noreply at github.com
Fri Nov 3 05:58:42 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 7c98ddb0421147924e97ef80ad8cf703dd1f0d54
      https://github.com/WebKit/WebKit/commit/7c98ddb0421147924e97ef80ad8cf703dd1f0d54
  Author: Elliott Williams <emw at apple.com>
  Date:   2023-11-03 (Fri, 03 Nov 2023)

  Changed paths:
    M Source/JavaScriptCore/Configurations/DebugRelease.xcconfig
    M Source/ThirdParty/libwebrtc/Configurations/DebugRelease.xcconfig
    M Source/WebKit/Configurations/DebugRelease.xcconfig
    M Tools/DumpRenderTree/mac/Configurations/DebugRelease.xcconfig
    M Tools/MobileMiniBrowser/Configurations/DebugRelease.xcconfig
    M Tools/Scripts/webkitdirs.pm
    M Tools/TestWebKitAPI/Configurations/DebugRelease.xcconfig
    M Tools/WebKitTestRunner/Configurations/DebugRelease.xcconfig

  Log Message:
  -----------
  [Xcode] Dead webkitdirs.pm code assumes all signing identities start with "iPhone Developer"
https://bugs.webkit.org/show_bug.cgi?id=263938
rdar://96538743

Reviewed by Alexey Proskuryakov.

They have been called "Apple Developer" for years now, so the code path
that disables code signing is always taken. Since it's not possible to
run open-source builds of WebKit on embedded devices, this has never
been noticed, and it can be cleaned up.

Move the code signing overrides into xcconfig variables that are only
active in open source builds. This has the nice side effect of making it
possible to build for embedded devices in Xcode, without using
build-webkit. Continue to use ad-hoc signing internally, where it is
allowed.

Works towards making webkitdirs aware of the active SDK in internal
builds (rdar://86340301).

* Source/JavaScriptCore/Configurations/DebugRelease.xcconfig:
* Source/ThirdParty/libwebrtc/Configurations/DebugRelease.xcconfig:
* Source/WebKit/Configurations/DebugRelease.xcconfig:
* Tools/DumpRenderTree/mac/Configurations/DebugRelease.xcconfig:
* Tools/MobileMiniBrowser/Configurations/DebugRelease.xcconfig:
* Tools/Scripts/webkitdirs.pm:
(XcodeOptions):
(hasIOSDevelopmentCertificate): Deleted.
* Tools/TestWebKitAPI/Configurations/DebugRelease.xcconfig:
* Tools/WebKitTestRunner/Configurations/DebugRelease.xcconfig:

Canonical link: https://commits.webkit.org/270170@main




More information about the webkit-changes mailing list