Bug 81999 - Tools/Scripts/build-jsc doesn't work after WTF move
Summary: Tools/Scripts/build-jsc doesn't work after WTF move
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 81844
Blocks:
  Show dependency tree / graph
 
Reported: 2012-03-22 17:30 PDT by Michael Saboff
Modified: 2012-03-22 17:55 PDT (History)
6 users (show)

See Also:

Attachments
the patch (3.88 KB, patch)
2012-03-22 17:43 PDT, Filip Pizlo 		eric: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Saboff 2012-03-22 17:30:38 PDT 
build-jsc doesn't appear to build WTF after the WTF move and fails with linker errors.

If WTF is built with build-webkit, then build-jsc appears to work.
Comment 1 Eric Seidel (no email) 2012-03-22 17:37:58 PDT 
It's slightly confusing becasue some of the ports build each directory independently, and some don't.  It looks liek Qt is already setup to build WTF before buidling JSC in this script.  I recommend we just hack AppleMac/AppleWin to do similar to Qt for now.
Comment 2 Filip Pizlo 2012-03-22 17:39:55 PDT 
I believe I have a fix for this.
Comment 3 Filip Pizlo 2012-03-22 17:43:18 PDT 
Created attachment 133400 [details]
the patch
Comment 4 Eric Seidel (no email) 2012-03-22 17:45:21 PDT 
Comment on attachment 133400 [details]
the patch

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

This looks reasonable.

> Tools/Scripts/build-jsc:92
> +    chdir "../..";

chdrWebKit?
Comment 5 Filip Pizlo 2012-03-22 17:46:42 PDT 
(In reply to comment #4)
> (From update of attachment 133400 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=133400&action=review
> 
> This looks reasonable.
> 
> > Tools/Scripts/build-jsc:92
> > +    chdir "../..";
> 
> chdrWebKit?

Aha!  Didn't know about that, and it indeed works.  Will land with this change.
Comment 6 Andy Estes 2012-03-22 17:46:51 PDT 
Comment on attachment 133400 [details]
the patch

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

> Source/JavaScriptCore/bytecode/SamplingTool.h:127
> -        static void dump();
> +        JS_EXPORT_PRIVATE static void dump();

Did you mean to include this? It doesn't look relevant.
Comment 7 Filip Pizlo 2012-03-22 17:47:41 PDT 
(In reply to comment #6)
> (From update of attachment 133400 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=133400&action=review
> 
> > Source/JavaScriptCore/bytecode/SamplingTool.h:127
> > -        static void dump();
> > +        JS_EXPORT_PRIVATE static void dump();
> 
> Did you mean to include this? It doesn't look relevant.

Good catch, and no, it is not relevant.  Will revert.
Comment 8 Filip Pizlo 2012-03-22 17:51:38 PDT 
Landed in http://trac.webkit.org/changeset/111797