Update to multimedia/transcode broke ImageMagick

[Coleman Kane]

On Tue, 2008-04-22 at 17:41 +0200, Martin Wilke wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On Tue, Apr 22, 2008 at 10:58:16AM -0400, Coleman Kane wrote:
> > Hi,
> > 
> > The upgrade of ImageMagick breaks the ability for transcode to build
> > WTIH_IMAGEMAGICK=yes, even more than the small Makefile change it
> > recently experienced.
> > 
> > It seems that the configure script is still looking for libMagick.so
> > (instead of libMagickCore.so or libMagickWand.so).
> > 
> > Additionally, the following:
> > 
> > .if (defined(WITH_IMAGEMAGICK) || exists(${LOCALBASE}/lib/libMagick.so)) && !defined(WITHOUT_IMAGEMAGICK)
> > LIB_DEPENDS+=	MagickWand.1:${PORTSDIR}/graphics/ImageMagick
> > CONFIGURE_ARGS+=	--enable-imagemagick --with-imagemagick-prefix=${LOCALBASE}
> > .else
> > CONFIGURE_ARGS+=	--enable-imagemagick=no
> > .endif
> > 
> > Should be changed to:
> > 
> > .if (defined(WITH_IMAGEMAGICK) || exists(${LOCALBASE}/lib/libMagickWand.so)) && !defined(WITHOUT_IMAGEMAGICK)
> > LIB_DEPENDS+=	MagickWand.1:${PORTSDIR}/graphics/ImageMagick
> > CONFIGURE_ARGS+=	--enable-imagemagick --with-imagemagick-prefix=${LOCALBASE}
> > .else
> > CONFIGURE_ARGS+=	--enable-imagemagick=no
> > .endif
> 
> You are right. Fixed.
> 
> > 
> > (The exists(...) clause needs to mimic the change on LIB_DEPENDS+= )
> > 
> > -- 
> > Coleman Kane

And the bug in the configure script?

-- 
Coleman Kane
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: This is a digitally signed message part
Url : http://lists.freebsd.org/pipermail/freebsd-multimedia/attachments/20080422/2d553166/attachment.pgp