autoconf-patches
[Top][All Lists]
Advanced

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

Re: [PATCH] Make inter-release --version output more useful.


From: Ralf Wildenhues
Subject: Re: [PATCH] Make inter-release --version output more useful.
Date: Sat, 27 Oct 2007 20:50:15 +0200
User-agent: Mutt/1.5.13 (2006-08-11)

Hi Jim,

All fine with me except nits:

* Jim Meyering wrote on Sat, Oct 27, 2007 at 07:38:23PM CEST:
[...git-version-gen...]
> +        COREUTILS-[0-9]*) v=`echo "$v" | sed 's/^COREUTILS-//;s/_/./g'` ;;

Does that need adjustment for Autoconf?

[...]
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index c1141c6..1972d6c 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -34,7 +34,7 @@ include ../lib/freeze.mk
>  ## package.m4.  ##
>  ## ------------ ##
> 
> -$(srcdir)/package.m4: $(top_srcdir)/configure.ac
> +$(srcdir)/package.m4: $(builddir)/Makefile

Please make that `Makefile', not `$(builddir)/Makefile'.  Non-GNU make
don't all identify `file' with `./file'.

* Jim Meyering wrote on Sat, Oct 27, 2007 at 08:11:39PM CEST:
> 
> I suppose we should be using gnulib-tool to pull in
> build-aux files, including announce-gen.

Why?  Except for announce-gen, all files are installed by `automake -a'.

Cheers,
Ralf




reply via email to

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