ImageMagick: jp2 support broken

[thorsted (Tyler Thorsted)]

It seems Jasper is no longer supported but installing openjpeg and recompiling ImageMagick does not add support for jp2. Additionally ImageMagick is not listed as an available port so I cannot see if I need to add a different variant other than +jpeg2

[mf2k (Frank Schima)]

In the future, please fill in the Port field and Cc the port maintainers (port info --maintainers ImageMagick).

[ryandesign (Ryan Carsten Schmidt)]

Replying to t.thorsted@…:

It seems Jasper is no longer supported but installing openjpeg and recompiling ImageMagick does not add support for jp2.

I didn't know jasper was no longer supported by ImageMagick, so thanks for letting me know. I'll have to look into this.

Additionally ImageMagick is not listed as an available port so I cannot see if I need to add a different variant other than +jpeg2

That's #40946.

[ryandesign (Ryan Carsten Schmidt)]

openjpeg support in ImageMagick relies on openjpeg supplying a pkg-config file, which version 2.0.0 which we have in MacPorts does not have. I've filed #43604 requesting the openjpeg port be updated to 2.1.0; hopefully that will have it.

[ryandesign (Ryan Carsten Schmidt)]

Has duplicate #43611.

[ryandesign (Ryan Carsten Schmidt)]

openjpeg has been updated to 2.1.0, and it does now provide the required pkg-config file, but unfortunately ImageMagick doesn't seem to be compatible with this version of openjpeg:

coders/jp2.c:343:44: error: too few arguments to function call, expected 3, have 2
  opj_stream_set_user_data(jp2_stream,image);
  ~~~~~~~~~~~~~~~~~~~~~~~~                 ^
/opt/local/include/openjpeg-2.1/openjpeg.h:1154:1: note: 'opj_stream_set_user_data' declared here
OPJ_API void OPJ_CALLCONV opj_stream_set_user_data (opj_stream_t* p_stream, void * p_data, opj_stream_free_user_data_fn p_function);
^
/opt/local/include/openjpeg-2.1/openjpeg.h:82:20: note: expanded from macro 'OPJ_API'
#define OPJ_API    __attribute__ ((visibility ("default")))
                   ^

I committed the work on ImageMagick so far in r119841.

[cooljeanius (Eric Gallager)]

Cc Me!

[mndavidoff (Monte Davidoff)]

Cc Me!

[ryandesign (Ryan Carsten Schmidt)]

The ImageMagick developers released 6.8.9-1 to fix this issue. I missed the notification from the ​forum thread about that a couple weeks ago and just noticed it now. This update is committed in r120380.