<div dir="ltr">Awesome, thanks! <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 7, 2022 at 10:02 AM Michal Prívozník <<a href="mailto:mprivozn@redhat.com">mprivozn@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 12/6/22 20:13, Simon Fairweather wrote:<br>
> Thanks for the update. What does this mean?<br>
> <br>
> The only problem is that QEMU feeds us PPM which is not wired to PHP<br>
> really.<br>
> <br>
> We are using this patch to remove imageMagik as it is not required for<br>
> our needs.<br>
> Eric - I strip out the requirement for ImageMagick because of all the<br>
> dependancies<br>
> needed for that peice.  ImageMagick is only used for libvirt's<br>
> screenshot API calls<br>
> which we dont even take advantage of in the unRAID web UI.<br>
> --- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> <<a href="http://configure.ac" rel="noreferrer" target="_blank">http://configure.ac</a>>  2016-10-05 22:20:39.172968837<br>
> -0500<br>
> +++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> <<a href="http://configure.ac" rel="noreferrer" target="_blank">http://configure.ac</a>>  2016-10-05 22:21:05.645301252<br>
> -0500<br>
> @@ -172,8 +172,6 @@<br>
>  LIBVIRT_CHECK_PHP_EXTENSIONDIR<br>
>  LIBVIRT_CHECK_PHP_CONFDIR<br>
>  <br>
> -LIBVIRT_CHECK_PHP_EXTENSION([imagick])<br>
> -<br>
<br>
Yeah, pretty much every distro uses this patch. The problem is, without<br>
imagick the examples won't work:<br>
<br>
libvirt-php.git/tests $ php -n -c php.ini -S localhost:8080<br>
../examples/index.php<br>
<br>
BUT, here's what I can do. I just found out that QEMU added support for<br>
taking screenshots in PNG format, so I can fix libvirt to prefer that<br>
(PPM is not very well supported anyway - I haven't found a browser that<br>
would render it) and drop imagick completely. Then I can do the release<br>
and distros can then just drop the patch.<br>
<br>
Michal<br>
<br>
</blockquote></div>