Bug 80680 – Move WTF-related Windows project files out of JavaScriptCore

• Component: New Bugs
• Status: RESOLVED FIXED

[Steve Falkenburg]

Move WTF-related Windows project files out of JavaScriptCore

[Steve Falkenburg]

Created attachment 130993 [details]
Patch

[Steve Falkenburg]

Created attachment 130998 [details]
Patch

[Jessie Berlin]

Comment on attachment 130998 [details]
Patch

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

Do we need to worry about this working with build-webkit? Eric's comment in https://bugs.webkit.org/show_bug.cgi?id=80553#c13 implies that it wouldn't work yet.

r=me (assuming it doesn't screw up build-webkit)

> Source/WTF/WTF.vcproj/work-around-vs-dependency-tracking-bugs.py:11
> +TOP_LEVEL_DIRECTORY = os.path.dirname(os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__)))))

Why is calling dirname no longer necessary here?

[Eric Seidel (no email)]

Last time I looked build-webkit did not attempt to build the Source/WTF directory.  I have a patch to do that on mac, but I haven't landed it yet.

I'm happy to land the mac part (it will be trivial to extend it for Win) if that would be helpful to you.

[Eric Seidel (no email)]

In case it's useful to you, I just posted a patch which will make build-webkit build Source/WTF when building Mac: bug 80728.

It would be trivial to change that line to add an "or isAppleWinWebKit()"

[Eric Seidel (no email)]

Once this lands, I will look into fixing the win-ews bot for bug 80911.

[Eric Seidel (no email)]

Is there anything I can do to help here?  I'm interested in seeing this land as it unblocks the next steps in the move. :)

[Steve Falkenburg]

This will need to be updated before landing (to pull in the latest vcproj changes).
The soonest I can get to this is next week.

We'd rather land this just before you're ready for your next steps on the move. Can you give an ETA on that?

[Eric Seidel (no email)]

(In reply to comment #8)
> This will need to be updated before landing (to pull in the latest vcproj changes).
> The soonest I can get to this is next week.
> 
> We'd rather land this just before you're ready for your next steps on the move. Can you give an ETA on that?

I've moved away from the "break everything at once, and then try and rebuild everything" model, to a "use the EWS bots to avoid breaking anyone" model, which I've been doing asynchronously with other things.

The patch to do the next step (of moving a single header out of JavaScriptCore/wtf to WTF/wtf) is already posted on bug 80911.  I'm fixing Chromium and the other EWS bots as we speak.

My current plan is as follows:

1.  Move a single header (Platform.h) from JSC/wtf to WTF/wtf, fix all the builders.
2.  Move a single compilation unit (probably NullPtr.*), fix all the builders.
3.  Announce on webkit-dev that I'm moving the rest.
4.  Move the rest (which should be trivial now that the concept is proven), fix all the builders.

I'm happy to revert to my previous plan of basically just steps 3/4.  I'd been having good luck with this slow async method, but whatever works for you all.


I can also look into editing the exiting JavaScriptCore/wtf/WTF.vcproj to point to ../../WTF/wtf/Platform.h and see if that builds.

[Eric Seidel (no email)]

Again, I'm not in any huge hurry here.  Just checking back on this project.  I'll see about editing JavaScriptCore/wtf/WTF.vcproj to point to WTF/wtf/Platform.h for my patch on bug 80911 and see if that builds.

[Brent Fulgham]

I'm getting tons of build errors locally with these changes (12>e:\Projects\WebKit\Source\WebKit2\config.h(35) : fatal error C1083: Cannot open include file: 'wtf/DisallowCType.h': No such file or directory).

Somehow the WinCairo port isn't seeing the same paths as the main Windows build bot.

I'm trying to figure things out...

[Jessie Berlin]

(In reply to comment #11)
> I'm getting tons of build errors locally with these changes (12>e:\Projects\WebKit\Source\WebKit2\config.h(35) : fatal error C1083: Cannot open include file: 'wtf/DisallowCType.h': No such file or directory).
> 
> Somehow the WinCairo port isn't seeing the same paths as the main Windows build bot.
> 
> I'm trying to figure things out...

You probably need to make sure the Cairo vsprops files have the correct new include path for the wtf headers (see http://trac.webkit.org/changeset/110133 for examples).

[Brent Fulgham]

I'm able to build locally now. Hopefully the build bot will catch up soon!

[Eric Seidel (no email)]

I believe I've now fixed all other platforms besides win (for bug 80911).  Once this patch lands, I can fix Win and move to the next stage of the WTF move. :)  Again, let me know if I can help!

[Eric Seidel (no email)]

Odd.  The patch does not apply because svn-apply doesn't seem to see WTF.vcproj as an add.  I suspect that's a bug in svn-apply!

[Steve Falkenburg]

Committed r111438: <http://trac.webkit.org/changeset/111438>

[Eric Seidel (no email)]

Thank you!

[Daniel Bates]

(In reply to comment #15)
> Odd.  The patch does not apply because svn-apply doesn't seem to see WTF.vcproj as an add.  I suspect that's a bug in svn-apply!

Oh noes! Will look into it shortly.