Summary: | Remove Bakefiles from svn | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Holger Freyther <zecke> | ||||
Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | alp, ap, mh+webkit | ||||
Priority: | P2 | ||||||
Version: | 420+ | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Holger Freyther
2007-03-29 14:32:09 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 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. Bug 11308 resolved as WONTFIX => resolving this one, too. 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 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.
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 on attachment 13878 [details]
Remove bakefile support
Go for it! Can WebKitTools/GdkLauncher/ENV go too?
(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. |