[GRASS-dev] Re: [GRASS GIS] #61: vdigit makefile uses hardwired compile and link flags

[GRASS GIS]

#61: vdigit makefile uses hardwired compile and link flags
------------------------+---------------------------------------------------
  Reporter:  kyngchaos  |       Owner:  grass-dev at lists.osgeo.org
      Type:  defect     |      Status:  new                      
  Priority:  major      |   Milestone:  6.4.0                    
 Component:  Python     |     Version:  6.3.0 RCs                
Resolution:             |    Keywords:                           
------------------------+---------------------------------------------------
Comment (by kyngchaos):

 Some more notes:

 On OSX, the proper linking for a python extension is -bundle, not
 -dynamiclib.  So "-shared" should be replaced by some configured value in
 Platform.make.

 Shouldn't COMPILE_FLAGS_CXX be in CXXFLAGS, and LINK_FLAGS in LDFLAGS?

 Also, with COMPILE_FLAGS_CXX and LINK_FLAGS, on OSX that's where the arch
 flags end up.  On 64bit OSX Leopard, wxPython is not available in 64bits
 due to limitations in the system, so if one is trying to build 64bits,
 vdigit linking will fail.  This will strip out any 64bit flags (if they
 are the only arch flags, it's OK because the default arch is 32bits on
 OSX):

 {{{
 CXXFLAGS = -c ${SHLIB_CFLAGS} $(subst -arch ppc64,,$(subst -arch
 x86_64,,${COMPILE_FLAGS_CXX})) \
 -I$(ARCH_DISTDIR)/include $(GDALCFLAGS) $(PYTHONCFLAGS)
 $(WXWIDGETSCXXFLAGS)

 LDFLAGS = -bundle ${SHLIB_CFLAGS} $(subst -arch ppc64,,$(subst -arch
 x86_64,,${LINK_FLAGS})) \
 -L$(ARCH_LIBDIR) $(VECTLIB) $(GISLIB) $(GDALLIBS) $(VEDITLIB)
 $(WXWIDGETSLIB) $(PYTHONLDFLAGS) -lgdi
 }}}

 See ticket #58 for info about linking _gdi_.so.

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/61#comment:2>
GRASS GIS <http://grass.osgeo.org>
GRASS Geographic Information System (GRASS GIS) - http://grass.osgeo.org/