Bug 13226

Summary: Remove Bakefiles from svn
Product: WebKit Reporter: Holger Freyther <zecke>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: alp, ap, mh+webkit
Priority: P2    
Version: 420+   
Hardware: All   
OS: All   
Attachments:
Description Flags
Remove bakefile support mrowe: review+

Description Holger Freyther 2007-03-29 14:32:09 PDT
As of bug #11308 it seems Bakefiles is not the preferred Gdk buildsystem anymore. It is orphaned non-working and still requires the special Bakefile release. The additional patches have either been not posted upstream or were rejected.
I propose to remove Bakefile support and concentrate on either qmake or cmake for all platforms. I have started to spend time on reviving cmake support.
Comment 1 Holger Freyther 2007-03-29 14:33:19 PDT
Created attachment 13878 [details]
Remove bakefile support

 Bakefiles/Bakefiles.bkgen                |   91 ---
 Bakefiles/ChangeLog                      |   35 -
 Bakefiles/Readme.txt                     |   99 ----
 Bakefiles/presets.bkl                    |  201 --------
 Bakefiles/update-file-lists.py           |  180 -------
 ChangeLog                                |   10 
 JavaScriptCore/ChangeLog                 |    7 
 JavaScriptCore/JavaScriptCoreSources.bkl |   86 ---
 JavaScriptCore/jscore.bkl                |  111 ----
 WebCore/ChangeLog                        |   11 
 WebCore/Projects/gdk/webcore-gdk.bkl     |   92 ---
 WebCore/Projects/wx/webcore-wx.bkl       |   50 --
 WebCore/WebCoreSources.bkl               |  748 -------------------------------
 WebCore/webcore-base.bkl                 |  142 -----
 WebKitTools/ChangeLog                    |    9 
 WebKitTools/GdkLauncher/Bakefiles.bkgen  |   42 -
 WebKitTools/GdkLauncher/gdklauncher.bkl  |   84 ---
 WebKitTools/GdkLauncher/mk               |    6 
 WebKitTools/Scripts/regenerate-makefiles |   73 ---
 19 files changed, 37 insertions(+), 2040 deletions(-)
Comment 2 Mark Rowe (bdash) 2007-04-06 19:55:44 PDT
Comment on attachment 13878 [details]
Remove bakefile support

As far as I'm aware the Bakefile build system is still what Gdk uses to build, so we can't go removing it just yet.  Bug 11308 is about replacing it with something better, and once that is done we can remove the Bakefile system.
Comment 3 Alexey Proskuryakov 2007-04-25 04:55:27 PDT
Bug 11308 resolved as WONTFIX => resolving this one, too.
Comment 4 Holger Freyther 2007-08-23 05:55:29 PDT
I think the situation has changed. The only user of the Bakefile system is the wx port which is not developed inside trunk but in a branch.

To avoid possible confusion one might want to remove the Bakefiles.
Comment 5 Holger Freyther 2007-09-23 08:21:23 PDT
Comment on attachment 13878 [details]
Remove bakefile support

I'm asking to re-evaluate this patch. I think either the Gtk+ specific Bakefiles should be removed or all of them.
Comment 6 Alp Toker 2007-09-24 21:12:35 PDT
I'm all for this cleanup. Leaving the old build system around is confusing to newcomers, especially when they read the out of date build instructions within the source tree and on old webpages.
Comment 7 Mark Rowe (bdash) 2007-09-28 22:59:54 PDT
Comment on attachment 13878 [details]
Remove bakefile support

Go for it!  Can WebKitTools/GdkLauncher/ENV go too?
Comment 8 Holger Freyther 2007-09-29 02:59:40 PDT
(In reply to comment #7)
> (From update of attachment 13878 [details] [edit])
> Go for it!  Can WebKitTools/GdkLauncher/ENV go too?

I can remove that as well. I assumed that it was not part of the Bakefiles but just for Mike Emmel's convenience. I'm going to land that tonight.

Comment 9 Holger Freyther 2007-09-29 08:52:30 PDT
Landed in r25808.