[webkit-changes] [WebKit/WebKit] ac35e5: Fix the Mac JSCOnly build

Elliott Williams noreply at github.com
Tue Nov 29 14:51:30 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: ac35e5bfba5d7ef551a4e102c57bbbd2a7b5f370
      https://github.com/WebKit/WebKit/commit/ac35e5bfba5d7ef551a4e102c57bbbd2a7b5f370
  Author: Elliott Williams <emw at apple.com>
  Date:   2022-11-29 (Tue, 29 Nov 2022)

  Changed paths:
    M Source/JavaScriptCore/API/WebKitAvailability.h
    M Source/WTF/wtf/FileSystem.h
    M Source/WTF/wtf/PlatformHave.h
    M Source/WTF/wtf/UUID.cpp
    M Source/WTF/wtf/text/StringHasher.h
    M Source/cmake/OptionsCommon.cmake
    M Source/cmake/WebKitFeatures.cmake
    M Tools/TestWebKitAPI/Tests/JavaScriptCore/PropertySlot.cpp
    M Tools/TestWebKitAPI/config.h

  Log Message:
  -----------
  Fix the Mac JSCOnly build
https://bugs.webkit.org/show_bug.cgi?id=247927

Reviewed by Brent Fulgham.

* Source/JavaScriptCore/API/WebKitAvailability.h: Disable
  JSC availability macros when building JSCOnly.
* Source/WTF/wtf/FileSystem.h: Add missing include.
* Source/WTF/wtf/PlatformHave.h: Don't redeclare HAVE macros from
  cmakeconfig.h.
* Source/WTF/wtf/UUID.cpp: Add missing include.
* Source/WTF/wtf/text/StringHasher.h: Add missing include.
* Source/cmake/OptionsCommon.cmake: Feature-detect some linker options,
  instead of just assuming any LLD is GNU-compatible ld.
* Source/cmake/WebKitFeatures.cmake: Use check_symbol_exists instead of
  check_function_exists, because it works better with macOS SDK headers.
* Tools/TestWebKitAPI/Tests/JavaScriptCore/PropertySlot.cpp: Add missing
  include.
* Tools/TestWebKitAPI/config.h: JSCOnly's test target is called
  "TestJavaScriptCore"; recognize it as TestJSC.

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




More information about the webkit-changes mailing list