Re: [patch] libdebian-installer poll()

2009-07-13 Thread Luca Favatella
On 09/07/2009, Luca Favatella wrote: > On 09/07/2009, Colin Watson wrote: [...] >> While // is a valid comment syntax in C99, it wasn't in earlier versions >> of C, and I believe that /* ... */ is still preferred in libd-i. > > Fixed in the attached version 2 of the patch. Committed in svn trunk

Re: [patch] libdebian-installer poll()

2009-07-09 Thread Luca Favatella
On 09/07/2009, Colin Watson wrote: > On Tue, Jul 07, 2009 at 01:05:41AM +0200, Luca Favatella wrote: [...] >> +// References: http://www.greenend.org.uk/rjk/2001/06/poll.html > > While // is a valid comment syntax in C99, it wasn't in earlier versions > of C, and I believe that /* ... */ is still

Re: [patch] libdebian-installer poll()

2009-07-09 Thread Colin Watson
On Tue, Jul 07, 2009 at 01:05:41AM +0200, Luca Favatella wrote: > This patch solves a poll() portability issue on kfreebsd-i386, and > shouldn't affect other archs. > > I tested this on kfreebsd-i386, solving an infinite loop. > Index: src/exec.c >

[patch] libdebian-installer poll()

2009-07-06 Thread Luca Favatella
This patch solves a poll() portability issue on kfreebsd-i386, and shouldn't affect other archs. I tested this on kfreebsd-i386, solving an infinite loop. Cheers, Luca Favatella Index: debian/changelog === --- debian/changelog (.../