Re: [Bug-gnulib] gnulib module for uintmax_t? (and best practice questions)

[James Youngman]

On Sat, Oct 23, 2004 at 01:16:18AM +0100, James Youngman wrote:

> $ aclocal -I . -I gnulib/m4 ; autoheader  ; autoconf  ; automake 
> --add-missing --copy
> gnulib/Makefile.am:19: required file `gnulib/waitpid.c' not found
> gnulib/Makefile.am:19: required file `gnulib/strspn.c' not found
> gnulib/Makefile.am:18: automatically discovered file `strftime.c' should not 
> be explicitly mentioned

I'm not sure why this problem should be diagnosed in this way, but it
turns out that I had specified them in AC_REPLACE_FUNCS (I had to add
lots of debug statements to the automake script to determine this,
though).

AC_FUNC_STRFTIME
AC_REPLACE_FUNCS(memcmp memset stpcpy strdup strstr strtol strtoul strftime 
strspn waitpid)

Removing "strftime strspn waitpid" fixes the problem.  I've added
waitpid back into a LIBOBJS-like facility specific to findutils (it
includes implementations of functions missing from both the local
system and gnulib, but otherwise serves a similar purpose).

During the course of this I've updated a few other files from automake
too (like install-sh).  I now find that to get "make distcheck" to
pass for findutils, I have to apply this patch to gnulib :-

Index: modules/regex
===================================================================
RCS file: /cvsroot/gnulib/gnulib/modules/regex,v
retrieving revision 1.6
diff -u -r1.6 regex
--- modules/regex       4 Oct 2004 20:17:39 -0000       1.6
+++ modules/regex       24 Oct 2004 22:07:33 -0000
@@ -15,7 +15,7 @@
 gl_REGEX

 Makefile.am:
-lib_SOURCES += regex.h
+lib_SOURCES += regex.h regex.c

 Include:
 "regex.h"



Without this patch, "make distcheck" on the findutils code fails
during "make update-po" (because regex.c is missing from the
distributed files).  Mind you, I think the included libintl code that
exists in the findutils CVS source base is a bit long in the tooth
now.  It's probably time to update that code from the real libintl,
too.

Regards,
James Youngman.