Re: [PATCH] {msvc} Regenerate with autoconf 2.67

[Stefano Lattarini]

On Wednesday 15 September 2010, Eric Blake wrote:
> [dropping all but bug-autoconf]
> 
> On 09/15/2010 09:15 AM, Eric Blake wrote:
> > [adding bug-autoconf]
> > 
> > On 09/15/2010 09:13 AM, Stefano Lattarini wrote:
> >> Ouch! On my Debian box, I had to build my own autoconf 2.67 from
> >> sources to use with automake, since at least this chunk of the
> >> debian patch:
> >> 
> >> --- autoconf-2.67.orig/lib/m4sugar/m4sh.m4
> >> +++ autoconf-2.67/lib/m4sugar/m4sh.m4
> >> @@ -843,7 +843,7 @@
> >> [m4_append_uniq([_AS_CLEANUP],
> >> [m4_divert_text([M4SH-INIT-FN], [_AS_ERROR_PREPARE[]])])],
> >> [as_fn_error m4_default([$2], [$?])
> >> "_AS_QUOTE([$1])"m4_ifval(AS_MESSAGE_LOG_FD,
> >> - [ "$LINENO" AS_MESSAGE_LOG_FD])])
> >> + [ "$LINENO" AS_MESSAGE_LOG_FD ])])
> > 
> > How nice of Debian to not tell upstream about the issue :(
> 
> Does anyone know which Debian bug this patch is supposed to
> address?
Wild guess: might it have been this one?
 <http://lists.freedesktop.org/archives/pkg-config/2010-August/000631.html>
(which has since been fixed in the pkg-config git repo, BTW).  There should
be a reference also on the autoconf list somewhere...

> And, is there any reason why:
> 
> [ "$LINENO" AS_MESSAGE_LOG_FD[]])])
> 
> instead of inserting a spurious space wasn't chosen as a way to
> patch things?

Regards,
   Stefano