Opened 7 years ago

Closed 6 years ago

#36496 closed defect (fixed)

Update to gdal to allow the use of the +mrsid variant

Reported by: jsm@… Owned by: seanasy@…
Priority: Normal Milestone:
Component: ports Version: 2.1.2
Keywords: haspatch Cc: cooljeanius (Eric Gallager)
Port: gdal

Description

The port gdal with the option +mrsid failed due to problems with the dependence geoexpress-sdk.

An update for the port geoexpress-sdk has been uploaded (see Ticket #30712). An update to the "gdal" Portfile is attached here. The patches "patch-mrsid-configure" and "patch-frmts_mrsid_mrsiddataset.cpp" are apparently not needed anymore.

Attachments (4)

Portfile-gdal.diff (677 bytes) - added by jsm@… 7 years ago.
gdal_Portfile (13.1 KB) - added by venabled@… 6 years ago.
gdal.diff (2.0 KB) - added by jsm@… 6 years ago.
patch of the gdal Portfile
gdal2.diff (2.0 KB) - added by jsm@… 6 years ago.

Download all attachments as: .zip

Change History (15)

Changed 7 years ago by jsm@…

Attachment: Portfile-gdal.diff added

comment:1 Changed 7 years ago by jsm@…

Cc: jsm@… added

Cc Me!

comment:2 Changed 7 years ago by mf2k (Frank Schima)

Cc: jsm@… removed
Owner: changed from macports-tickets@… to seanasy@…

In the future please Cc the port maintainer. As the reporter, you are automatically Cc'ed.

comment:3 Changed 6 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:4 Changed 6 years ago by venabled@…

I still can't link against the geoexpress-sdk. For some reason gdal builds but is missing links from the library + the applications to libltidsdk.8.dylib which is hidden away in /opt/local/share/Geo_DSDK/Raster_DSDK/lib/ (from geoexpress-sdk port)

Changed 6 years ago by venabled@…

Attachment: gdal_Portfile added

comment:5 Changed 6 years ago by venabled@…

I created a patch which fixes the broken links to the geoexpress-sdk and attached it, if anyone is trying to use the +mrsid variant this should come in handy.

comment:6 in reply to:  5 Changed 6 years ago by mf2k (Frank Schima)

Replying to venabled@…:

I created a patch which fixes the broken links to the geoexpress-sdk and attached it, if anyone is trying to use the +mrsid variant this should come in handy.

Per the guidelines, please instead attach a unified diff of the Portfile vs. trunk so we can easily what changes you are proposing.

Changed 6 years ago by jsm@…

Attachment: gdal.diff added

patch of the gdal Portfile

comment:7 Changed 6 years ago by jsm@…

I have uploaded a patch (gdal.diff) that combines my original patch and the changes of venabled@…. I have tested it in a fresh installation of MacPorts in Mavericks and seems to be working all right.

comment:8 Changed 6 years ago by mf2k (Frank Schima)

I cannot install this because geoexpress-sdk has been again updated and I cannot install it. See #42328 about my attempt to update the geoexpress-sdk port.

comment:9 Changed 6 years ago by jsm@…

Oh, indeed, I was working with the old version of the SDK.

comment:10 Changed 6 years ago by jsm@…

The next patch (gdal2.diff) should work with the new version of the geoexpress-sdk libraries.

Changed 6 years ago by jsm@…

Attachment: gdal2.diff added

comment:11 Changed 6 years ago by mf2k (Frank Schima)

Resolution: fixed
Status: newclosed

Thanks! r116694.

Note: See TracTickets for help on using tickets.