[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 10:06:38 +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:
> 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?
Thanks,
Stefano