Bug 178424 - REGRESSION (r223476): WebCore exports symbols with names belonging to other frameworks
Summary: REGRESSION (r223476): WebCore exports symbols with names belonging to other f...
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Keith Miller
Keywords: InRadar
: 178495 (view as bug list)
Depends on:
Reported: 2017-10-17 20:19 PDT by mitz
Modified: 2017-10-20 06:35 PDT (History)
10 users (show)

See Also:

Patch (1.99 KB, patch)
2017-10-18 19:44 PDT, Keith Miller
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2017-10-17 20:19:22 PDT
<https://trac.webkit.org/r223476>, the fix for bug 178229, made WebCore export symbols identical to the ones it’s “soft linking” using the SOFT_LINK_FUNCTION_FOR_SOURCE macro. For example, now WebCore exports a function called CMClockGetTime, which is different from the CMClockGetTime that the Core Media framework exports. WebCore’s exports should all be in WebCore’s C++ namespaces or have the Web or _Web prefix.
Comment 1 mitz 2017-10-17 20:27:00 PDT
Comment 2 Keith Miller 2017-10-18 19:43:07 PDT
*** Bug 178495 has been marked as a duplicate of this bug. ***
Comment 3 Keith Miller 2017-10-18 19:44:28 PDT
Created attachment 324202 [details]
Comment 4 Keith Miller 2017-10-18 19:47:42 PDT
From nm WebCore.build/Objects-normal/x86_64/XSLTUnicodeSort.o (which calls a function, xsltEvalAttrValueTemplate, from libxslt):

(__TEXT,__text) weak private external _xsltEvalAttrValueTemplate
Comment 5 David Kilzer (:ddkilzer) 2017-10-19 09:52:22 PDT
Comment on attachment 324202 [details]

r=me.  We should stop soft-linking libxslt as well, but that's a separate patch.
Comment 6 WebKit Commit Bot 2017-10-19 11:18:35 PDT
Comment on attachment 324202 [details]

Clearing flags on attachment: 324202

Committed r223693: <https://trac.webkit.org/changeset/223693>
Comment 7 WebKit Commit Bot 2017-10-19 11:18:37 PDT
All reviewed patches have been landed.  Closing bug.