Re: address@hidden: [PATCH] sharutils-4.3.80: configure.ac tweak to build uuencode/uudecode OOTB on Cygwin]

[Aharon Robbins]

For what it's worth, the $(...) syntax was the only one that was documented
in the original V7 Make documentation, and portability weenies like myself
thus prefer.  Practically speaking, it probably doesn't make a difference,
since ${...} seems to work everywhere too.

Arnold

In article <address@hidden> you write:
>Hi Corinna,
>
>Thank you for the patch.  I have applied it, substituting
>'{' for '(' and '}' for ')'.  Next release (not too far away,
>since the FSF address needs to change and there's a better
>libtool about now....).
>
>Regards, Bruce
>
>On Monday 30 May 2005 09:18 am, Karl Berry wrote:
>> FYI, SIAK ...
>> 
>> Date: Mon, 30 May 2005 13:49:56 +0200
>> From: Corinna Vinschen <address@hidden>
>> To: address@hidden
>> Subject: [PATCH] sharutils-4.3.80: configure.ac tweak to build
>>      uuencode/uudecode OOTB on Cygwin
>> 
>> Hi,
>> 
>> the below patch adds a minor tweak to configure.ac relative to the latest
>> sharutils 4.3.80.  The problem is that the configure script misses to add
>> the $(EXEEXT) suffix to uuencode and uudecode when adding them to Makefile's
>> bin_PROGRAMS.  As a result, the build dependencies are incorrect when
>> building uuencode and uudecode on Cygwin and the build fails.  The below
>> patch fixes this and allows to build the whole package on Cygwin OOTB.
>> 
>> 
>> Thanks for considering,
>> Corinna
>> 
>> 
>>      * configure.ac: Add EXEEXT suffix to uuencode/uudecode.
>> 
>> --- configure.ac.ORIG   2005-05-30 13:45:19.816616300 +0200
>> +++ configure.ac        2005-05-30 13:21:43.093750000 +0200
>> @@ -111,7 +111,7 @@ AC_ARG_ENABLE(uucode,
>>  [  --disable-uucode        disable installation of uuencode and uudecode],
>>  [if test "$enableval" = yes; then
>>    AC_MSG_RESULT(yes)
>> -  ADD_PROG="uudecode uuencode"
>> +  ADD_PROG='uudecode$(EXEEXT) uuencode$(EXEEXT)'
>>    DIST_MAN1PAGES="uuencode.1"
>>    NODIST_MAN1PAGES="uudecode.1"
>>    MAN5PAGES="uuencode.5"
>> @@ -119,7 +119,7 @@ else
>>    AC_MSG_RESULT(no)
>>  fi], [
>>    AC_MSG_RESULT(yes)
>> -  ADD_PROG="uudecode uuencode"
>> +  ADD_PROG='uudecode$(EXEEXT) uuencode$(EXEEXT)'
>>    DIST_MAN1PAGES="uuencode.1"
>>    NODIST_MAN1PAGES="uudecode.1"
>>    MAN5PAGES="uuencode.5"
>> 
>> 
>
>


-- 
Aharon (Arnold) Robbins --- Pioneer Consulting Ltd.     arnold AT skeeve DOT com
P.O. Box 354            Home Phone: +972  8 979-0381    Fax: +1 206 350 8765
Nof Ayalon              Cell Phone: +972 50  729-7545
D.N. Shimshon 99785     ISRAEL