libtool-patches
[Top][All Lists]
Advanced

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

Re: FYI: libtool--devo--1.0--patch-114


From: Noah Misch
Subject: Re: FYI: libtool--devo--1.0--patch-114
Date: Thu, 5 Aug 2004 14:45:42 -0700
User-agent: Mutt/1.5.6i

On Thu, Aug 05, 2004 at 04:33:55PM +0100, Gary V. Vaughan wrote:

>       * bootstrap: Run the bootstrap rules from a file named Makefile,
>       copied from Makefile.am for fresh checkouts.
> 
> --- orig/bootstrap
> +++ mod/bootstrap
> @@ -54,8 +54,10 @@
>  # Building distributed files from configure is bad for automake, so we
>  # generate them here, and have Makefile rules to keep them up to date:
>  rm -f ./config/ltmain.sh ./m4/ltversion.m4
> +test -f Makefile || { makefile=Makefile; cp Makefile.am $makefile; }
>  make -f Makefile.am ./config/ltmain.sh ./m4/ltversion.m4 \
>      top_srcdir=. PACKAGE="$2" VERSION="$3"
> +test -z "$makefile" || rm -f "$makefile"

Is this not a no-op since the intervening line still has ``-f Makefile.am''?

> --- orig/m4/libtool.m4
> +++ mod/m4/libtool.m4
> @@ -1545,8 +1545,8 @@
>           striplib="$STRIP -x"
>           AC_MSG_RESULT([yes])
>         else
> -  AC_MSG_RESULT([no])
> -fi
> +         AC_MSG_RESULT([no])
> +       fi

(A trivial cleanup that does not need a ChangeLog mention, I take it?)




reply via email to

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