This is the mail archive of the cygwin-apps@sources.redhat.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Texinfo-4.0


Carlos Puig wrote:
> 
> Note: One small patch is needed to make texinfo-4.0 under cygwin.  On line 106
> of the generated makeinfo/Makefile, add -DWIN32 as follows:
> 
>     DEFS = -DHAVE_CONFIG_H -I. -I$(srcdir) -I..
> 
> to
> 
>     DEFS = -DWIN32 -DHAVE_CONFIG_H -I. -I$(srcdir) -I..

AArrrrggghhh!!!  No.  Don't rely on -DWIN32 -- it often turns on too
many windows specific hacks, and leads to huge troubles later on. 
Cygwin is not windows.  What you should do, is inspect every .h and .c
file, looking for '#ifdef WIN32'.  If the block of code 'turned on' by
the WIN32 flag is truly necessary on cygwin, then change the '#ifdef
WIN32' to 

  #if defined(WIN32) || defined(__CYGWIN__)

If the block of code 'turned on' by the WIN32 flag is NOT needed on
cygwin, then change the '#ifdef WIN32'
 
  #if defined(WIN32) && !defined(__CYGWIN__)

--Chuck

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]