[GRASS-dev] [GRASS GIS] #1758: Disabling python support does not prevent entry into python directories during build

[GRASS GIS]

#1758: Disabling python support does not prevent entry into python directories
during build
-----------------------+----------------------------------------------------
  Reporter:  brook     |       Owner:  grass-dev@…              
      Type:  defect    |      Status:  reopened                 
  Priority:  critical  |   Milestone:  6.4.3                    
 Component:  Python    |     Version:  6.4.2                    
Resolution:            |    Keywords:                           
  Platform:  All       |         Cpu:  Unspecified              
-----------------------+----------------------------------------------------
Changes (by glynn):

  * status:  closed => reopened
  * resolution:  fixed =>


Comment:

 Replying to [ticket:1758 brook]:
 > The configure option --without-python is presumably supposed to disable
 all python support.

 Incorrect. It's supposed to disable building C/C++ components which link
 against the Python library (are there any of these left in 6.x?).

 > However, the Makefiles are set to unconditionally enter the gui/wxpython
 and lib/python subdirectories.

 This is correct.

 > As a result, this option results in a failed build.

 Error message? Other symptoms?

 > The solution is to conditionally include those directories in the list
 of subdirectories for make.  Two patches against the 6.4.2 release that
 fix this are attached.  This has been compile-checked on a NetBSD system
 both with and without the --without-python option.
 >
 > Presumably this should be pulled up to the more recent versions as well.

 It shouldn't go into any version;  r53326 and r53327 are in error. If the
 build fails due to a missing Python interpreter, the Makefiles should be
 fixed so that such errors are reported and ignored, rather than aborting
 the build.

 A couple of things which stand out are the menustrings.py rule in
 gui/wxpython/Makefile, as well as the rule which runs create__init__.py.
 The former is decidedly non-critical (it just means that "make -C locale
 pot" won't get the strings from the XML files for the menus). The latter
 should ideally be fixed to not require Python; the wxGUI should should
 only require Python on the target system, not on the build system.

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/1758#comment:2>
GRASS GIS <http://grass.osgeo.org>