projects / dana / cg-glib.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 24273ca)
INCLUDES is the right way to add to CFLAGS, not DEFS. Also there are bugs
authorJeff Garzik <jgarzik@src.gnome.org>
Thu, 12 Nov 1998 04:28:52 +0000 (04:28 +0000)
committerJeff Garzik <jgarzik@src.gnome.org>
Thu, 12 Nov 1998 04:28:52 +0000 (04:28 +0000)
INCLUDES is the right way to add to CFLAGS, not DEFS.
Also there are bugs with '+=' in makefiles.

Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
Removed redundant -I from INCLUDES.

This fixes an annoying bug where autoconf's DEFS, which includes
the important -DHAVE_CONFIG_H, was lost.  (due to += weirdness)

12 files changed:
ChangeLog patch | blob | history
ChangeLog.pre-2-0 patch | blob | history
ChangeLog.pre-2-10 patch | blob | history
ChangeLog.pre-2-12 patch | blob | history
ChangeLog.pre-2-2 patch | blob | history
ChangeLog.pre-2-4 patch | blob | history
ChangeLog.pre-2-6 patch | blob | history
ChangeLog.pre-2-8 patch | blob | history
Makefile.am patch | blob | history
glib/Makefile.am patch | blob | history
gmodule/ChangeLog patch | blob | history
gmodule/Makefile.am patch | blob | history

diff --git a/ChangeLog b/ChangeLog
index 8622b707f6c3612e31b0eba9617b283b598b1fcb..83bdc892fdcdf61e3f33608234a248c07592fa16 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+       * Makefile.am : INCLUDES is the right way to add to CFLAGS, not
+       DEFS.  Also there are bugs with '+=' in makefiles.
+       Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+       Removed redundant -I from INCLUDES.
+
 Wed Nov 11 18:11:24 EST 1998 Gregory McLean <gregm@comstar.net>
 
        * docs/*.sgml : Batch of new documentation that should be easier
diff --git a/ChangeLog.pre-2-0 b/ChangeLog.pre-2-0
index 8622b707f6c3612e31b0eba9617b283b598b1fcb..83bdc892fdcdf61e3f33608234a248c07592fa16 100644 (file)
--- a/ChangeLog.pre-2-0
+++ b/ChangeLog.pre-2-0
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+       * Makefile.am : INCLUDES is the right way to add to CFLAGS, not
+       DEFS.  Also there are bugs with '+=' in makefiles.
+       Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+       Removed redundant -I from INCLUDES.
+
 Wed Nov 11 18:11:24 EST 1998 Gregory McLean <gregm@comstar.net>
 
        * docs/*.sgml : Batch of new documentation that should be easier
diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10
index 8622b707f6c3612e31b0eba9617b283b598b1fcb..83bdc892fdcdf61e3f33608234a248c07592fa16 100644 (file)
--- a/ChangeLog.pre-2-10
+++ b/ChangeLog.pre-2-10
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+       * Makefile.am : INCLUDES is the right way to add to CFLAGS, not
+       DEFS.  Also there are bugs with '+=' in makefiles.
+       Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+       Removed redundant -I from INCLUDES.
+
 Wed Nov 11 18:11:24 EST 1998 Gregory McLean <gregm@comstar.net>
 
        * docs/*.sgml : Batch of new documentation that should be easier
diff --git a/ChangeLog.pre-2-12 b/ChangeLog.pre-2-12
index 8622b707f6c3612e31b0eba9617b283b598b1fcb..83bdc892fdcdf61e3f33608234a248c07592fa16 100644 (file)
--- a/ChangeLog.pre-2-12
+++ b/ChangeLog.pre-2-12
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+       * Makefile.am : INCLUDES is the right way to add to CFLAGS, not
+       DEFS.  Also there are bugs with '+=' in makefiles.
+       Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+       Removed redundant -I from INCLUDES.
+
 Wed Nov 11 18:11:24 EST 1998 Gregory McLean <gregm@comstar.net>
 
        * docs/*.sgml : Batch of new documentation that should be easier
diff --git a/ChangeLog.pre-2-2 b/ChangeLog.pre-2-2
index 8622b707f6c3612e31b0eba9617b283b598b1fcb..83bdc892fdcdf61e3f33608234a248c07592fa16 100644 (file)
--- a/ChangeLog.pre-2-2
+++ b/ChangeLog.pre-2-2
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+       * Makefile.am : INCLUDES is the right way to add to CFLAGS, not
+       DEFS.  Also there are bugs with '+=' in makefiles.
+       Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+       Removed redundant -I from INCLUDES.
+
 Wed Nov 11 18:11:24 EST 1998 Gregory McLean <gregm@comstar.net>
 
        * docs/*.sgml : Batch of new documentation that should be easier
diff --git a/ChangeLog.pre-2-4 b/ChangeLog.pre-2-4
index 8622b707f6c3612e31b0eba9617b283b598b1fcb..83bdc892fdcdf61e3f33608234a248c07592fa16 100644 (file)
--- a/ChangeLog.pre-2-4
+++ b/ChangeLog.pre-2-4
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+       * Makefile.am : INCLUDES is the right way to add to CFLAGS, not
+       DEFS.  Also there are bugs with '+=' in makefiles.
+       Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+       Removed redundant -I from INCLUDES.
+
 Wed Nov 11 18:11:24 EST 1998 Gregory McLean <gregm@comstar.net>
 
        * docs/*.sgml : Batch of new documentation that should be easier
diff --git a/ChangeLog.pre-2-6 b/ChangeLog.pre-2-6
index 8622b707f6c3612e31b0eba9617b283b598b1fcb..83bdc892fdcdf61e3f33608234a248c07592fa16 100644 (file)
--- a/ChangeLog.pre-2-6
+++ b/ChangeLog.pre-2-6
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+       * Makefile.am : INCLUDES is the right way to add to CFLAGS, not
+       DEFS.  Also there are bugs with '+=' in makefiles.
+       Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+       Removed redundant -I from INCLUDES.
+
 Wed Nov 11 18:11:24 EST 1998 Gregory McLean <gregm@comstar.net>
 
        * docs/*.sgml : Batch of new documentation that should be easier
diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8
index 8622b707f6c3612e31b0eba9617b283b598b1fcb..83bdc892fdcdf61e3f33608234a248c07592fa16 100644 (file)
--- a/ChangeLog.pre-2-8
+++ b/ChangeLog.pre-2-8
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+       * Makefile.am : INCLUDES is the right way to add to CFLAGS, not
+       DEFS.  Also there are bugs with '+=' in makefiles.
+       Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+       Removed redundant -I from INCLUDES.
+
 Wed Nov 11 18:11:24 EST 1998 Gregory McLean <gregm@comstar.net>
 
        * docs/*.sgml : Batch of new documentation that should be easier
diff --git a/Makefile.am b/Makefile.am
index 75e448345d3c25a5920038ed42356ca1f90369c0..bf03bcc65e2c676b9cc2c7ece9c70f829dc13c45 100644 (file)
--- a/Makefile.am
+++ b/Makefile.am
@@ -12,7 +12,7 @@ bin_SCRIPTS=glib-config
 BUILT_SOURCES=glib-config
 glib-config: glib-config.in
 
-DEFS += -DG_LOG_DOMAIN=g_log_domain_glib
+INCLUDES = -DG_LOG_DOMAIN=g_log_domain_glib
 
 EXTRA_DIST = \
        glib.m4         \
@@ -63,7 +63,6 @@ stamp-gc-h: config.status
 
 libglib_la_LDFLAGS = \
        -version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE) -release $(LT_RELEASE)
-INCLUDES = -I$(top_srcdir)
 
 noinst_PROGRAMS = testglib
 testglib_LDADD = libglib.la
diff --git a/glib/Makefile.am b/glib/Makefile.am
index 75e448345d3c25a5920038ed42356ca1f90369c0..bf03bcc65e2c676b9cc2c7ece9c70f829dc13c45 100644 (file)
--- a/glib/Makefile.am
+++ b/glib/Makefile.am
@@ -12,7 +12,7 @@ bin_SCRIPTS=glib-config
 BUILT_SOURCES=glib-config
 glib-config: glib-config.in
 
-DEFS += -DG_LOG_DOMAIN=g_log_domain_glib
+INCLUDES = -DG_LOG_DOMAIN=g_log_domain_glib
 
 EXTRA_DIST = \
        glib.m4         \
@@ -63,7 +63,6 @@ stamp-gc-h: config.status
 
 libglib_la_LDFLAGS = \
        -version-info $(LT_CURRENT):$(LT_REVISION):$(LT_AGE) -release $(LT_RELEASE)
-INCLUDES = -I$(top_srcdir)
 
 noinst_PROGRAMS = testglib
 testglib_LDADD = libglib.la
diff --git a/gmodule/ChangeLog b/gmodule/ChangeLog
index ad6c76a448719ad4a5fda1e3046b3c6f594af3aa..2ee2167088f561c955d95dae64e6ec0a22095b45 100644 (file)
--- a/gmodule/ChangeLog
+++ b/gmodule/ChangeLog
@@ -1,3 +1,10 @@
+Wed Nov 11 23:23:22 EST 1998 Jeff Garzik <jgarzik@pobox.com>
+
+        * Makefile.am : INCLUDES is the right way to add to CFLAGS, not 
+        DEFS.  Also there are bugs with '+=' in makefiles.
+        Got rid of DEFS line by moving G_LOG_DOMAIN setting into INCLUDES.
+        Removed redundant -I from INCLUDES.
+
 Tue Oct 27 04:00:11 1998  Tim Janik  <timj@gtk.org>
 
        * testgmodule.c (main): changed the #ifdef WIN32 test to NATIVE_WIN32,
diff --git a/gmodule/Makefile.am b/gmodule/Makefile.am
index 818f0a01f23ebd0164eb129e50e494f96c1bd949..4d182ed68d193379aaaea41fa8252bb2fb5dc320 100644 (file)
--- a/gmodule/Makefile.am
+++ b/gmodule/Makefile.am
@@ -1,7 +1,6 @@
 ## Process this file with automake to produce Makefile.in
 
-INCLUDES = -I$(top_srcdir) -I$(top_srcdir)/gmodule
-DEFS += -DG_LOG_DOMAIN=g_log_domain_gmodule
+INCLUDES = -I$(top_srcdir)/gmodule -DG_LOG_DOMAIN=g_log_domain_gmodule
 
 EXTRA_DIST =   \
                gmoduleconf.h.in \
Improved data structures for GLib. Specifically, a persistent version of the GTree.