Bug 13226 - Remove Bakefiles from svn
Summary: Remove Bakefiles from svn
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-03-29 14:32 PDT by Holger Freyther
Modified: 2007-09-29 08:52 PDT (History)
3 users (show)

See Also:


Attachments
Remove bakefile support (81.55 KB, patch)
2007-03-29 14:33 PDT, Holger Freyther
mrowe: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.