[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: * tests/amhello-binpkg.test: Add missing $EXEEXT usage.
From: |
Stefano Lattarini |
Subject: |
Re: * tests/amhello-binpkg.test: Add missing $EXEEXT usage. |
Date: |
Mon, 5 Sep 2011 11:37:20 +0200 |
User-agent: |
KMail/1.13.7 (Linux/2.6.30-2-686; KDE/4.6.5; i686; ; ) |
On Monday 05 September 2011, Peter Rosin wrote:
> Den 2011-09-05 10:06 skrev Stefano Lattarini:
> > On Monday 05 September 2011, Peter Rosin wrote:
> >> Hi!
> >>
> > Hi Peter, thanks for the patch.
> >
> >> This fixes a fail on Cygwin (and others I suppose).
> >>
> >> I'm aware that the lax non-gnu-tar branch adds even more laxness
> >> since $EXEEXT normally contains a dot for the oddball cases when
> >> it's non-empty, but that's so minor that I didn't bother to code
> >> around it... Ok for maint?
> >>
> > I only have a minor nit: I'd prefer the extraction of `EXEEXT' from
> > Makefile to be done by something like this (avoiding use of "eval"):
> >
> > EXEEXT=`sed -n -e 's/^EXEEXT *= *//p'`
> >
> > And BTW, this could also be improved to allow escaping of literal
> > dots, as in:
> >
> > EXEEXT=`sed -n '/^EXEEXT *=/{ s/^EXEEXT *= *//; s/\./\\./g; p; }'`
> >
> > WDYT?
>
> I'm ok with your first alternative, but the second is undefined
> according to posix (at least 'Limitations of usual tools' in
> Autoconf states so; you can't have semicolon after a '{' verb)
>
Ah right; thank you for digging that up.
> and, even worse, it breaks for the gnu tar branch.
>
Ouch! Silly me for not thinking about that.
> So, unless someone else chimes in I'm pushing with this
>
> EXEEXT=`sed -n -e 's/^EXEEXT *= *//p' < ../Makefile`
>
> sometime later today.
>
Fine by me.
Thanks,
Stefano