m4-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Drop ltdl configure banner (was: use build-aux)


From: Ralf Wildenhues
Subject: Drop ltdl configure banner (was: use build-aux)
Date: Thu, 8 Nov 2007 22:19:34 +0100
User-agent: Mutt/1.5.13 (2006-08-11)

Hello Eric,

* Eric Blake wrote on Tue, Nov 06, 2007 at 02:30:41PM CET:
> According to Eric Blake on 11/6/2007 6:15 AM:
> > According to Ralf Wildenhues on 11/6/2007 4:31 AM:
> >> You need to ensure build-aux/mkstamp exists at the end of the bootstrap.
> >> You could copy it from ltdl/config/.
> > 
> > Or is there a better way to make libtool share build-aux, rather than
> > creating its own subdirectory?  Perhaps using the libtoolize
> > --nonrecursive or --recursive option?
> 
> Nevermind.  I see that I missed a file; I wrote the above thinking that
> mkstamp belonged to libtool, but see now that it belongs to m4.  Applying:

But libltdl makes use of mkstamp as well, see ltdl/configure.ac:

# This is a sanity check so we can see which version is used in bug reports.
# It is assumed that we only want to see the date extension for cvs libtool
# versions (i.e. "odd" letters) and not actual alpha releases.
TIMESTAMP=
case $lt_alpha in
  [[acegikmoqsuwy]])
    TIMESTAMP=`${CONFIG_SHELL} ${ac_aux_dir}/mkstamp < ${srcdir}/ChangeLog`
    AS_BOX([Configuring AC_PACKAGE_TARNAME (Build:$TIMESTAMP) 
AC_PACKAGE_VERSION])
    echo
    ;;
esac
AC_SUBST([TIMESTAMP])

Hmm, that code is broken: lt_alpha is not set, and ltdl should just ship
with mkstamp.  OTOH, I must say that I don't like this banner in libltdl
much: we haven't displayed it in a long time, other users of libltdl
have their own ways of printing banners so this would look a bit ugly
for them, and the libltdl version used is always shown in the head of
config.log.

Any reason not to apply this patch to Libtool HEAD?

Cheers,
Ralf

2007-11-07  Ralf Wildenhues  <address@hidden>

        * libltdl/configure.ac: Drop the broken and long-disabled code
        to output a banner for the libltdl configure.  Avoids the need
        to ship mkstamp with libltdl.

Index: libltdl/configure.ac
===================================================================
RCS file: /cvsroot/libtool/libtool/libltdl/configure.ac,v
retrieving revision 1.21
diff -u -r1.21 configure.ac
--- libltdl/configure.ac        25 Mar 2007 12:12:42 -0000      1.21
+++ libltdl/configure.ac        8 Nov 2007 18:26:54 -0000
@@ -1,6 +1,6 @@
 # Process this file with autoconf to create configure. -*- autoconf -*-
 #
-#    Copyright (C) 2004, 2005 Free Software Foundation, Inc.
+#    Copyright (C) 2004, 2005, 2007 Free Software Foundation, Inc.
 #    Written by Gary V. Vaughan, 2004
 #
 #    NOTE: The canonical source of this file is maintained with the
@@ -48,30 +48,13 @@
 LT_CONFIG_LTDL_DIR([.]) # I am me!
 
 
-## ---------------------------------------- ##
-## Display a configure time version banner. ##
-## ---------------------------------------- ##
-
-# This is a sanity check so we can see which version is used in bug reports.
-# It is assumed that we only want to see the date extension for cvs libtool
-# versions (i.e. "odd" letters) and not actual alpha releases.
-TIMESTAMP=
-case $lt_alpha in
-  [[acegikmoqsuwy]])
-    TIMESTAMP=`${CONFIG_SHELL} ${ac_aux_dir}/mkstamp < ${srcdir}/ChangeLog`
-    AS_BOX([Configuring AC_PACKAGE_TARNAME (Build:$TIMESTAMP) 
AC_PACKAGE_VERSION])
-    echo
-    ;;
-esac
-AC_SUBST([TIMESTAMP])
-
-
 ## ------------------------ ##
 ## Automake Initialisation. ##
 ## ------------------------ ##
 
 AM_INIT_AUTOMAKE([gnits])
 
+
 ## ------------------------------- ##
 ## Libtool specific configuration. ##
 ## ------------------------------- ##




reply via email to

[Prev in Thread] Current Thread [Next in Thread]