Re: Weird failure with autoconf 2.69c in gmp

2020-11-11 Thread Zack Weinberg
On Wed, Nov 11, 2020 at 8:03 AM Ross Burton wrote: > FWIW, I just upgraded to 2.69d and hit the same problem: > > m4:configure.ac:40: Warning: excess arguments to builtin `m4_define' ignored > > Line 40 is: > > AC_INIT(GNU MP, GMP_VERSION, [gmp-b...@gmplib.org, see > https://gmplib.org/manual/Repo

Re: Weird failure with autoconf 2.69c in gmp

2020-11-11 Thread Ross Burton
FWIW, I just upgraded to 2.69d and hit the same problem: m4:configure.ac:40: Warning: excess arguments to builtin `m4_define' ignored Line 40 is: AC_INIT(GNU MP, GMP_VERSION, [gmp-b...@gmplib.org, see https://gmplib.org/manual/Reporting-Bugs.html], gmp) Cheers, Ross On Tue, 20 Oct 2020 at 14:0

Re: Weird failure with autoconf 2.69c in gmp

2020-10-20 Thread Zack Weinberg
On Wed, Oct 14, 2020 at 5:04 PM Zack Weinberg wrote: > > If you could both please test this patch? I would also appreciate a > second pair of eyes on it -- even with an expanded "AC_INIT with > unusual strings" test and careful inspection of all the places where > the AC_PACKAGE_* macros are used

Re: Weird failure with autoconf 2.69c in gmp

2020-10-15 Thread Ross Burton
Thanks Zack. I can confirm that the patch fixes both of the failures I've seen in AC_INIT. Ross On Wed, 14 Oct 2020 at 23:05, Zack Weinberg wrote: > > On Wed, Oct 14, 2020 at 5:47 PM Ross Burton wrote: > > That patch got totally mangled by mail, can you push it somewhere or attach > > it? > >

Re: Weird failure with autoconf 2.69c in gmp

2020-10-14 Thread Zack Weinberg
On Wed, Oct 14, 2020 at 5:47 PM Ross Burton wrote: > That patch got totally mangled by mail, can you push it somewhere or attach > it? Sorry about that. You can get it from https://git.savannah.gnu.org/git/autoconf.git in the 'zack/ac-init-quoting' branch. zw

Re: Weird failure with autoconf 2.69c in gmp

2020-10-14 Thread Zack Weinberg
Man, M4 quotation is a headache and a half. If you could both please test this patch? I would also appreciate a second pair of eyes on it -- even with an expanded "AC_INIT with unusual strings" test and careful inspection of all the places where the AC_PACKAGE_* macros are used, I'm not 100% sure

Re: Weird failure with autoconf 2.69c in gmp

2020-10-14 Thread Nick Bowler
On 2020-10-14, Ross Burton wrote: > No it's the # in the URL. Simply removing #libidn2 fixes this problem. > > Presumably some quoting problem which just needs more precision []? [...] > On Wed, 14 Oct 2020 at 09:26, Ross Burton wrote: >> >> Similar in libidn2: >> >> | m4:configure.ac:16: Warning

Re: Weird failure with autoconf 2.69c in gmp

2020-10-14 Thread Ross Burton
No it's the # in the URL. Simply removing #libidn2 fixes this problem. Presumably some quoting problem which just needs more precision []? Ross On Wed, 14 Oct 2020 at 09:26, Ross Burton wrote: > > Similar in libidn2: > > | m4:configure.ac:16: Warning: excess arguments to builtin `m4_define' ig

Re: Weird failure with autoconf 2.69c in gmp

2020-10-14 Thread Ross Burton
Similar in libidn2: | m4:configure.ac:16: Warning: excess arguments to builtin `m4_define' ignored 16 AC_INIT([libidn2], [2.3.0], [help-lib...@gnu.org],, 17 [https://www.gnu.org/software/libidn/#libidn2]) Not handling the tarname being unset? Ross On Tue, 13 Oct 2020 at 20:51, Ross

Re: Weird failure with autoconf 2.69c in gmp

2020-10-13 Thread Ross Burton
Yep, that fixes it. Ross On Tue, 13 Oct 2020 at 20:26, Nick Bowler wrote: > > On 13/10/2020, Ross Burton wrote: > > Hi, > > > > Using autoconf 2.69c (upgrading from 2.69b meant we could drop two > > patches, so that's good news!) to build gmp fails in a rather > > mysterious way: > [...] > > |

Re: Weird failure with autoconf 2.69c in gmp

2020-10-13 Thread Nick Bowler
On 13/10/2020, Ross Burton wrote: > Hi, > > Using autoconf 2.69c (upgrading from 2.69b meant we could drop two > patches, so that's good news!) to build gmp fails in a rather > mysterious way: [...] > | m4:configure.ac:40: Warning: excess arguments to builtin `m4_define' > ignored > | autom4te: er

Re: Weird failure with autoconf 2.69c in gmp

2020-10-13 Thread Zack Weinberg
On Tue, Oct 13, 2020 at 3:10 PM Ross Burton wrote: > | m4:configure.ac:40: Warning: excess arguments to builtin `m4_define' ignored > | autom4te: error: m4 failed with exit status: 1 > | aclocal: error: echo failed with exit status: 1 > | autoreconf: error: aclocal failed with exit status: 1 > > L

Weird failure with autoconf 2.69c in gmp

2020-10-13 Thread Ross Burton
Hi, Using autoconf 2.69c (upgrading from 2.69b meant we could drop two patches, so that's good news!) to build gmp fails in a rather mysterious way: | autoreconf: export WARNINGS=cross | autoreconf: Entering directory '.' | autoreconf: configure.ac: not using Gettext | autoreconf: running: acloca