[GRASS5] Re: [GRASSLIST:7448] Re: r.terraflow failured

[Hamish]

> > On Tue, 05 Jul 2005 11:16:47 -0400
> > Laura Toma <ltoma at bowdoin.edu> wrote:
> > > 
> > > I changed in  r.terraflow/IOStream/lib/src/Makefile
> > > 
> > > line9: CXX = g++ -Wall
> > > 
> > > to
> > > 
> > > CXX = g++ -Wall -D_FILE_OFFSET_BITS=64
> > > 
> > > Then I cleaned everything, and recompiled, and it works now for me (on
> > > a Linux machine).
> > 
> > 
> > GRASS 6's  r.terraflow/IOStream/lib/src/Makefile  mentions $(CXX) and
> > $(CPPFLAGS).
> > 
> > Does either of those actually get the needed LFS flags if GRASS is 
> > configured with --enable-largefile ?
> > 
> 
> I don't think so. Look at 
> grass61/lib/gis/Makefile
> 
> there is:
> 
> #compile if LFS Large File Support present:
> ifneq ($(USE_LARGEFILES),)
>         EXTRA_CFLAGS = -D_FILE_OFFSET_BITS=64
> endif
> 
> This must the added to the r.terraflow Makefiles.


Should that be changed to:

#compile if LFS Large File Support present:
ifneq ($(USE_LARGEFILES),)
        EXTRA_CFLAGS = -D_FILE_OFFSET_BITS=64
        EXTRA_CPPFLAGS = -D_FILE_OFFSET_BITS=64
endif


and then add EXTRA_CPPFLAGS to raster/r.terraflow/IOStream/lib/src/Makefile?

I notice raster/r.terraflow/Makefile also uses $(CXXFLAGS).


a) what's up with having both $(CPPFLAGS) and $(CXXFLAGS) ??
b) add EXTRA_C??FLAGS to all r.terraflow Makefiles or just 
    IOStream/lib/src ??



Hamish