[Webkit-unassigned] [Bug 16669] autotools update and fixes

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=16669


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #18171|review?                     |review-
               Flag|                            |




------- Comment #2 from darin at apple.com  2007-12-29 14:22 PDT -------
(From update of attachment 18171)
Although it's not new to this patch, I think it's a seriously bad idea that
GNUMakeFile.am has a copy of the contents of DerivedSources.make -- the whole
idea of DerivedSources.make is to have a shared makefile for these more
complicated rules. Surely we can find a way to incorporate those rules without
copying and pasting the entire file.

+#if ENABLE(DATABASE)
+#include "Database.h"
+#endif

This change is wrong. The file "Database.h" already has an #if ENABLE(DATABASE)
around its contents. We don't want to ifdef it in both places.

+#if ENABLE(DATABASE)
+#include "DatabaseTracker.h"
+#endif

We need to be consistent. The ENABLE #if statements are inside Database.h and
should go inside DatabaseTracker.h too. Or we could go the other way and change
how we handle optional things like database.

ENABLE_XBL should not be in the makefile. XBL was an experiment done more than
two years ago. It doesn't make sense to have it commented-out code in there
with a FIXME.

Everything else looks fine.

review- because of the database include files issue.


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.