[libvirt PATCH v2 43/56] util: conditionalize FD stream to exclude WIN32
Daniel P. Berrangé
berrange at redhat.com
Wed Feb 5 17:16:05 UTC 2020
On Fri, Jan 31, 2020 at 04:13:45PM +0100, Pavel Hrdina wrote:
> On Tue, Jan 28, 2020 at 01:11:24PM +0000, Daniel P. Berrangé wrote:
> > Almost none of the virFDStream code will actually work
> > on WIN32 builds, nor is it used except for in the
> > virtualbox driver for screenshots. It is simpler to
> > wrap it all in a '#ifndef WIN32'.
> >
> > Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> > ---
> > src/util/virfdstream.c | 122 +++++++++++++++++++++++++++++++----------
> > 1 file changed, 93 insertions(+), 29 deletions(-)
>
> If we are OK with breaking screenshots on virtualbox. I guess that if
> someone is really using it we will have a bug report and we can solve it.
>
> Reviewed-by: Pavel Hrdina <phrdina at redhat.com>
I forgot we don't even support virtualbox on Windows any more, since
we moved it into libvirtd & we don't build libvirtd on Windows.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
More information about the libvir-list
mailing list