[GTK] Enable introspection always for developer builds
authormrobinson@webkit.org <mrobinson@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 22 Apr 2013 22:15:37 +0000 (22:15 +0000)
committermrobinson@webkit.org <mrobinson@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 22 Apr 2013 22:15:37 +0000 (22:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=114983

Reviewed by Gustavo Noronha Silva.

.:

* Source/autotools/SetupAutoconfHeader.m4: No longer expose the ENABLE_INTROSPECTION
autoconf header variable. It isn't used and it means that when introspection is enabled
or disabled, there is an unnecessary full rebuild.

Source/WebKit/gtk:

* GNUmakefile.am: Make warnings during gir scanning cause the build to fail. Eliminate
warnings about deprecated API that the g-ir-scanner uses when generating
the scanner program. Use --quiet to avoid printing lots of output to
non-verbose builds.

Source/WebKit2:

* GNUmakefile.am: Make warnings during gir scanning cause the build to fail. Eliminate
warnings about deprecated API that the g-ir-scanner uses when generating
the scanner program. Use --quiet.

Tools:

* Scripts/webkitdirs.pm:
(runAutogenForAutotoolsProjectIfNecessary): Always enable introspection
when using build-webkit.
* gtk/jhbuildrc: Enable documentation for all jhbuild modules and explicitly
choose the memory gsettings backend while in the jhbuild. This eliminates
warnings from programs that fall back to the memory backend.

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

ChangeLog
Source/WebKit/gtk/ChangeLog
Source/WebKit/gtk/GNUmakefile.am
Source/WebKit2/ChangeLog
Source/WebKit2/GNUmakefile.am
Source/autotools/SetupAutoconfHeader.m4
Tools/ChangeLog
Tools/Scripts/webkitdirs.pm
Tools/gtk/jhbuildrc

index df562f0..4362335 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2013-04-22  Martin Robinson  <mrobinson@igalia.com>
+
+        [GTK] Enable introspection always for developer builds
+        https://bugs.webkit.org/show_bug.cgi?id=114983
+
+        Reviewed by Gustavo Noronha Silva.
+
+        * Source/autotools/SetupAutoconfHeader.m4: No longer expose the ENABLE_INTROSPECTION
+        autoconf header variable. It isn't used and it means that when introspection is enabled
+        or disabled, there is an unnecessary full rebuild.
+
 2013-04-22  Zan Dobersek  <zdobersek@igalia.com>
 
         [GTK] Set up libPlatform.la
index 4e10e4e..9987cf5 100644 (file)
@@ -1,5 +1,17 @@
 2013-04-22  Martin Robinson  <mrobinson@igalia.com>
 
+        [GTK] Enable introspection always for developer builds
+        https://bugs.webkit.org/show_bug.cgi?id=114983
+
+        Reviewed by Gustavo Noronha Silva.
+
+        * GNUmakefile.am: Make warnings during gir scanning cause the build to fail. Eliminate
+        warnings about deprecated API that the g-ir-scanner uses when generating
+        the scanner program. Use --quiet to avoid printing lots of output to
+        non-verbose builds.
+
+2013-04-22  Martin Robinson  <mrobinson@igalia.com>
+
         [GTK] Fix remaining introspection warnings
         https://bugs.webkit.org/show_bug.cgi?id=114980
 
index 48c2b26..ab6f9b3 100644 (file)
@@ -295,7 +295,12 @@ pkgconfig_DATA += Source/WebKit/gtk/@WEBKITGTK_PC_NAME@-@WEBKITGTK_API_VERSION@.
 
 if ENABLE_INTROSPECTION
 WebKit-@WEBKITGTK_API_VERSION@.gir: $(G_IR_SCANNER) JavaScriptCore-@WEBKITGTK_API_VERSION@.gir libwebkitgtk-@WEBKITGTK_API_MAJOR_VERSION@.@WEBKITGTK_API_MINOR_VERSION@.la
-       $(AM_V_GEN)$(G_IR_SCANNER) -v --warn-all \
+       $(AM_V_GEN) \
+            CFLAGS="$(CFLAGS) -Wno-deprecated-declarations" \
+            $(G_IR_SCANNER) \
+            --quiet \
+            --warn-all \
+            --warn-error \
             --symbol-prefix=webkit \
             --identifier-prefix=WebKit \
             --namespace=WebKit \
index 5172d62..9d5d19c 100644 (file)
@@ -1,3 +1,14 @@
+2013-04-22  Martin Robinson  <mrobinson@igalia.com>
+
+        [GTK] Enable introspection always for developer builds
+        https://bugs.webkit.org/show_bug.cgi?id=114983
+
+        Reviewed by Gustavo Noronha Silva.
+
+        * GNUmakefile.am: Make warnings during gir scanning cause the build to fail. Eliminate
+        warnings about deprecated API that the g-ir-scanner uses when generating
+        the scanner program. Use --quiet.
+
 2013-04-22  Anders Carlsson  <andersca@apple.com>
 
         Make it easier to extend NetworkResourceLoadParameters
index 5598f7e..73b295d 100644 (file)
@@ -262,7 +262,12 @@ pkgconfig_DATA += Source/WebKit2/webkit2gtk-@WEBKITGTK_API_VERSION@.pc
 if ENABLE_INTROSPECTION
 
 WebKit2-@WEBKITGTK_API_VERSION@.gir: $(G_IR_SCANNER) libwebkit2gtk-@WEBKITGTK_API_MAJOR_VERSION@.@WEBKITGTK_API_MINOR_VERSION@.la
-       $(AM_V_GEN)$(G_IR_SCANNER) -v --warn-all \
+       $(AM_V_GEN) \
+            CFLAGS="$(CFLAGS) -Wno-deprecated-declarations" \
+            $(G_IR_SCANNER) \
+            --quiet \
+            --warn-all \
+            --warn-error \
             --symbol-prefix=webkit \
             --identifier-prefix=WebKit \
             --namespace=WebKit2 \
index 4f5cb2e..2cb40ea 100644 (file)
@@ -58,10 +58,6 @@ if test "$enable_opcode_stats" = "yes"; then
     AC_DEFINE([ENABLE_OPCODE_STATS], [1], [Define to enable Opcode statistics])
 fi
 
-if test "$enable_introspection" = "yes"; then
-    AC_DEFINE([ENABLE_INTROSPECTION], [1], [Define to enable GObject introspection support])
-fi
-
 if test "$enable_video" = "yes" || test "$enable_web_audio" = "yes"; then
     AC_DEFINE([WTF_USE_GSTREAMER], [1], [ ])
     if test "$enable_debug" = "yes"; then
index b3a9b64..c618d12 100644 (file)
@@ -1,3 +1,17 @@
+2013-04-22  Martin Robinson  <mrobinson@igalia.com>
+
+        [GTK] Enable introspection always for developer builds
+        https://bugs.webkit.org/show_bug.cgi?id=114983
+
+        Reviewed by Gustavo Noronha Silva.
+
+        * Scripts/webkitdirs.pm:
+        (runAutogenForAutotoolsProjectIfNecessary): Always enable introspection
+        when using build-webkit.
+        * gtk/jhbuildrc: Enable documentation for all jhbuild modules and explicitly
+        choose the memory gsettings backend while in the jhbuild. This eliminates
+        warnings from programs that fall back to the memory backend.
+
 2013-04-22  Zan Dobersek  <zdobersek@igalia.com>
 
         [GTK] Set up libPlatform.la
index 3826584..028543e 100755 (executable)
@@ -1994,6 +1994,9 @@ sub runAutogenForAutotoolsProjectIfNecessary($@)
 {
     my ($dir, $prefix, $sourceDir, $project, $joinedOverridableFeatures, @buildArgs) = @_;
 
+    # Always enable introspection when building WebKitGTK+.
+    unshift(@buildArgs, "--enable-introspection");
+
     my $joinedBuildArgs = join(" ", @buildArgs);
 
     if (-e "GNUmakefile") {
index 71e7f37..f582b27 100644 (file)
@@ -54,3 +54,11 @@ module_makeargs['gtk+'] = 'LDFLAGS="-lm" ' + makeargs
 # Avoid runtime conflicts with GStreamer system-wide plugins. We want
 # to use only the plugins we build in JHBuild.
 os.environ['GST_PLUGIN_SYSTEM_PATH'] = ''
+
+# We often end up using the memory backend anyway, so explicitly choosing
+# it will prevent the warning message.
+os.environ['GSETTINGS_BACKEND'] = 'memory'
+
+# We always enable introspection so that we can sniff out problems with our
+# annotations sooner rather than later.
+autogenargs='--enable-introspection'