WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED INVALID
40237
[Qt] QtWebKit exports WebCore symbols
https://bugs.webkit.org/show_bug.cgi?id=40237
Summary
[Qt] QtWebKit exports WebCore symbols
Felix Geyer
Reported
2010-06-07 09:17:58 PDT
Created
attachment 58036
[details]
Patch to reduce visibility similar to the JavaScriptCore commit QtWebKit exports the WebCore symbols when building as a standalone package. Previously the JavaScriptCore symbols were exported too. This has been fixed by this commit:
http://trac.webkit.org/changeset/58100/trunk/JavaScriptCore/JavaScriptCore.pro
The same should be done in WebCore.
Attachments
Patch to reduce visibility similar to the JavaScriptCore commit
(648 bytes, patch)
2010-06-07 09:17 PDT
,
Felix Geyer
eric
: review-
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Jocelyn Turcotte
Comment 1
2010-06-10 03:32:50 PDT
Looks good to me. The CONFIG(standalone_package) condition should clearly be removed since we will start getting releases separated from Qt. If you want a reviewer to look at your patch you should set the review flag of the attachment to '?' (and the commit queue flag to '?' if you want it to be landed automatically)
Eric Seidel (no email)
Comment 2
2010-06-12 19:11:01 PDT
Comment on
attachment 58036
[details]
Patch to reduce visibility similar to the JavaScriptCore commit Please generate the patch from a WebKit checkout, and include a ChangeLog per our contribution guidelines:
http://webkit.org/coding/contributing.html
webkit-patch upload is one script which can help you make sure you're following the guildes.
Jocelyn Turcotte
Comment 3
2014-02-03 03:06:49 PST
=== Bulk closing of Qt bugs === If you believe that this bug report is still relevant for a non-Qt port of webkit.org, please re-open it and remove [Qt] from the summary. If you believe that this is still an important QtWebKit bug, please fill a new report at
https://bugreports.qt-project.org
and add a link to this issue. See
http://qt-project.org/wiki/ReportingBugsInQt
for additional guidelines.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug