Hi! Thanks for your contribution! :)

The imagick extension for PHP is an extension and not a package, so the commit message should be phpExtensions.imagick: message because that's actual attribute name of the derivation.

I would say that I'm in favor of this change, we won't backport it to stable of course, so it ends up in the next stable. I mean ImageMagick 6 is older than 7. Sooner or later we would have to move forward.

@ofborg build phpExtensions.imagick

Whoops! Typo'd it. I amended the commit with just the change in message.

Hmm, just realised, URLs to GitHub PRs like that are not necessarily stable, at least not while they are open. I may have to make changes to the upstream PR because of feedback. So it may not be wise to merge this yet. 😕

@stephank You could use https://github.com/Imagick/imagick/commit/09551fbf38c16cdaf4ade7c08744501cd82d2747.patch which is stable at least.

Missed some versions before :)

@ofborg build php73Extensions.imagick php74Extensions.imagick php80Extensions.imagick

I tried building on darwin, but it fails due to the anticipated hash mismatch on the patch.

@talyz Ah, sorry. I just updated the upstream PR, you're quick. :)

I rebased this and updated it with the fixed URL.

Great! Now it builds fine on darwin :)

@ofborg build php73Extensions.imagick php74Extensions.imagick php80Extensions.imagick

@ofborg eval