[ccan] [PATCH] tap: fix pthread uses

Kirill A. Shutemov kirill at shutemov.name
Mon Jul 4 16:07:12 EST 2011


On Mon, Jul 04, 2011 at 08:44:19AM +0930, Rusty Russell wrote:
> On Sun,  3 Jul 2011 20:36:48 +0300, "Kirill A. Shutemov" <kirill at shutemov.name> wrote:
> > From: "Kirill A. Shutemov" <kirill at shutemov.name>
> > 
> > tap expects that HAVE_LIBPTHREAD will be defined, but configurator
> > doesn't provide it:
> > 
> > ccan/tap/tap.c:49:5: warning: "HAVE_LIBPTHREAD" is not defined [-Wundef]
> > 
> > Lets check for <pthread.h> in configurator and provide HAVE_PTHREAD_H
> > based on the results.
> > 
> > Signed-off-by: Kirill A. Shutemov <kirill at shutemov.name>
> 
> Hmm, this predates me taking over maintenance of tap.
> 
> I'm not entirely convinced that having multiple threads use tap at the
> same time makes sense, nor sure that dragging in pthreads for every tap
> user is desirable.
> 
> I'm tempted to change it to WANT_PTHREAD.  Do you actually need it?

No, I want clean build log ;)

-- 
 Kirill A. Shutemov


More information about the ccan mailing list