[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gnulib-bug
Subject:    Re: gnulib-tool: restore the ability to create a big testdir
From:       Ralf Wildenhues <Ralf.Wildenhues () gmx ! de>
Date:       2006-10-24 20:40:47
Message-ID: 20061024204047.GH5692 () iam ! uni-bonn ! de
[Download RAW message or body]

* Ralf Wildenhues wrote on Sat, Oct 14, 2006 at 12:24:52PM CEST:
> * Paul Eggert wrote on Fri, Oct 13, 2006 at 12:01:16AM CEST:
> 
> > If there are actual projects that don't use AC_CONFIG_HEADERS then of
> > course we have to be cautious about assuming config-h.  But if not,
> > then why wouldn't the following patch work for lib/argz.c?

> Libtool folks, do you agree to this change, or would you rather have the
> gnulib argz module diverge from the Libtool copy (please let's not) or
> something else?

No comments, so I applied this to CVS Libtool, and pulled the change
over to gnulib, as below.

Cheers,
Ralf

> 2006-10-14  Paul Eggert  <eggert@cs.ucla.edu>,
> 	    Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
> 
> 	* doc/libtool.texi (Distributing libltdl): In recursive and
> 	nonrecursive mode, a config header is now required.
> 	* libltdl/argz.c: Remove check for HAVE_CONFIG_H, to conform
> 	to gnulib's policy of including config.h unconditionally.

2006-10-24  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>

	Sync from Libtool:

	2006-10-24  Paul Eggert  <eggert@cs.ucla.edu>

	* lib/argz.c: Remove check for HAVE_CONFIG_H, to conform
	to gnulib's policy of including config.h unconditionally.

Index: lib/argz.c
===================================================================
RCS file: /cvsroot/gnulib/gnulib/lib/argz.c,v
retrieving revision 1.7
diff -u -r1.7 argz.c
--- lib/argz.c	29 Aug 2006 06:15:20 -0000	1.7
+++ lib/argz.c	24 Oct 2006 20:38:41 -0000
@@ -19,12 +19,10 @@
    with this program; if not, write to the Free Software Foundation,
    Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. */
 
-#ifdef HAVE_CONFIG_H
-#  if defined(LTDL) && defined LT_CONFIG_H
-#    include LT_CONFIG_H
-#  else
-#    include <config.h>
-#  endif
+#if defined(LTDL) && defined LT_CONFIG_H
+#  include LT_CONFIG_H
+#else
+#  include <config.h>
 #endif
 
 #include <argz.h>



[prev in list] [next in list] [prev in thread] [next in thread]