[webkit-reviews] review denied: [Bug 122787] JavaScriptCore tests should run on Windows : [Attachment 214231] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Oct 16 10:53:37 PDT 2013


Brent Fulgham <bfulgham at webkit.org> has denied Alex Christensen
<alex.christensen at flexsim.com>'s request for review:
Bug 122787: JavaScriptCore tests should run on Windows
https://bugs.webkit.org/show_bug.cgi?id=122787

Attachment 214231: Patch
https://bugs.webkit.org/attachment.cgi?id=214231&action=review

------- Additional Comments from Brent Fulgham <bfulgham at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=214231&action=review


The overall idea here is great, but your change would cause catastrophic
breaking of our internal build process so I must reject this patch.

> Source/JavaScriptCore/JavaScriptCore.vcxproj/JavaScriptCore.submit.sln:44
> +Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "WTF",
"..\..\WTF\WTF.vcxproj\WTF.vcxproj", "{8EF73779-BED3-45BB-816D-9FF58399AFA5}"

You can't do this. WTF.submit.sln and JavaScriptCore.submit.sln are isolated
from each-other for a reason.

What you COULD do, is make sure there is a JavaScriptCore.sln that looks
exactly like this file, and have build-jsc use that file (instead of
JavaScriptCore.submit.sln).

> Source/WTF/WTF.vcxproj/WTF.submit.sln:-2
> -Microsoft Visual Studio Solution File, Format Version 11.00

Why are you removing this file?  It's critical.


More information about the webkit-reviews mailing list