Now that cmake is being used, it should be trivial to use the 'Eclipse CDT4 - Ninja' generator, which does the same as the 'Ninja' generator currently in use, but also creates .project and .cproject files suitable for Eclipse. This would be much better than the approach used in Autotools+Makefile times, where the source code import had some troubles.
Created attachment 230167 [details] Patch
Just for the record, I've been used this patch successfully in my box for some time. The only thing needed is to create two symbolic links: .project -> WebKitBuild/Release/.project .cproject -> WebKitBuild/Release/.cproject This is because the project file must be in the root directory of the project (WebKit). After that, just create the project in Eclipse via File, Import, General, Existing projects into workspace. Select the WebKit root dir and perform the import. Enjoy!
Comment on attachment 230167 [details] Patch Looks OK to me.
This seems risky. There are probably people who have eclipse in their path who don't want to use Eclipse to build.
The eclipse target is a superset of the ninja target. It also generates ninja files, so you can build it "the ninja way". That's how I build it. Assuming that you want to use eclipse if you have it in your path is as risky as assuming that you want to use ninja if you have it in your path (the current approach).
Are the ninja files generated for the eclipse target the same as for the ninja target?
The rules.ninja files are equal. In the build.ninja files, some rules are in different places in the file (some are before, some are after, but I think that's irrelevant for ninja). I know it's not orthodox, but I've "sort"ed both versions of build.ninja and "diff"ed them after that to bypass the place shifting differences. The resulting diff showed 3393 different lines out of 31633. Examining them closely, the main part of them just differ in the position of lib/libANGLESupport.a in the argument list (it's even missing in some of the blocks generated by the Eclipse Ninja target). Apart from that, the "build build.ninja" line is different, as it calls different cmake system files, some only related to Eclipse.
Okay. Sound safe, but we should be ready to only do this when requested if it causes issues.
Comment on attachment 230167 [details] Patch Clearing flags on attachment: 230167 Committed r167932: <http://trac.webkit.org/changeset/167932>
All reviewed patches have been landed. Closing bug.
*** Bug 132062 has been marked as a duplicate of this bug. ***