Re: [wdiff-bugs] Build problems fixed, please test for 0.5.4 release

[Santiago Vila]

Hi.

The current code does not pass my test for well-behaved makefiles:

cvs -z3 -d:pserver:address@hidden:/sources/wdiff co wdiff
cp -a wdiff wdiff.orig
cd wdiff
./configure
make
make distclean
cd ..
diff wdiff.orig wdiff

The output for the last commmand should be empty, but it's not:

Only in wdiff: autom4te.cache
diff -ru wdiff.orig/config.h.in wdiff/config.h.in
--- wdiff.orig/config.h.in      2007-06-19 23:04:21.000000000 +0200
+++ wdiff/config.h.in   2007-06-23 00:42:03.007672413 +0200
@@ -1,64 +1,21 @@
-/* config.h.in.  Generated automatically from configure.in by autoheader.  */
-
-/* Define if on AIX 3.
-   System headers sometimes define this.
-   We just want to avoid a redefinition error message.  */
-#ifndef _ALL_SOURCE
-#undef _ALL_SOURCE
-#endif
-
-/* Define if using alloca.c.  */
-#undef C_ALLOCA
-
-/* Define to empty if the keyword does not work.  */
-#undef const
-
-/* Define to one of _getb67, GETB67, getb67 for Cray-2 and Cray-YMP systems.
-   This function is required for alloca.c support on those systems.  */
-#undef CRAY_STACKSEG_END
-
-/* Define if you have alloca, as a function or macro.  */
-#undef HAVE_ALLOCA
-
-/* Define if you have <alloca.h> and it should be used (not on Ultrix).  */
-#undef HAVE_ALLOCA_H
-
-/* Define if you don't have vprintf but do have _doprnt.  */
-#undef HAVE_DOPRNT
-
-/* Define if you have the vprintf function.  */
-#undef HAVE_VPRINTF
-
-/* Define as __inline if that's what the C compiler calls it.  */
-#undef inline
-
-/* Define if on MINIX.  */
-#undef _MINIX
-
-/* Define if the system does not provide POSIX.1 features except
-   with this defined.  */
-#undef _POSIX_1_SOURCE
-
-/* Define if you need to in order for stat and other things to work.  */
-#undef _POSIX_SOURCE
-
-/* Define as the return type of signal handlers (int or void).  */
-#undef RETSIGTYPE
-
-/* Define to `unsigned' if <sys/types.h> doesn't define.  */
-#undef size_t
-
-/* If using the C implementation of alloca, define if you know the
-   direction of stack growth for your system; otherwise it will be
-   automatically deduced at run-time.
- STACK_DIRECTION > 0 => grows toward higher addresses
- STACK_DIRECTION < 0 => grows toward lower addresses
- STACK_DIRECTION = 0 => direction of growth unknown
- */
-#undef STACK_DIRECTION
-
-/* Define if you have the ANSI C header files.  */
-#undef STDC_HEADERS
+/* config.h.in.  Generated from configure.in by autoheader.  */
+/* Special definitions for wdiff, processed by autoheader.
+   Copyright (C) 1994, 1997, 1998, 1999 Free Software Foundation, Inc.
+   Francois Pinard <address@hidden>, 1994.
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2, or (at your option)
+   any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software Foundation,
+   Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.  */
 
 /* Define to the path of the `diff' utility.  */
 #undef DIFF_PROGRAM
@@ -75,6 +32,9 @@
 /* Define if your locale.h file contains LC_MESSAGES.  */
 #undef HAVE_LC_MESSAGES
 
+/* Define to 1 if stpcpy function is available.  */
+#undef HAVE_STPCPY
+
 /* Define to 1 if tputs function is available.  */
 #undef HAVE_TPUTS
 
@@ -84,6 +44,15 @@
 /* Define to the path of the `less' or `more' utility.  */
 #undef PAGER_PROGRAM
 
+/* Define to the name of the distribution.  */
+#undef PACKAGE
+
+/* Define to 1 if ANSI function prototypes are usable.  */
+#undef PROTOTYPES
+
+/* Define to the version of the distribution.  */
+#undef VERSION
+
 /* Define to 1 for better use of the debugging malloc library.  See
    site ftp.antaire.com in antaire/src, file dmalloc/dmalloc.tar.gz.  */
 #undef WITH_DMALLOC
@@ -94,78 +63,177 @@
 /* Define to rpl_realloc if the replacement function should be used.  */
 #undef realloc
 
-/* Define if you have the DONE_WORKING_MALLOC_CHECK function.  */
+/* Define to one of `_getb67', `GETB67', `getb67' for Cray-2 and Cray-YMP
+   systems. This function is required for `alloca.c' support on those systems.
+   */
+#undef CRAY_STACKSEG_END
+
+/* Define to 1 if using `alloca.c'. */
+#undef C_ALLOCA
+
+/* Define to 1 if you have `alloca', as a function or macro. */
+#undef HAVE_ALLOCA
+
+/* Define to 1 if you have <alloca.h> and it should be used (not on Ultrix).
+   */
+#undef HAVE_ALLOCA_H
+
+/* Define to 1 if you have the <curses.h> header file. */
+#undef HAVE_CURSES_H
+
+/* Define to 1 if you have the `dcgettext' function. */
+#undef HAVE_DCGETTEXT
+
+/* Define to 1 if you have the `DONE_WORKING_MALLOC_CHECK' function. */
 #undef HAVE_DONE_WORKING_MALLOC_CHECK
 
-/* Define if you have the DONE_WORKING_REALLOC_CHECK function.  */
+/* Define to 1 if you have the `DONE_WORKING_REALLOC_CHECK' function. */
 #undef HAVE_DONE_WORKING_REALLOC_CHECK
 
-/* Define if you have the dcgettext function.  */
-#undef HAVE_DCGETTEXT
+/* Define to 1 if you don't have `vprintf' but do have `_doprnt.' */
+#undef HAVE_DOPRNT
 
-/* Define if you have the gettext function.  */
+/* Define to 1 if you have the `gettext' function. */
 #undef HAVE_GETTEXT
 
-/* Define if you have the setlocale function.  */
-#undef HAVE_SETLOCALE
+/* Define to 1 if you have the <inttypes.h> header file. */
+#undef HAVE_INTTYPES_H
 
-/* Define if you have the stpcpy function.  */
-#undef HAVE_STPCPY
+/* Define to 1 if you have the `curses' library (-lcurses). */
+#undef HAVE_LIBCURSES
 
-/* Define if you have the strerror function.  */
-#undef HAVE_STRERROR
+/* Define to 1 if you have the <libintl.h> header file. */
+#undef HAVE_LIBINTL_H
 
-/* Define if you have the strstr function.  */
-#undef HAVE_STRSTR
+/* Define to 1 if you have the `malloc' library (-lmalloc). */
+#undef HAVE_LIBMALLOC
 
-/* Define if you have the <curses.h> header file.  */
-#undef HAVE_CURSES_H
+/* Define to 1 if you have the `ncurses' library (-lncurses). */
+#undef HAVE_LIBNCURSES
 
-/* Define if you have the <libintl.h> header file.  */
-#undef HAVE_LIBINTL_H
+/* Define to 1 if you have the `termcap' library (-ltermcap). */
+#undef HAVE_LIBTERMCAP
+
+/* Define to 1 if you have the `termlib' library (-ltermlib). */
+#undef HAVE_LIBTERMLIB
 
-/* Define if you have the <locale.h> header file.  */
+/* Define to 1 if you have the <locale.h> header file. */
 #undef HAVE_LOCALE_H
 
-/* Define if you have the <ncurses.h> header file.  */
+/* Define to 1 if you have the <memory.h> header file. */
+#undef HAVE_MEMORY_H
+
+/* Define to 1 if you have the <ncurses.h> header file. */
 #undef HAVE_NCURSES_H
 
-/* Define if you have the <string.h> header file.  */
+/* Define to 1 if you have the `setlocale' function. */
+#undef HAVE_SETLOCALE
+
+/* Define to 1 if you have the <stdint.h> header file. */
+#undef HAVE_STDINT_H
+
+/* Define to 1 if you have the <stdlib.h> header file. */
+#undef HAVE_STDLIB_H
+
+/* Define to 1 if you have the `stpcpy' function. */
+#undef HAVE_STPCPY
+
+/* Define to 1 if you have the `strerror' function. */
+#undef HAVE_STRERROR
+
+/* Define to 1 if you have the <strings.h> header file. */
+#undef HAVE_STRINGS_H
+
+/* Define to 1 if you have the <string.h> header file. */
 #undef HAVE_STRING_H
 
-/* Define if you have the <termcap.h> header file.  */
+/* Define to 1 if you have the `strstr' function. */
+#undef HAVE_STRSTR
+
+/* Define to 1 if you have the <sys/stat.h> header file. */
+#undef HAVE_SYS_STAT_H
+
+/* Define to 1 if you have the <sys/types.h> header file. */
+#undef HAVE_SYS_TYPES_H
+
+/* Define to 1 if you have the <termcap.h> header file. */
 #undef HAVE_TERMCAP_H
 
-/* Define if you have the <termlib.h> header file.  */
+/* Define to 1 if you have the <termlib.h> header file. */
 #undef HAVE_TERMLIB_H
 
-/* Define if you have the <unistd.h> header file.  */
+/* Define to 1 if you have the <unistd.h> header file. */
 #undef HAVE_UNISTD_H
 
-/* Define if you have the curses library (-lcurses).  */
-#undef HAVE_LIBCURSES
+/* Define to 1 if you have the `vprintf' function. */
+#undef HAVE_VPRINTF
 
-/* Define if you have the malloc library (-lmalloc).  */
-#undef HAVE_LIBMALLOC
+/* Name of package */
+#undef PACKAGE
 
-/* Define if you have the ncurses library (-lncurses).  */
-#undef HAVE_LIBNCURSES
+/* Define to the address where bug reports for this package should be sent. */
+#undef PACKAGE_BUGREPORT
 
-/* Define if you have the termcap library (-ltermcap).  */
-#undef HAVE_LIBTERMCAP
+/* Define to the full name of this package. */
+#undef PACKAGE_NAME
 
-/* Define if you have the termlib library (-ltermlib).  */
-#undef HAVE_LIBTERMLIB
+/* Define to the full name and version of this package. */
+#undef PACKAGE_STRING
 
-/* Name of package */
-#undef PACKAGE
+/* Define to the one symbol short name of this package. */
+#undef PACKAGE_TARNAME
 
-/* Version number of package */
-#undef VERSION
+/* Define to the version of this package. */
+#undef PACKAGE_VERSION
 
 /* Define if compiler has function prototypes */
 #undef PROTOTYPES
 
+/* Define as the return type of signal handlers (`int' or `void'). */
+#undef RETSIGTYPE
+
+/* If using the C implementation of alloca, define if you know the
+   direction of stack growth for your system; otherwise it will be
+   automatically deduced at runtime.
+       STACK_DIRECTION > 0 => grows toward higher addresses
+       STACK_DIRECTION < 0 => grows toward lower addresses
+       STACK_DIRECTION = 0 => direction of growth unknown */
+#undef STACK_DIRECTION
+
+/* Define to 1 if you have the ANSI C header files. */
+#undef STDC_HEADERS
+
+/* Version number of package */
+#undef VERSION
+
 /* Define if using the dmalloc debugging malloc package */
 #undef WITH_DMALLOC
 
+/* Define to 1 if on AIX 3.
+   System headers sometimes define this.
+   We just want to avoid a redefinition error message.  */
+#ifndef _ALL_SOURCE
+# undef _ALL_SOURCE
+#endif
+
+/* Define to 1 if on MINIX. */
+#undef _MINIX
+
+/* Define to 2 if the system does not provide POSIX.1 features except with
+   this defined. */
+#undef _POSIX_1_SOURCE
+
+/* Define to 1 if you need to in order for `stat' and other things to work. */
+#undef _POSIX_SOURCE
+
+/* Define to empty if `const' does not conform to ANSI C. */
+#undef const
+
+/* Define to `__inline__' or `__inline' if that's what the C compiler
+   calls it, or to nothing if 'inline' is not supported under any name.  */
+#ifndef __cplusplus
+#undef inline
+#endif
+
+/* Define to `unsigned int' if <sys/types.h> does not define. */
+#undef size_t
Only in wdiff: config.h.in~
Only in wdiff/src: mdiff.1
Only in wdiff/src: unify.1
Only in wdiff/src: wdiff.1
Only in wdiff: stamp-pot

I believe it's the GNU standards who say this should be empty, but I'm not
completely sure. I would call it a highly desirable thing anyway.

Another thing which I noticed: It would be a good idea to send the .pot
file for version 0.5.4 to the Translation Project Robot, and wait a few
days, so that translators have some opportunity to update the translations.

Also:

./configure; make dist

says:

make: *** No rule to make target `README-alpha', needed by `distdir'.  Stop.

which I think it should not happen.

BTW: Would be possible to generate a protoype tarball for 0.5.4pre?
I would like to be sure that the final tarball will not include CVS
directories.

Thanks.