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

[Bruce Korb]

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"
> 
>