Bug 180691 – Symbol not found: __ZN3WTF8LockBase10unlockSlowEv

• Component: New Bugs
• Status: RESOLVED FIXED

[Devin Rousso]

I just tried running my local build via `Tools/Scripts/run-safari --release` and got this error:

Starting SafariForWebKitDevelopment with DYLD_FRAMEWORK_PATH set to point to built WebKit in WebKitBuild/Release.
dyld: Symbol not found: __ZN3WTF8LockBase10unlockSlowEv
  Referenced from: /System/Library/PrivateFrameworks/Safari.framework/Versions/A/Safari
  Expected in: WebKitBuild/Release/JavaScriptCore.framework/Versions/A/JavaScriptCore
 in /System/Library/PrivateFrameworks/Safari.framework/Versions/A/Safari

I'm guessing that this is related to <https://webkit.org/b/180495>, as it removed LockBase.

* Info:
ProductName:	Mac OS X
ProductVersion:	10.13.2
BuildVersion:	17C88

[Joseph Pecoraro]

Looks like system macOS 10.13.2 Safari needs this symbol:

    $ c++filt __ZN3WTF8LockBase10unlockSlowEv
    WTF::LockBase::unlockSlow()

Typically we add back something like this into:
WTF/wtf/mac/DeprecatedSymbolsUsedBySafari.mm

I don't know the full list of symbols we need to add back. Probably a few of these, which just moved from WTF::LockBase to WTF::Lock.

    WTF_EXPORT_PRIVATE void lockSlow();
    WTF_EXPORT_PRIVATE void unlockSlow();
    WTF_EXPORT_PRIVATE void unlockFairlySlow();
    WTF_EXPORT_PRIVATE void safepointSlow();

[Joseph Pecoraro]

Created attachment 329096 [details]
[PATCH] Proposed Fix

[Joseph Pecoraro]

Comment on attachment 329096 [details]
[PATCH] Proposed Fix

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

> Source/WTF/wtf/mac/DeprecatedSymbolsUsedBySafari.mm:132
> +#if PLATFORM(MAC)

Not sure if it is worth adding a restriction here, such as:

    && __MAC_OS_X_VERSION_MIN_REQUIRED < 101340

But reviewers let me know if you think that would be worth doing.

[Yusuke Suzuki]

Comment on attachment 329096 [details]
[PATCH] Proposed Fix

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

>> Source/WTF/wtf/mac/DeprecatedSymbolsUsedBySafari.mm:132
>> +#if PLATFORM(MAC)
> 
> Not sure if it is worth adding a restriction here, such as:
> 
>     && __MAC_OS_X_VERSION_MIN_REQUIRED < 101340
> 
> But reviewers let me know if you think that would be worth doing.

I think this restriction is useful since it can be a marker for dropping this in the future.
Adding `FIXME` comment is also nice.

[Joseph Pecoraro]

<https://trac.webkit.org/r225792>

[Radar WebKit Bug Importer]

<rdar://problem/36001432>