Re: Cygwin Build Fix Candidates

[Greg Hunt]

Ben,
I tried it last night and that fixes the problem with libiconv. 

The problem with p2c{EXEEXT} in the generic rule for building c files 
from p files is still present.  Deleting the {EXEEXT} from that line in 
the generated Makefile allows the build to proceed. 

Greg

Ben Pfaff wrote:
> Greg Hunt <address@hidden> writes:
>
>   
> > The missing libiconv reference under Cygwin - I could add an 
> > AC_CHECK_LIB to configure.ac, but 
> > http://www.gnu.org/software/gettext/manual/html_mono/gettext.html#SEC188 
> > discusses some variables that do not appear in the Makefile or other 
> > sources that should deal properly with the various permutations of 
> > iconv.  Unfortunately my inpexperience with automake and the like is 
> > showing at this point and I'll have to leave it to you to work out what 
> > is best.
> >     
>
> I think that the current CVS version fixes this particular
> problem, which was not Cygwin-specific, with the following
> change:
>
> Index: automake.mk
> ===================================================================
> RCS file: /cvsroot/pspp/pspp/src/ui/terminal/automake.mk,v
> retrieving revision 1.8
> retrieving revision 1.9
> diff -u -p -u -p -r1.8 -r1.9
> --- automake.mk 5 May 2006 04:53:13 -0000       1.8
> +++ automake.mk 27 Jun 2006 15:23:30 -0000      1.9
> @@ -37,5 +37,6 @@ src_ui_terminal_pspp_LDADD =                                  
> \
>         $(top_builddir)/src/data/libdata.a \
>         $(top_builddir)/src/libpspp/libpspp.a \
>         $(top_builddir)/gl/libgl.a      \
> +       $(LIBICONV) \
>         @LIBINTL@ @LIBREADLINE@
>  
>