New Ticket     Tickets     Wiki     Browse Source     Timeline     Roadmap     Ticket Reports     Search

Ticket #34592 (new defect)

Opened 12 months ago

Last modified 12 months ago

ImageMagick: do not link with linkclparser.dylib on Snow Leopard

Reported by: frederic.devernay@… Owned by: ryandesign@…
Priority: Normal Milestone:
Component: ports Version: 2.1.1
Keywords: haspatch Cc:
Port: ImageMagick

Description

Hi, the standard distribution of ImageMagick tries different libraries for OpenCL support, uncluding -lOpenCL, -lCL and -lclparser Unfortunately, libclparser.dylib only exists - as part of the OpenCL framework - on Snow Leopard, and this results in binaries which cannot run on Lion or later MacOSX versions. Google for "libclparse.dylib lion" to see that only ImageMagick binaries are affected by this, so this is clearly IM's fault. Maybe libclparser is a legitimate OpenCL library on other platforms, but on the Mac it's not used at all. The attached patch just removes the reference to libclparser from the configure script (it could also be removed from m4/ax_opencl.m4) When compiling with this patch, the IM binaries and libraries can be run on Lion (and possibly Mountain Lion)

Attachments

noclparser.patch (417 bytes) - added by frederic.devernay@… 12 months ago.
patch for IM sources (to be added to the Portfile)

Change History

Changed 12 months ago by frederic.devernay@…

patch for IM sources (to be added to the Portfile)

comment:1 follow-up: ↓ 2 Changed 12 months ago by ryandesign@…

  • Owner changed from macports-tickets@… to ryandesign@…
  • Keywords haspatch added

Have you already reported this problem to the developers of ImageMagick so that they can fix it?

comment:2 in reply to: ↑ 1 Changed 12 months ago by frederic.devernay@…

Replying to ryandesign@…:

Have you already reported this problem to the developers of ImageMagick so that they can fix it?

It's already been reported months ago: http://www.imagemagick.org/discourse-server/viewtopic.php?f=3&t=19037 and it probably won't be fixed (there aren't that many people who compile on 10.6 and run on 10.7). They now only provide a 10.7 binary distribution anyway http://www.imagemagick.org/script/binary-releases.php#macosx . This is a shame, because there are still lots of people running 10.6 (32-bits CPUs, or people who need Rosetta, essentially).

That check for libclparser is probably here for some other platform which requires it, though I couldn't find which, so removing it may break the build on that other platform. Fixing it in MacPorts seemed the best way to go.

Note: See TracTickets for help on using tickets.