Hi Stepan, * Stepan Kasal wrote on Mon, Jun 27, 2005 at 12:34:12PM CEST: > On Wed, Jun 22, 2005 at 06:45:35PM +0200, Ralf Wildenhues wrote: > *snip nice explanations*
> But: > There are two cases when the second argument to m4_default is not > double quoted; I think these should be fixed. > > So I propose the following patch. I agree, and have committed it to HEAD and branch-2-0. Thanks! Ralf 2005-06-27 Stepan Kasal <[EMAIL PROTECTED]> * m4/ltsugar.m4 (lt_combine, _lt_combine): Fix default separator. > Index: m4/ltsugar.m4 > =================================================================== > RCS file: /cvsroot/libtool/libtool/m4/ltsugar.m4,v > retrieving revision 1.5 > diff -u -r1.5 ltsugar.m4 > --- m4/ltsugar.m4 17 Jun 2005 12:42:34 -0000 1.5 > +++ m4/ltsugar.m4 27 Jun 2005 10:23:09 -0000 > @@ -52,12 +52,12 @@ > # has the form PREFIXmINFIXSUFFIXn. > m4_define([lt_combine], > [m4_if([$2], [], [], > - [lt_join(m4_quote(m4_default([$1], [, ])), > + [lt_join(m4_quote(m4_default([$1], [[, ]])), > _$0([$1], lt_car($2)[$3], m4_shiftn(3, $@)), > $0([$1], lt_cdr($2), m4_shiftn(2, $@)))])]) > m4_define([_lt_combine], > [m4_if([$3], [], [], > - [lt_join(m4_quote(m4_default([$1], [, ])), > + [lt_join(m4_quote(m4_default([$1], [[, ]])), > [$2$3], > $0([$1], [$2], m4_shiftn(3, $@)))])[]dnl > ]) > >