Needed to add pthread library check to configure.ac

[GoogleCodeExporter]

I tried to build r437 from subversion and it complained about pthread_create 
not being there. I had to make this change to make it build:

--- configure.ac~   2010-10-14 23:17:21.977142077 -0400
+++ configure.ac    2010-10-14 23:17:23.055284667 -0400
@@ -51,6 +51,8 @@ PKG_CHECK_MODULES(FUSE, fuse,,
     [AC_MSG_ERROR(["fuse" not found in pkg-config])])

 # Check for required libraries
+AC_CHECK_LIB(pthread, pthread_create,,
+   [AC_MSG_ERROR([required library pthread missing])])
 AC_CHECK_LIB(curl, curl_version,,
    [AC_MSG_ERROR([required library libcurl missing])])
 AC_CHECK_LIB(crypto, BIO_new,,

Thanks for creating and maintaining this package!

Original issue reported on code.google.com by jikam...@gmail.com on 15 Oct 2010 at 4:00

[GoogleCodeExporter]

What operating system is this?

Might be fixed in r438.

Original comment by archie.c...@gmail.com on 15 Oct 2010 at 2:30

• Changed state: Accepted

[GoogleCodeExporter]

It's Fedora 14.
I'm confused about why you hard-coded -lpthread on Linux in r438. The 
AC_CHECK_LIB will work just fine on Linux, so why make the code for Linux 
different from the other platforms that require -lpthread?

Original comment by jikam...@gmail.com on 15 Oct 2010 at 2:49

[GoogleCodeExporter]

Because as I understand it, the correct flag on Linux is -pthread, not 
-lpthread (although -lpthread seems to link just fine).

I'm still fuzzy on what the truly correct thing to do here is though.

Original comment by archie.c...@gmail.com on 15 Oct 2010 at 3:51

[GoogleCodeExporter]

Original comment by archie.c...@gmail.com on 19 Oct 2010 at 4:54

• Changed state: Fixed

[GoogleCodeExporter]

Original comment by archie.c...@gmail.com on 19 Oct 2010 at 4:55

[GoogleCodeExporter]

Original comment by archie.c...@gmail.com on 22 Oct 2010 at 8:02

• Added labels: AffectsVersion-1.3.1, FixVersion-1.3.2