[Webkit-unassigned] [Bug 60146] [CMAKE] Refactoring wtf related code.

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Ryuan Choi <ryuan.choi at samsung.com>  2011-05-04 04:10:52 PST ---
(In reply to comment #2)
> (From update of attachment 92208 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=92208&action=review
> 
> Expect for the wtf-files, LGTM
> 
> > Source/JavaScriptCore/CMakeLists.txt:-174
> > -
> > -    wtf/DateMath.cpp
> > -    wtf/PageAllocationAligned.cpp
> > -    wtf/PageBlock.cpp
> 
> I'm not sure if that's correct. Some other ports add this to JSC and not to WTF too. (I don't know why they do that and if i remenber corretly I had some problem with this files in in the WinCE port)
> e.g. http://trac.webkit.org/browser/trunk/Source/JavaScriptCore/JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj?rev=83506#L641
> 
> At lest the move of the files can be put into an addtional patch.

Thank you for information.

Yes, I got an errors while moving those and just add "${JavaScriptCore_INCLUDE_DIRECTORIES}" in WTF_INCLUDE_DIRECTORIES to avoid.

I'll investigate it more.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.