Opened 6 years ago

Closed 6 years ago

Last modified 5 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: UI/UX:

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 6 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 6 years ago by Russell Keith-Magee

Triage Stage: UnreviewedAccepted

comment:2 Changed 6 years ago by jbronn

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

comment:3 in reply to:  description Changed 6 years ago by jbronn

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.

Last edited 6 years ago by jbronn (previous) (diff)

Changed 6 years ago by jbronn

Attachment: 15194.1.diff added

comment:4 Changed 6 years ago by jbronn

Keywords: blocker added

comment:5 Changed 6 years ago by jbronn

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 Changed 6 years ago by jbronn

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 Changed 5 years ago by Jacob

milestone: 1.3

Milestone 1.3 deleted

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