Awesome, thanks! 

On Wed, Dec 7, 2022 at 10:02 AM Michal Prívozník <mprivozn@redhat.com> wrote:
On 12/6/22 20:13, Simon Fairweather wrote:
> Thanks for the update. What does this mean?
>
> The only problem is that QEMU feeds us PPM which is not wired to PHP
> really.
>
> We are using this patch to remove imageMagik as it is not required for
> our needs.
> Eric - I strip out the requirement for ImageMagick because of all the
> dependancies
> needed for that peice.  ImageMagick is only used for libvirt's
> screenshot API calls
> which we dont even take advantage of in the unRAID web UI.
> --- a/configure.ac <http://configure.ac>  2016-10-05 22:20:39.172968837
> -0500
> +++ b/configure.ac <http://configure.ac>  2016-10-05 22:21:05.645301252
> -0500
> @@ -172,8 +172,6 @@
>  LIBVIRT_CHECK_PHP_EXTENSIONDIR
>  LIBVIRT_CHECK_PHP_CONFDIR
>  
> -LIBVIRT_CHECK_PHP_EXTENSION([imagick])
> -

Yeah, pretty much every distro uses this patch. The problem is, without
imagick the examples won't work:

libvirt-php.git/tests $ php -n -c php.ini -S localhost:8080
../examples/index.php

BUT, here's what I can do. I just found out that QEMU added support for
taking screenshots in PNG format, so I can fix libvirt to prefer that
(PPM is not very well supported anyway - I haven't found a browser that
would render it) and drop imagick completely. Then I can do the release
and distros can then just drop the patch.

Michal