Opened 16 years ago

Closed 16 years ago

#2069 closed defect (fixed)

Mac OS X patches - framework libname fix

Reported by: kyngchaos Owned by: hobu
Priority: high Milestone: 1.5.0
Component: ConfigBuild Version: 1.5.0 betas/RCs
Severity: major Keywords:
Cc:

Description

Sorry, forgot about this. I worked out a way to get the OSX framework install_name set correctly during compilation, instead of during install. Review:

The problem with setting the install_name during install, is that the SWIG python install is controlled by Python and there is no simple way to find the installed files to change the linked gdal framework install_name. Roughly:

  1. libgdal built as libgdal.dylib
  2. SWIG links libgdal.dylib
  3. install libgdal.dylib
  4. change installed libgdal.dylib -> GDAL
  5. SWIG Python installed, still linked to libgdal.dylib
  6. can't locate installed SWIG Python to change linking to GDAL

These patches will change the install_name of libgdal.dylib right after compilation of the library, so that SWIG will link to the correct install_name from the start, and there is no need to change the linking afterwards.

Attachments (2)

GNUmakefile.patch (1.1 KB ) - added by kyngchaos 16 years ago.
main makefile
GNUmakefile.2.patch (508 bytes ) - added by kyngchaos 16 years ago.
apps makefile

Download all attachments as: .zip

Change History (6)

by kyngchaos, 16 years ago

Attachment: GNUmakefile.patch added

main makefile

by kyngchaos, 16 years ago

Attachment: GNUmakefile.2.patch added

apps makefile

comment:1 by hobu, 16 years ago

Owner: changed from warmerdam to hobu
Status: newassigned

applied in r13286

comment:2 by hobu, 16 years ago

William,

Can you confirm and close?

Howard

comment:3 by kyngchaos, 16 years ago

Sorry, I missed your revision note. Looks good - I see it in the beta2 source. Thanks.

PS. Trac isn't letting me resolve this, but gives me a database is locked error.

comment:4 by hobu, 16 years ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.