Re: [PATCH] testsuite for poll(2)

2008-09-16 Thread Paolo Bonzini
> This solved the 'unconnected socket test': > Unconnected socket test... passed > > Connected sockets test... failed (expecting POLLHUP after shutdown) > General socket test with fork... failed (expecting POLLHUP after shutdown) Yes, it was not meant to fix the others. Paolo

Re: [PATCH] testsuite for poll(2)

2008-09-16 Thread Yoann Vandoorselaere
Le mardi 16 septembre 2008 à 10:28 +0200, Paolo Bonzini a écrit : > Yoann Vandoorselaere wrote: > > Le lundi 15 septembre 2008 à 21:21 +0200, Paolo Bonzini a écrit : > >>> The poll-tests module that is now available in the GnuLib repository > >>> currently fail on OpenBSD: > >>> > >>> Unconnected s

Re: [PATCH] testsuite for poll(2)

2008-09-16 Thread Paolo Bonzini
Yoann Vandoorselaere wrote: > Le lundi 15 septembre 2008 à 21:21 +0200, Paolo Bonzini a écrit : >>> The poll-tests module that is now available in the GnuLib repository >>> currently fail on OpenBSD: >>> >>> Unconnected socket test... failed (huh, connect succeeded?) >>> Connected sockets test... f

Re: [PATCH] testsuite for poll(2)

2008-09-16 Thread Yoann Vandoorselaere
Le lundi 15 septembre 2008 à 21:21 +0200, Paolo Bonzini a écrit : > > The poll-tests module that is now available in the GnuLib repository > > currently fail on OpenBSD: > > > > Unconnected socket test... failed (huh, connect succeeded?) > > Connected sockets test... failed (huh, connect succeeded

Re: [PATCH] testsuite for poll(2)

2008-09-15 Thread Paolo Bonzini
> The poll-tests module that is now available in the GnuLib repository > currently fail on OpenBSD: > > Unconnected socket test... failed (huh, connect succeeded?) > Connected sockets test... failed (huh, connect succeeded?) Uhm, this means O_NONBLOCK sockets do not work there? > failed (expect

Re: [PATCH] testsuite for poll(2)

2008-09-15 Thread Yoann Vandoorselaere
d_format = "160x600_as"; google_ad_channel = "8427791634"; google_color_border = "FF"; google_color_bg = "FF"; google_color_link = "006792"; google_color_url = "006792"; google_color_text = "00"; //--> Re

Re: [PATCH] testsuite for poll(2)

2008-09-13 Thread Paolo Bonzini
Bruno Haible wrote: > Paolo Bonzini wrote: >> Ok, will do so (s/HAVE_IO_H/__MSVCRT__). > > But __MSVCRT__ is specific to mingw: the MSVC compiler does not define it. > > OK to commit this patch? I am not sure if __MSVCRT__ should be defined by the compiler, or rather by the library (in which cas

Re: [PATCH] testsuite for poll(2)

2008-09-13 Thread Bruno Haible
Paolo Bonzini wrote: > Ok, will do so (s/HAVE_IO_H/__MSVCRT__). But __MSVCRT__ is specific to mingw: the MSVC compiler does not define it. (See also http://predef.sourceforge.net/) Some people use wrapper scripts that make MSVC usable for autoconf based projects (cccl, cc-msvc.sh or similar). Bett

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Paolo Bonzini
Eric Blake wrote: > Paolo Bonzini gnu.org> writes: > >> Here is the final patch I pushed for Windows compatibility of tests. It >> does not test much because the socket syscalls are not compatible >> enough, but it compiles at least (using the Debian mingw32 and wine >> packages). > > But it br

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Eric Blake
Paolo Bonzini gnu.org> writes: > > Here is the final patch I pushed for Windows compatibility of tests. It > does not test much because the socket syscalls are not compatible > enough, but it compiles at least (using the Debian mingw32 and wine > packages). But it breaks on cygwin. > > #ifd

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Paolo Bonzini
Here is the final patch I pushed for Windows compatibility of tests. It does not test much because the socket syscalls are not compatible enough, but it compiles at least (using the Debian mingw32 and wine packages). Paolo commit b03b6bfb8daa20dceff4593229f0df2c68c37e6e Author: Paolo Bonzini <[EM

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Paolo Bonzini
> test-poll.o: In function `open_server_socket': > /home/yoann/dev/prelude/git/gnulib/poll/gltests/test-poll.c:91: > undefined reference to `_inet_aton' > test-poll.o: In function `connect_to_socket': > /home/yoann/dev/prelude/git/gnulib/poll/gltests/test-poll.c:121: > undefined reference to `_ine

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Yoann Vandoorselaere
Le vendredi 12 septembre 2008 à 15:49 +0200, Paolo Bonzini a écrit : > > - enum { TRUE, FALSE } will fail to compile on WIN32, better use > > stdbool, or : > > See patch at end of message. > > > - The code need to be linked to -lws2_32 to avoid undefined symbols, yet > > the dependency is not pul

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Paolo Bonzini
Thanks Yoann! > - enum { TRUE, FALSE } will fail to compile on WIN32, better use > stdbool, or : See patch at end of message. > - The code need to be linked to -lws2_32 to avoid undefined symbols, yet > the dependency is not pulled automatically. Also fixed in the patch by using the sockets mod

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Yoann Vandoorselaere
Le vendredi 12 septembre 2008 à 15:11 +0200, Paolo Bonzini a écrit : > It is minimal in that it only tests one descriptor at a time. But it > has several tests for both sockets and pipes (and if compiled with > -DINTERACTIVE, TTYs too), so while it does not cover a lot of code in > the poll emulat

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Paolo Bonzini
Eric Blake wrote: > According to Paolo Bonzini on 9/12/2008 7:11 AM: >> It is minimal in that it only tests one descriptor at a time. But it >> has several tests for both sockets and pipes (and if compiled with >> -DINTERACTIVE, TTYs too), so while it does not cover a lot of code in >> the poll em

Re: [PATCH] testsuite for poll(2)

2008-09-12 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Paolo Bonzini on 9/12/2008 7:11 AM: > It is minimal in that it only tests one descriptor at a time. But it > has several tests for both sockets and pipes (and if compiled with > -DINTERACTIVE, TTYs too), so while it does not cover a lot o

[PATCH] testsuite for poll(2)

2008-09-12 Thread Paolo Bonzini
It is minimal in that it only tests one descriptor at a time. But it has several tests for both sockets and pipes (and if compiled with -DINTERACTIVE, TTYs too), so while it does not cover a lot of code in the poll emulation it does cover all the subtleties. In particular it highlighted a differe