Title: [89675] trunk
Revision
89675
Author
[email protected]
Date
2011-06-24 08:54:10 -0700 (Fri, 24 Jun 2011)

Log Message

2011-06-24  Carlos Garcia Campos  <[email protected]>

        Reviewed by Martin Robinson.

        [GTK] Add more debug options to compilation
        https://bugs.webkit.org/show_bug.cgi?id=63318

        Add --enable-debug-symbols to enable/disable compilation with
        debug symbols and --enable-debug-features to enable/disable debug
        features. Both options are unconditionally enabled for debug
        builds.

        * configure.ac:

Modified Paths

Diff

Modified: trunk/ChangeLog (89674 => 89675)


--- trunk/ChangeLog	2011-06-24 15:43:35 UTC (rev 89674)
+++ trunk/ChangeLog	2011-06-24 15:54:10 UTC (rev 89675)
@@ -1,3 +1,17 @@
+2011-06-24  Carlos Garcia Campos  <[email protected]>
+
+        Reviewed by Martin Robinson.
+
+        [GTK] Add more debug options to compilation
+        https://bugs.webkit.org/show_bug.cgi?id=63318
+
+        Add --enable-debug-symbols to enable/disable compilation with
+        debug symbols and --enable-debug-features to enable/disable debug
+        features. Both options are unconditionally enabled for debug
+        builds.
+
+        * configure.ac:
+
 2011-06-23  Oliver Hunt  <[email protected]>
 
         Qt build-fix: remove warning that no other platform appears to use.

Modified: trunk/configure.ac (89674 => 89675)


--- trunk/configure.ac	2011-06-24 15:43:35 UTC (rev 89674)
+++ trunk/configure.ac	2011-06-24 15:54:10 UTC (rev 89675)
@@ -812,6 +812,22 @@
               [],[if test "$enable_debug" = "yes"; then enable_fast_malloc="no"; else enable_fast_malloc="yes"; fi])
 AC_MSG_RESULT([$enable_fast_malloc])
 
+# check whether to enable debug symbols
+AC_MSG_CHECKING([whether to enable debug symbols])
+AC_ARG_ENABLE(debug_symbols,
+              AC_HELP_STRING([--enable-debug-symbols],
+                             [enable debug symbols default=no, default=yes for debug builds]),
+              [],[if test "$enable_debug" = "yes"; then enable_debug_symbols="yes"; else enable_debug_symbols="no"; fi])
+AC_MSG_RESULT([$enable_debug_symbols])
+
+# check whether to enable debug features
+AC_MSG_CHECKING([whether to enable debug features])
+AC_ARG_ENABLE(debug_features,
+              AC_HELP_STRING([--enable-debug-features],
+                             [enable debug features default=no, default=yes for debug builds]),
+              [],[if test "$enable_debug" = "yes"; then enable_debug_features="yes"; else enable_debug_features="no"; fi])
+AC_MSG_RESULT([$enable_debug_features])
+
 AC_MSG_CHECKING([whether to enable JIT compilation])
 AC_ARG_ENABLE([jit],
               AC_HELP_STRING([--enable-jit],
@@ -950,11 +966,13 @@
    CFLAGS="$CFLAGS -mieee -w"
 fi
 
-# Add '-g' flag to gcc if it's debug build
-if test "$enable_debug" = "yes"; then
+# Add '-g' flag to gcc to build with debug symbols
+if test "$enable_debug_symbols" = "yes"; then
    CXXFLAGS="$CXXFLAGS -g"
    CFLAGS="$CFLAGS -g"
-else
+fi
+
+if test "$enable_debug_features" = "no"; then
    AC_DEFINE([NDEBUG], [1], [Define to disable debugging features])
 fi
 
@@ -1113,7 +1131,7 @@
 AM_CONDITIONAL([USE_GSTREAMER], [test "$have_gstreamer" = "yes"])
 
 # WebKit feature conditionals
-AM_CONDITIONAL([ENABLE_DEBUG],[test "$enable_debug" = "yes"])
+AM_CONDITIONAL([ENABLE_DEBUG],[test "$enable_debug_features" = "yes"])
 AM_CONDITIONAL([ENABLE_3D_TRANSFORMS],[test "$enable_3d_transforms" = "yes"])
 AM_CONDITIONAL([ENABLE_WEBGL],[test "$enable_webgl" = "yes"])
 AM_CONDITIONAL([ENABLE_BLOB],[test "$enable_blob" = "yes"])
@@ -1216,6 +1234,8 @@
 
 Build configuration:
  Enable debugging (slow)                                  : $enable_debug
+ Compile with debug symbols (slow)                        : $enable_debug_symbols
+ Enable debug features (slow)                             : $enable_debug_features
  Enable GCC build optimization                            : $enable_optimizations
  Code coverage support                                    : $enable_coverage
  Unicode backend                                          : $with_unicode_backend
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to