[geos-devel] Python/Swig/Configure Bug

Mark Cave-Ayland mark.cave-ayland at ilande.co.uk
Fri Sep 7 13:28:34 EDT 2007


On Sat, 2007-09-01 at 02:13 +0100, Mark Cave-Ayland wrote:

> > I think the issue is the test against $SWIG.  I changed it to this:
> > 
> > 	if test -n x"$SWIG"x; then
> > 		use_swig=true
> > 	fi


Hi Charlie,

Having looked at this commit, I now realise it's wrong :(  According to
the man page, -n tests for return string length and according to the
macro, SWIG is set to an empty string. However, the resulting string
would then become 'xx' which would still evaluate as true since it has a
length greater than zero.

The attached patch fixes this, and does a little bit of tidying up at
the same time.


ATB,

Mark.

-- 
ILande - Open Source Consultancy
http://www.ilande.co.uk

-------------- next part --------------
A non-text attachment was scrubbed...
Name: geos-autoconf.patch
Type: text/x-patch
Size: 833 bytes
Desc: not available
Url : http://lists.osgeo.org/pipermail/geos-devel/attachments/20070907/c9d36b8d/geos-autoconf.bin


More information about the geos-devel mailing list