Opened 17 years ago
Closed 17 years ago
#38 closed task (fixed)
configure.in: wxwidgets and python checks
Reported by: | martinl | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 6.4.0 |
Component: | Default | Version: | svn-trunk |
Keywords: | Cc: | ||
CPU: | Unspecified | Platform: | Unspecified |
Description
Hi,
I modified the configure script to support wxWidgets. It checks
- if wx-config is presented
- wxWidgets version (>=2.8.1 required)
- check for wx/wxprec.h and wx/wxPython/pseudodc.h (wxPython required too)
The patch also checks for Python.h header if --with-python is given.
This should solve compilation of wxGUI vdigit component
http://trac.osgeo.org/grass/browser/grass/trunk/gui/wxpython/vdigit/Makefile.in
Martin
Attachments (4)
Change History (7)
by , 17 years ago
Attachment: | configure_wx.diff added |
---|
comment:1 by , 17 years ago
Type: | defect → task |
---|
comment:2 by , 17 years ago
From ML by Glynn Clements:
You should probably use python-config to get this information.
I would suggest that --with-wxwidgets and --with-python take optional arguments specifying the path to the wx-config and python-config utilities respectively, in case they aren't in the path. --with-gdal already behaves like this.
Also, is there any need to have vdigit/Makefile generated from Makefile.in. Can the configuration settings not go into Platform.make[.in]? vdigit/Makefile already includes Platform.make via Lib.make.
The patch checks for Python.h and include directory, it should work on Mac/Windows too.
The configure checks might work, but the hard-coded linking switches and ".so" won't work. I suggest trying to mimic a normal library build as closely as possible.
Based on Glynn's notes I updated the patch (see configure_wx-1.diff)
Instead of --with-python=path/py-config is used --with-python=path/python. Configure also checks for python-config.
--with-python[=path/python] enable Python support (python with path, e.g. '--with-python=/usr/bin/python2.5', default: no) --with-wxwidgets[=path/wx-config] enable wxWidgets support (wx-config with path, e.g. '--with-wxwidgets=/usr/local/bin/wx-config', default: no)
by , 17 years ago
Attachment: | configure_wx-3.diff added |
---|
Requires python-config, --with-python=[path/python-config]
comment:3 by , 17 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Submitted (r30171).
See also discussion in ML http://lists.osgeo.org/pipermail/grass-dev/2008-February/035303.html
Add check for wxWidgets