Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#15194 closed (fixed)

Need 'geos_c' as library name for Windows

Reported by: master Owned by: jbronn
Component: GIS Version: 1.2
Severity: Keywords: geos_c blocker
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The OSGeo4W project (http://trac.osgeo.org/osgeo4w/) provides the GEOS library as 'geos_c.dll' (by default in C:\OSGeo4W\bin\).

So adding the name 'geos_c' in gis/geos/libgeos.py in the "os.name == 'nt'" case would be nice and save us the pain to set a GEOS_LIBRARY_PATH setting.

By the way, who provides the lib as 'libgeos_c-1' ? Cygwin ? A comment about originators is welcome.

Attachments (1)

15194.1.diff (451 bytes ) - added by jbronn 13 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 by Russell Keith-Magee, 13 years ago

Triage Stage: UnreviewedAccepted

comment:2 by jbronn, 13 years ago

milestone: 1.3
Owner: changed from nobody to jbronn
Status: newassigned

in reply to:  description comment:3 by jbronn, 13 years ago

Replying to master:

By the way, who provides the lib as 'libgeos_c-1' ? Cygwin ? A comment about originators is welcome.

Excellent question. The Application Stack Builder (ASB) that comes with the Windows distribution of PostgreSQL allowed users to install PostGIS, and is the source of libgeos_c-1.dll. I assume it was named that way to prevent conflicting with other libraries, like !FWTools.

Version 0, edited 13 years ago by jbronn (next)

by jbronn, 13 years ago

Attachment: 15194.1.diff added

comment:4 by jbronn, 13 years ago

Keywords: blocker added

comment:5 by jbronn, 13 years ago

Resolution: fixed
Status: assignedclosed

In [15852]:

Fixed #15194 -- Add OSGeo4W DLL library names to GEOS and GDAL interfaces. Thanks, master for ticket and initial patch.

comment:6 by jbronn, 13 years ago

In [15853]:

[1.2.X] Fixed #15194 -- Add OSGeo4W DLL library names to GEOS and GDAL interfaces. Thanks, master for ticket and initial patch.

Backport of r15852 from trunk.

comment:7 by Jacob, 13 years ago

milestone: 1.3

Milestone 1.3 deleted

Note: See TracTickets for help on using tickets.
Back to Top