Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#1229 closed bug (fixed)

FindGEOS does not detect GEOS.Framework on OSX 10.5

Reported by: mloskot Owned by: nobody
Priority: major: does not work as expected Milestone:
Component: Build/Install Version: Trunk
Keywords: cmake geos Cc:
Must Fix for Release: No Platform: OS X
Platform Version: 10.5 Awaiting user input: no

Description

Under Mac OS X 10.5, current version of [trunk/qgis/cmake/FindGEOS.cmake@9093 FindGEOS.cmake] macro does not detect GEOS installed from the excellent GEOS framework v3.0.0-2 package by William Kyngesburye.

I've followed the official BuildingFromSource instructions and cmake command reported that some dependencies are not found, but no specific error for GEOS was printed.

Anyway, I patched the FindGEOS.cmake in more portable way based on FindGDAL.cmake and used it to successfully build QGIS from SVN trunk (r9104).

I'm attaching patch with my version of the FindGEOS.cmake macro.

Attachments (1)

findgeos-using-geos-config.patch (5.1 KB ) - added by mloskot 16 years ago.
Patch with portable version of FindGEOS.cmake macro that fixes problems under OSX 10.5

Download all attachments as: .zip

Change History (6)

by mloskot, 16 years ago

Patch with portable version of FindGEOS.cmake macro that fixes problems under OSX 10.5

comment:1 by mloskot, 16 years ago

Hmm, as I see I can not edit Ticket description for QGIS, so please could anyone fix broken link to FindGEOS.cmake file adding source: prefix?

[source:trunk/qgis/cmake/FindGEOS.cmake@9093 FindGEOS.cmake]

comment:2 by jctull, 16 years ago

I can confirm this issue. I've always mapped it manually with: -D GEOS_INCLUDE_DIR=/Library/Frameworks/GEOS.framework/unix/include -D GEOS_LIBRARY=/Library/Frameworks/GEOS.framework/unix/lib/libgeos.dylib in my cmake statement. There are some other frameworks I similarly map (proj, gdal, and sqlite).

Using a generic 'cmake ..' run, it is only the geos framework that is not discovered. I am also using William Kyngesburye's frameworks, and this was rev 9104.

comment:3 by jef, 16 years ago

Resolution: fixed
Status: newclosed

thanks for the patch. applied in r9105

comment:4 by mloskot, 16 years ago

Jef,

Thanks for applying. I confirm it works.

comment:5 by jctull, 16 years ago

Worked for me also.

Note: See TracTickets for help on using tickets.