76257 – [Win][JSC] Eliminate JavaScriptCore.def

RESOLVED DUPLICATE of bug 117050 76257

[Win][JSC] Eliminate JavaScriptCore.def

https://bugs.webkit.org/show_bug.cgi?id=76257

Summary [Win][JSC] Eliminate JavaScriptCore.def

Hajime Morrita

Reported 2012-01-13 02:01:55 PST

This is a twin of Bug 72854. We now have export macros in the tree. it should be possible to get rid of the .def file.

Attachments
Patch to check EWS (64.05 KB, patch) 2012-07-28 00:18 PDT, Patrick R. Gansterer	no flags	Details Formatted Diff Diff
Patch to check EWS (64.70 KB, patch) 2012-07-28 00:34 PDT, Patrick R. Gansterer	no flags	Details Formatted Diff Diff
Patch to check EWS (65.34 KB, patch) 2012-07-28 02:00 PDT, Patrick R. Gansterer	buildbot: commit-queue-	Details Formatted Diff Diff
Patch to check EWS (65.87 KB, patch) 2012-07-28 06:12 PDT, Patrick R. Gansterer	no flags	Details Formatted Diff Diff
Patch to check EWS (65.13 KB, patch) 2012-07-28 06:48 PDT, Patrick R. Gansterer	buildbot: commit-queue-	Details Formatted Diff Diff
Patch to check EWS (63.45 KB, patch) 2012-07-28 07:47 PDT, Patrick R. Gansterer	no flags	Details Formatted Diff Diff
Show Obsolete (5) View All Add attachment proposed patch, testcase, etc.

Adam Roben (:aroben)

Comment 1 2012-01-13 07:33:48 PST

I wonder if this will change the mangling of exported symbols and thus break nightly builds?

Hajime Morrita

Comment 2 2012-01-15 16:57:51 PST

Hi Adam, thanks for watching this ;-) > I wonder if this will change the mangling of exported symbols and thus break nightly builds? I tried it locally and found we need some changes to make it buildable. The trickiest part is that some WTF string related files are built into WebCore. We need some workaround for that. I'm investigating better way. Other fixes will be more trivial - just adding some preprocessor #define-s to compiler options.

Adam Roben (:aroben)

Comment 3 2012-01-17 08:32:18 PST

(In reply to comment #2) > Hi Adam, thanks for watching this ;-) > > > I wonder if this will change the mangling of exported symbols and thus break nightly builds? > I tried it locally and found we need some changes to make it buildable. OK. What I'm wondering though is: once this configuration is "buildable", will nightly builds still work? I.e., will the JavaScriptCore.dll that gets produced using the new process be usable by current versions of Safari?

Hajime Morrita

Comment 4 2012-01-17 17:27:29 PST

(In reply to comment #3) > OK. What I'm wondering though is: once this configuration is "buildable", will nightly builds still work? I.e., will the JavaScriptCore.dll that gets produced using the new process be usable by current versions of Safari? Good point. I need to check it locally as a part of this change. Assuming there is no windows specific JSC API, it should work.

Patrick R. Gansterer

Comment 5 2012-07-28 00:18:27 PDT

Created attachment 155116 [details] Patch to check EWS

Patrick R. Gansterer

Comment 6 2012-07-28 00:34:39 PDT

Created attachment 155117 [details] Patch to check EWS

WebKit Review Bot

Comment 7 2012-07-28 00:42:22 PDT

Attachment 155117 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1 Source/WTF/wtf/text/AtomicString.h:82: The parameter name "s" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 1 in 10 files If any of these errors are false positives, please file a bug against check-webkit-style.

Patrick R. Gansterer

Comment 8 2012-07-28 02:00:16 PDT

Created attachment 155122 [details] Patch to check EWS

WebKit Review Bot

Comment 9 2012-07-28 02:01:46 PDT

Attachment 155122 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1 Source/WTF/wtf/text/AtomicString.h:82: The parameter name "s" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 1 in 10 files If any of these errors are false positives, please file a bug against check-webkit-style.

Build Bot

Comment 10 2012-07-28 05:08:01 PDT

Comment on attachment 155122 [details] Patch to check EWS Attachment 155122 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13370925

Patrick R. Gansterer

Comment 11 2012-07-28 06:12:08 PDT

Created attachment 155126 [details] Patch to check EWS

WebKit Review Bot

Comment 12 2012-07-28 06:20:33 PDT

Attachment 155126 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1 Source/WTF/wtf/text/AtomicString.h:82: The parameter name "s" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 1 in 11 files If any of these errors are false positives, please file a bug against check-webkit-style.

Patrick R. Gansterer

Comment 13 2012-07-28 06:48:03 PDT

Created attachment 155129 [details] Patch to check EWS

WebKit Review Bot

Comment 14 2012-07-28 06:49:35 PDT

Attachment 155129 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1 Source/WTF/wtf/text/AtomicString.h:82: The parameter name "s" adds no information, so it should be removed. [readability/parameter_name] [5] Total errors found: 1 in 11 files If any of these errors are false positives, please file a bug against check-webkit-style.

Build Bot

Comment 15 2012-07-28 07:22:30 PDT

Comment on attachment 155129 [details] Patch to check EWS Attachment 155129 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13374885

Patrick R. Gansterer

Comment 16 2012-07-28 07:47:48 PDT

Created attachment 155130 [details] Patch to check EWS

Hajime Morrita

Comment 17 2012-07-29 19:51:16 PDT

This is great!! My picky feeling says WTF_EXPORT_PRIVATE_STRINGSTUFF is too long and strnage. Something like WTF_EXPORT_STRING_API will be better. But that's tiny point. This is a big deal. Thanks for taking this!

Patrick R. Gansterer

Comment 18 2013-10-31 01:35:30 PDT

*** This bug has been marked as a duplicate of bug 117050 ***

Note You need to log in before you can comment on or make changes to this bug.

Status RESOLVED

Resolution DUPLICATE

of bug 117050

Priority P2

Severity Normal

Classification Unclassified

Version 528+ (Nightly build)

Hardware Unspecified

OS Unspecified

Product WebKit

Component Platform

Assignee

Hajime Morrita

Reported

2012-01-13 02:01 PST

Modified

2013-10-31 01:35 PDT History

CC List

7 users Show

URL

Keywords

Depends on

92624 92789 92842 92851

Blocks

27551

Dependencies

tree graph