[libvirt] [PATCH] virsh: fix mingw failure on creating nonblocking pipe

Daniel P. Berrange berrange at redhat.com
Fri Apr 1 09:44:33 UTC 2011


On Thu, Mar 31, 2011 at 04:12:09PM -0600, Eric Blake wrote:
> On 03/31/2011 04:03 PM, Eric Blake wrote:
> > * .gnulib: Update to latest, for nonblocking module.
> > * bootstrap.conf (gnulib_modules): Add nonblocking.
> > * src/util/util.c (virSetBlocking): Defer to gnulib.
> > ---
> > 
> > Matthias, does this work for you on your mingw build?  So far,
> > I have only tested that it cross-compiles, and that the new
> > gnulib module passes its self-test when run in isolation on mingw.
> > 
> >  int virSetBlocking(int fd, bool blocking) {
> 
> > -    if (blocking)
> > -        flags &= ~O_NONBLOCK;
> 
> > +    return set_nonblocking_flag (fd, blocking);
> 
> Oops - that _can't_ work.  s/blocking/!blocking/ in that return statement.

ACK with that fix :-)

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|




More information about the libvir-list mailing list