[GTK] Auto generate webkitmarshal.list
authorchristian@webkit.org <christian@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 Jul 2008 18:17:24 +0000 (18:17 +0000)
committerchristian@webkit.org <christian@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 30 Jul 2008 18:17:24 +0000 (18:17 +0000)
Auto generate webkitmarshal.list from source files to reduce the
number of conflicts when merging git branches or when applying
patches.
Also rename webkit-marshal.* to webkitmarshal.* for consistency.

Reviewed by Holger.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@35455 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
GNUmakefile.am
WebKit/gtk/ChangeLog
WebKit/gtk/webkit/webkitwebframe.cpp
WebKit/gtk/webkit/webkitwebview.cpp

index de36663..974d00e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2008-07-30  Marco Barisione  <marco.barisione@collabora.co.uk>
+
+        Reviewed by Holger.
+
+        http://bugs.webkit.org/show_bug.cgi?id=19742
+        [GTK] Auto generate webkitmarshal.list
+
+        Auto generate webkitmarshal.list from source files to reduce the
+        number of conflicts when merging git branches or when applying
+        patches.
+        Also rename webkit-marshal.* to webkitmarshal.* for consistency.
+
+        * GNUmakefile.am:
+
 2008-07-25  Jan Michael Alonzo  <jmalonzo@webkit.org>
 
         Reviewed by Mark Rowe.
index c5edcc0..9b2f06f 100644 (file)
@@ -320,8 +320,8 @@ webkitgtk_h_api += \
        WebKit/gtk/webkit/webkitwebview.h
 
 webkitgtk_built_sources += \
-       DerivedSources/webkit-marshal.h \
-       DerivedSources/webkit-marshal.cpp \
+       DerivedSources/webkitmarshal.h \
+       DerivedSources/webkitmarshal.cpp \
        DerivedSources/webkitenumtypes.cpp \
        WebKit/gtk/webkit/webkitenumtypes.h
 
@@ -367,26 +367,37 @@ pkgconfigdir = $(libdir)/pkgconfig
 pkgconfig_DATA = WebKit/gtk/webkit-1.0.pc
 
 stamp_files := \
-       stamp-webkit-marshal.cpp \
-       stamp-webkit-marshal.h \
+       stamp-webkitmarshal.cpp \
+       stamp-webkitmarshal.h \
        stamp-webkitenumtypes.cpp \
        stamp-webkitenumtypes.h
 
-WEBKIT_MARSHAL = $(GENSOURCES)/webkit-marshal
-WEBKIT_MARSHAL_LIST = $(srcdir)/WebKit/gtk/webkit/webkit-marshal.list
-
-DerivedSources/webkit-marshal.cpp: stamp-webkit-marshal.cpp
+WEBKIT_MARSHAL = $(GENSOURCES)/webkitmarshal
+WEBKIT_MARSHAL_LIST = $(WEBKIT_MARSHAL).list
+
+$(WEBKIT_MARSHAL_LIST): $(webkitgtk_sources) GNUmakefile.am
+       ( cd $(top_srcdir) && \
+       sed -n -e 's/.*webkit_marshal_\([[:upper:][:digit:]]*__[[:upper:][:digit:]_]*\).*/\1/p' \
+       $(webkitgtk_sources) ) \
+       | sed -e 's/__/:/' -e 'y/_/,/' | sort -u > $@.tmp
+       if cmp -s $@.tmp $@; then \
+               rm $@.tmp; \
+       else \
+               mv $@.tmp $@; \
+       fi
+
+$(WEBKIT_MARSHAL).cpp: stamp-webkitmarshal.cpp
        @true
 
-DerivedSources/webkit-marshal.h: stamp-webkit-marshal.h
+$(WEBKIT_MARSHAL).h: stamp-webkitmarshal.h
        @true
 
-stamp-webkit-marshal.cpp: $(WEBKIT_MARSHAL_LIST)
+stamp-webkitmarshal.cpp: $(WEBKIT_MARSHAL_LIST)
        echo "extern \"C\" {" > $(WEBKIT_MARSHAL).cpp && \
        $(GLIB_GENMARSHAL) --prefix=webkit_marshal $(WEBKIT_MARSHAL_LIST) --body >> $(WEBKIT_MARSHAL).cpp && echo '}' >> $(WEBKIT_MARSHAL).cpp && \
        echo timestamp > $(@F)
 
-stamp-webkit-marshal.h: $(WEBKIT_MARSHAL_LIST)
+stamp-webkitmarshal.h: $(WEBKIT_MARSHAL_LIST)
        $(GLIB_GENMARSHAL) --prefix=webkit_marshal $(WEBKIT_MARSHAL_LIST) --header > $(WEBKIT_MARSHAL).h && \
        echo timestamp > $(@F)
 
index 46685a8..32439e0 100644 (file)
@@ -1,3 +1,19 @@
+2008-07-30  Marco Barisione  <marco.barisione@collabora.co.uk>
+
+        Reviewed by Holger.
+
+        http://bugs.webkit.org/show_bug.cgi?id=19742
+        [GTK] Auto generate webkitmarshal.list
+
+        Auto generate webkitmarshal.list from source files to reduce the
+        number of conflicts when merging git branches or when applying
+        patches.
+        Also rename webkit-marshal.* to webkitmarshal.* for consistency.
+
+        * webkit/webkitwebframe.cpp: Include webkitmarshal.h instead of
+         webkit-marshal.h.
+        * webkit/webkitwebview.cpp: Ditto.
+
 2008-07-30  Gustavo Noronha Silva  <gns@gnome.org>
 
         Reviewed by Sam.
index a7f27ef..102dc05 100644 (file)
@@ -25,7 +25,7 @@
 
 #include "webkitwebframe.h"
 #include "webkitwebview.h"
-#include "webkit-marshal.h"
+#include "webkitmarshal.h"
 #include "webkitprivate.h"
 
 #include "CString.h"
index b9ec617..7f99949 100644 (file)
@@ -25,7 +25,7 @@
 #include "config.h"
 
 #include "webkitwebview.h"
-#include "webkit-marshal.h"
+#include "webkitmarshal.h"
 #include "webkitprivate.h"
 #include "webkitwebbackforwardlist.h"
 #include "webkitwebhistoryitem.h"