[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: 284-gary-report-macro-files-for-aclocal.diff
From: |
Ralf Wildenhues |
Subject: |
Re: 284-gary-report-macro-files-for-aclocal.diff |
Date: |
Fri, 30 Sep 2005 14:06:42 +0200 |
User-agent: |
Mutt/1.5.11 |
Hi Gary,
* Gary V. Vaughan wrote on Fri, Sep 30, 2005 at 12:23:03PM CEST:
> Okay to commit?
Observe major nit below, then commit, please!
By the way, I believe Patrick meant a generic command line switch to
list the necessary macro files.
Cheers, and thanks,
Ralf
> from Gary V. Vaughan <address@hidden>
> * libtoolize.m4sh: Copying just libtool.m4 for hand maintained
> aclocal.m4 doesn't work. List all required files in that case.
>
> Index: libtool--devo--1.0/libtoolize.m4sh
> ===================================================================
> --- libtool--devo--1.0.orig/libtoolize.m4sh
> +++ libtool--devo--1.0/libtoolize.m4sh
> @@ -996,7 +996,13 @@ func_nonemptydir_p ()
> func_copy_some_files "$pkgmacro_files" "$aclocaldir" \
> "$m4dir" func_serial_update
> else
> - $opt_quiet || func_echo "You should add the contents of
> \`$aclocaldir/libtool.m4' to \`aclocal.m4'."
> + $opt_quiet || {
> + func_echo "You should add the contents of the following files to
> \`aclocal.m4':"
> + for need in libtool.m4 ltoptions.m4 ltversion.m4 ltsugar.m4; do
> + func_echo "\`$aclocaldir/libtool.m4'"
s,libtool\.m4,$need,
> + done
> + $seen_ltdl && func_echo "\`$aclocaldir/ltdl.m4'"
> + }
> fi
>
> # Suggest modern idioms for storing autoconf macros: