[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: address@hidden: [PATCH] sharutils-4.3.80: configure.ac tweak to buil
From: |
Aharon Robbins |
Subject: |
Re: address@hidden: [PATCH] sharutils-4.3.80: configure.ac tweak to build uuencode/uudecode OOTB on Cygwin] |
Date: |
Wed, 08 Jun 2005 09:06:21 +0300 |
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
- Re: address@hidden: [PATCH] sharutils-4.3.80: configure.ac tweak to build uuencode/uudecode OOTB on Cygwin],
Aharon Robbins <=