[GRASS-dev] [GRASS-SVN] r53326 - in grass/branches/releasebranch_6_4: gui lib

Hi Markus,

this change has serious consequences for wingrass compilation. wxGUI
is not included because of missing `--with-python`. OSGeo4W doesn't
include python2.7-config utility, so we would need our own version
like for geos or gdal. For now I would prefer to revert these changes
as Glynn noted. Configuration switch `--with-python` should be used
only for the parts which are build against Python library. Which is
not true for wxGUI or Python scripting library (it's only using python
interpreter for generating bytecode).

Martin

PS: in GRASS 7 we could probably introduce --without-gui switch which
would cause skipping whole `gui` directory during compilation.

2012/10/6 <svn_grass@osgeo.org>:

Author: neteler
Date: 2012-10-06 10:00:17 -0700 (Sat, 06 Oct 2012)
New Revision: 53326

Modified:
   grass/branches/releasebranch_6_4/gui/Makefile
   grass/branches/releasebranch_6_4/lib/Makefile
Log:
enable Python code compilation only if Python present (fixes trac #1758)

Modified: grass/branches/releasebranch_6_4/gui/Makefile

--- grass/branches/releasebranch_6_4/gui/Makefile 2012-10-06 03:08:12 UTC (rev 53325)
+++ grass/branches/releasebranch_6_4/gui/Makefile 2012-10-06 17:00:17 UTC (rev 53326)
@@ -1,9 +1,14 @@
MODULE_TOPDIR = ..

-SUBDIRS = icons images scripts xml tcltk wxpython
+SUBDIRS = icons images scripts xml tcltk

include $(MODULE_TOPDIR)/include/Make/Platform.make

+#compile if python present:
+ifneq ($(USE_PYTHON),)
+ SUBDIRS += wxpython
+endif
+
include $(MODULE_TOPDIR)/include/Make/Dir.make

default: parsubdirs

--
Martin Landa <landa.martin gmail.com> * http://geo.fsv.cvut.cz/~landa

Hi Markus,

2012/10/8 Martin Landa <landa.martin@gmail.com>:

like for geos or gdal. For now I would prefer to revert these changes
as Glynn noted. Configuration switch `--with-python` should be used

I have reverted these changes. Thanks for the collaboration off-list.
Now I will rebuild winGRASS.

Martin

--
Martin Landa <landa.martin gmail.com> * http://geo.fsv.cvut.cz/~landa