Bug 79783 – [Qt] Buildfix after moving WTF out of JavaScriptCore

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: Tools / Tests

[Csaba Osztrogonác]

Patch is coming soon.

[Csaba Osztrogonác]

Created attachment 129244 [details]
proposed fix

It should be landed after WTF moved out.

[Csaba Osztrogonác]

Comment on attachment 129244 [details]
proposed fix

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

> Tools/DumpRenderTree/qt/DumpRenderTree.pro:22
> +    $${ROOT_WEBKIT_DIR}/Source/JavaScriptCore \
> +    $${ROOT_WEBKIT_DIR}/Source/JavaScriptCore/runtime

It is needed to make DRT build, because
- Tools/DumpRenderTree/config.h includes <runtime/JSExportMacros.h>
- Source/WTF/config.h includes "JSExportMacros.h"

I know it isn't the nicest fix, but it works. :)

[Early Warning System Bot]

Comment on attachment 129244 [details]
proposed fix

Attachment 129244 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/11650160

[Csaba Osztrogonác]

(In reply to comment #3)
> (From update of attachment 129244 [details])
> Attachment 129244 [details] did not pass qt-ews (qt):
> Output: http://queues.webkit.org/results/11650160

Of course it doesn't build now, but it will after moving out WTF. ;)

[Tor Arne Vestbø]

Comment on attachment 129244 [details]
proposed fix

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

r=me, but try the WEBKIT trick first

>> Tools/DumpRenderTree/qt/DumpRenderTree.pro:22
>> +    $${ROOT_WEBKIT_DIR}/Source/JavaScriptCore/runtime
> 
> It is needed to make DRT build, because
> - Tools/DumpRenderTree/config.h includes <runtime/JSExportMacros.h>
> - Source/WTF/config.h includes "JSExportMacros.h"
> 
> I know it isn't the nicest fix, but it works. :)

You should be able to replace this with modifying WEBKIT above:

WEBKIT += wtf javascriptcore webcore

[Csaba Osztrogonác]

(In reply to comment #5)
> You should be able to replace this with modifying WEBKIT above:
> 
> WEBKIT += wtf javascriptcore webcore

We have javascriptcore.prf, qt_webkit.prf, webcore.prf, webkit2.prf and
wtf.prf named files now. I think we should change them later, separately
from this bug if we really want to do it.

[Tor Arne Vestbø]

(In reply to comment #6)
> (In reply to comment #5)
> > You should be able to replace this with modifying WEBKIT above:
> > 
> > WEBKIT += wtf javascriptcore webcore
> 
> We have javascriptcore.prf, qt_webkit.prf, webcore.prf, webkit2.prf and
> wtf.prf named files now. I think we should change them later, separately
> from this bug if we really want to do it.

What do you mean?

[Csaba Osztrogonác]

(In reply to comment #7)
> (In reply to comment #6)
> > (In reply to comment #5)
> > > You should be able to replace this with modifying WEBKIT above:
> > > 
> > > WEBKIT += wtf javascriptcore webcore
> > 
> > We have javascriptcore.prf, qt_webkit.prf, webcore.prf, webkit2.prf and
> > wtf.prf named files now. I think we should change them later, separately
> > from this bug if we really want to do it.
> 
> What do you mean?

I meant that prf's has lowercase only names now.

If we change wtf to WTF, we should change javascriptcore 
to JavaScriptCore and webcore to WebCore, ...

[Tor Arne Vestbø]

(In reply to comment #8)
> (In reply to comment #7)
> > (In reply to comment #6)
> > > (In reply to comment #5)
> > > > You should be able to replace this with modifying WEBKIT above:
> > > > 
> > > > WEBKIT += wtf javascriptcore webcore
> > > 
> > > We have javascriptcore.prf, qt_webkit.prf, webcore.prf, webkit2.prf and
> > > wtf.prf named files now. I think we should change them later, separately
> > > from this bug if we really want to do it.
> > 
> > What do you mean?
> 
> I meant that prf's has lowercase only names now.
> 
> If we change wtf to WTF, we should change javascriptcore 
> to JavaScriptCore and webcore to WebCore, ...

Um, the WEBKIT variable has all lowercase values, there's no inconsistency, and no reason to make 'wtf' be 'WTF'. It's consistent with the QT variable, which also lists the qt modules you depend on, all lowercase.

What I was suggesting was the proper fix for the includepath-problem. If you disagree about the casing of the values of the WEBKIT variable that's no reason to not use the mechanism to solve the issue of the includepaths?

[Csaba Osztrogonác]

Sorry, I absolutely misinterpreted you.

You suggested using "WEBKIT += wtf javascriptcore webcore"
instead of adding more includepaths to DRT.pro. I got it now. :)

I can check it tomorrow, but not today.

[Csaba Osztrogonác]

It was merged to https://bugs.webkit.org/show_bug.cgi?id=81844

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