Awesome, thanks! On Wed, Dec 7, 2022 at 10:02 AM Michal Prívozník <mpriv...@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 > >
