unofficial mirror of bug-guile@gnu.org 
 help / color / mirror / Atom feed
* portable pthreads test
@ 2004-09-08  5:51 Andreas Vögele
  2004-09-08 22:01 ` Kevin Ryde
  2004-09-21  0:11 ` Marius Vollmer
  0 siblings, 2 replies; 7+ messages in thread
From: Andreas Vögele @ 2004-09-08  5:51 UTC (permalink / raw)


[-- Attachment #1: Type: text/plain, Size: 408 bytes --]

Here's a patch that replaces the test for the pthread library with the 
macro from 
<http://www.gnu.org/software/ac-archive/htmldoc/acx_pthread.html>. The 
patch appends acx_pthread.m4 to acinclude.m4 and replaces 
AC_CHECK_LIB(pthread, ...) with ACX_PTHREAD in configure.in. Without 
this patch Guile CVS cannot be built on FreeBSD. I've tested the patch 
on Debian Sarge, Mac OS X, OpenBSD and FreeBSD 5.


[-- Attachment #2: pthread-check.diff --]
[-- Type: application/octet-stream, Size: 9124 bytes --]

Index: configure.in
===================================================================
RCS file: /cvsroot/guile/guile/guile-core/configure.in,v
retrieving revision 1.252
diff -u -r1.252 configure.in
--- configure.in	3 Sep 2004 19:45:35 -0000	1.252
+++ configure.in	8 Sep 2004 05:39:48 -0000
@@ -1000,17 +1000,13 @@
 
 case "$with_threads" in
   "yes" | "pthread" | "pthreads" | "pthread-threads" | "")
-    AC_CHECK_LIB(pthread, main,
-      LIBS="-lpthread $LIBS"
+    ACX_PTHREAD(CC="$PTHREAD_CC"
+      CFLAGS="$PTHREAD_CFLAGS $CFLAGS"
+      LIBS="$PTHREAD_LIBS $LIBS"
       SCM_I_GSC_USE_PTHREAD_THREADS=1
       with_threads="pthreads",
       with_threads="null")
       
-    if test $GCC = yes; then
-    AC_DEFINE(_THREAD_SAFE, 1,
-      [Use thread safe versions of GNU Libc functions.])
-    fi
-    
     AC_MSG_CHECKING(if pthread_mutexattr_settype is declared)
     AC_CACHE_VAL(guile_cv_mutexattr_settype_declared,
       [AC_TRY_COMPILE([#include <pthread.h>],
Index: acinclude.m4
===================================================================
RCS file: /cvsroot/guile/guile/guile-core/acinclude.m4,v
retrieving revision 1.16
diff -u -r1.16 acinclude.m4
--- acinclude.m4	25 Jan 2004 13:02:21 -0000	1.16
+++ acinclude.m4	8 Sep 2004 05:39:48 -0000
@@ -107,3 +107,206 @@
 ])dnl
 fi
 ])
+
+
+
+
+dnl Available from the GNU Autoconf Macro Archive at:
+dnl http://www.gnu.org/software/ac-archive/htmldoc/acx_pthread.html
+dnl
+AC_DEFUN([ACX_PTHREAD], [
+AC_REQUIRE([AC_CANONICAL_HOST])
+AC_LANG_SAVE
+AC_LANG_C
+acx_pthread_ok=no
+
+# We used to check for pthread.h first, but this fails if pthread.h
+# requires special compiler flags (e.g. on True64 or Sequent).
+# It gets checked for in the link test anyway.
+
+# First of all, check if the user has set any of the PTHREAD_LIBS,
+# etcetera environment variables, and if threads linking works using
+# them:
+if test x"$PTHREAD_LIBS$PTHREAD_CFLAGS" != x; then
+        save_CFLAGS="$CFLAGS"
+        CFLAGS="$CFLAGS $PTHREAD_CFLAGS"
+        save_LIBS="$LIBS"
+        LIBS="$PTHREAD_LIBS $LIBS"
+        AC_MSG_CHECKING([for pthread_join in LIBS=$PTHREAD_LIBS with CFLAGS=$PTHREAD_CFLAGS])
+        AC_TRY_LINK_FUNC(pthread_join, acx_pthread_ok=yes)
+        AC_MSG_RESULT($acx_pthread_ok)
+        if test x"$acx_pthread_ok" = xno; then
+                PTHREAD_LIBS=""
+                PTHREAD_CFLAGS=""
+        fi
+        LIBS="$save_LIBS"
+        CFLAGS="$save_CFLAGS"
+fi
+
+# We must check for the threads library under a number of different
+# names; the ordering is very important because some systems
+# (e.g. DEC) have both -lpthread and -lpthreads, where one of the
+# libraries is broken (non-POSIX).
+
+# Create a list of thread flags to try.  Items starting with a "-" are
+# C compiler flags, and other items are library names, except for "none"
+# which indicates that we try without any flags at all, and "pthread-config"
+# which is a program returning the flags for the Pth emulation library.
+
+acx_pthread_flags="pthreads none -Kthread -kthread lthread -pthread -pthreads -mthreads pthread --thread-safe -mt pthread-config"
+
+# The ordering *is* (sometimes) important.  Some notes on the
+# individual items follow:
+
+# pthreads: AIX (must check this before -lpthread)
+# none: in case threads are in libc; should be tried before -Kthread and
+#       other compiler flags to prevent continual compiler warnings
+# -Kthread: Sequent (threads in libc, but -Kthread needed for pthread.h)
+# -kthread: FreeBSD kernel threads (preferred to -pthread since SMP-able)
+# lthread: LinuxThreads port on FreeBSD (also preferred to -pthread)
+# -pthread: Linux/gcc (kernel threads), BSD/gcc (userland threads)
+# -pthreads: Solaris/gcc
+# -mthreads: Mingw32/gcc, Lynx/gcc
+# -mt: Sun Workshop C (may only link SunOS threads [-lthread], but it
+#      doesn't hurt to check since this sometimes defines pthreads too;
+#      also defines -D_REENTRANT)
+# pthread: Linux, etcetera
+# --thread-safe: KAI C++
+# pthread-config: use pthread-config program (for GNU Pth library)
+
+case "${host_cpu}-${host_os}" in
+        *solaris*)
+
+        # On Solaris (at least, for some versions), libc contains stubbed
+        # (non-functional) versions of the pthreads routines, so link-based
+        # tests will erroneously succeed.  (We need to link with -pthread or
+        # -lpthread.)  (The stubs are missing pthread_cleanup_push, or rather
+        # a function called by this macro, so we could check for that, but
+        # who knows whether they'll stub that too in a future libc.)  So,
+        # we'll just look for -pthreads and -lpthread first:
+
+        acx_pthread_flags="-pthread -pthreads pthread -mt $acx_pthread_flags"
+        ;;
+esac
+
+if test x"$acx_pthread_ok" = xno; then
+for flag in $acx_pthread_flags; do
+
+        case $flag in
+                none)
+                AC_MSG_CHECKING([whether pthreads work without any flags])
+                ;;
+
+                -*)
+                AC_MSG_CHECKING([whether pthreads work with $flag])
+                PTHREAD_CFLAGS="$flag"
+                ;;
+
+		pthread-config)
+		AC_CHECK_PROG(acx_pthread_config, pthread-config, yes, no)
+		if test x"$acx_pthread_config" = xno; then continue; fi
+		PTHREAD_CFLAGS="`pthread-config --cflags`"
+		PTHREAD_LIBS="`pthread-config --ldflags` `pthread-config --libs`"
+		;;
+
+                *)
+                AC_MSG_CHECKING([for the pthreads library -l$flag])
+                PTHREAD_LIBS="-l$flag"
+                ;;
+        esac
+
+        save_LIBS="$LIBS"
+        save_CFLAGS="$CFLAGS"
+        LIBS="$PTHREAD_LIBS $LIBS"
+        CFLAGS="$CFLAGS $PTHREAD_CFLAGS"
+
+        # Check for various functions.  We must include pthread.h,
+        # since some functions may be macros.  (On the Sequent, we
+        # need a special flag -Kthread to make this header compile.)
+        # We check for pthread_join because it is in -lpthread on IRIX
+        # while pthread_create is in libc.  We check for pthread_attr_init
+        # due to DEC craziness with -lpthreads.  We check for
+        # pthread_cleanup_push because it is one of the few pthread
+        # functions on Solaris that doesn't have a non-functional libc stub.
+        # We try pthread_create on general principles.
+        AC_TRY_LINK([#include <pthread.h>],
+                    [pthread_t th; pthread_join(th, 0);
+                     pthread_attr_init(0); pthread_cleanup_push(0, 0);
+                     pthread_create(0,0,0,0); pthread_cleanup_pop(0); ],
+                    [acx_pthread_ok=yes])
+
+        LIBS="$save_LIBS"
+        CFLAGS="$save_CFLAGS"
+
+        AC_MSG_RESULT($acx_pthread_ok)
+        if test "x$acx_pthread_ok" = xyes; then
+                break;
+        fi
+
+        PTHREAD_LIBS=""
+        PTHREAD_CFLAGS=""
+done
+fi
+
+# Various other checks:
+if test "x$acx_pthread_ok" = xyes; then
+        save_LIBS="$LIBS"
+        LIBS="$PTHREAD_LIBS $LIBS"
+        save_CFLAGS="$CFLAGS"
+        CFLAGS="$CFLAGS $PTHREAD_CFLAGS"
+
+        # Detect AIX lossage: threads are created detached by default
+        # and the JOINABLE attribute has a nonstandard name (UNDETACHED).
+        AC_MSG_CHECKING([for joinable pthread attribute])
+        AC_TRY_LINK([#include <pthread.h>],
+                    [int attr=PTHREAD_CREATE_JOINABLE;],
+                    ok=PTHREAD_CREATE_JOINABLE, ok=unknown)
+        if test x"$ok" = xunknown; then
+                AC_TRY_LINK([#include <pthread.h>],
+                            [int attr=PTHREAD_CREATE_UNDETACHED;],
+                            ok=PTHREAD_CREATE_UNDETACHED, ok=unknown)
+        fi
+        if test x"$ok" != xPTHREAD_CREATE_JOINABLE; then
+                AC_DEFINE(PTHREAD_CREATE_JOINABLE, $ok,
+                          [Define to the necessary symbol if this constant
+                           uses a non-standard name on your system.])
+        fi
+        AC_MSG_RESULT(${ok})
+        if test x"$ok" = xunknown; then
+                AC_MSG_WARN([we do not know how to create joinable pthreads])
+        fi
+
+        AC_MSG_CHECKING([if more special flags are required for pthreads])
+        flag=no
+        case "${host_cpu}-${host_os}" in
+                *-aix* | *-freebsd* | *-darwin*) flag="-D_THREAD_SAFE";;
+                *solaris* | *-osf* | *-hpux*) flag="-D_REENTRANT";;
+        esac
+        AC_MSG_RESULT(${flag})
+        if test "x$flag" != xno; then
+                PTHREAD_CFLAGS="$flag $PTHREAD_CFLAGS"
+        fi
+
+        LIBS="$save_LIBS"
+        CFLAGS="$save_CFLAGS"
+
+        # More AIX lossage: must compile with cc_r
+        AC_CHECK_PROG(PTHREAD_CC, cc_r, cc_r, ${CC})
+else
+        PTHREAD_CC="$CC"
+fi
+
+AC_SUBST(PTHREAD_LIBS)
+AC_SUBST(PTHREAD_CFLAGS)
+AC_SUBST(PTHREAD_CC)
+
+# Finally, execute ACTION-IF-FOUND/ACTION-IF-NOT-FOUND:
+if test x"$acx_pthread_ok" = xyes; then
+        ifelse([$1],,AC_DEFINE(HAVE_PTHREAD,1,[Define if you have POSIX threads libraries and header files.]),[$1])
+        :
+else
+        acx_pthread_ok=no
+        $2
+fi
+AC_LANG_RESTORE
+])dnl ACX_PTHREAD

[-- Attachment #3: Type: text/plain, Size: 137 bytes --]

_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: portable pthreads test
  2004-09-08  5:51 portable pthreads test Andreas Vögele
@ 2004-09-08 22:01 ` Kevin Ryde
  2004-09-15 11:36   ` Andreas Vögele
                     ` (2 more replies)
  2004-09-21  0:11 ` Marius Vollmer
  1 sibling, 3 replies; 7+ messages in thread
From: Kevin Ryde @ 2004-09-08 22:01 UTC (permalink / raw)
  Cc: bug-guile

Andreas Vögele <voegelas@gmx.net> writes:
>
> Here's a patch that replaces the test for the pthread library with the
> macro from
> <http://www.gnu.org/software/ac-archive/htmldoc/acx_pthread.html>.

Copyright paperwork will be needed before that can be used.

> Without this patch Guile CVS cannot be built on FreeBSD.

Yep, I struck that too.

Using "gcc -pthread" is probably enough, but what "guile-config
compile" spits out needs to be considered.  Giving "-pthread" might
make that compiler-dependent (which is not really a problem in
practice on a free system).


_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: portable pthreads test
  2004-09-08 22:01 ` Kevin Ryde
@ 2004-09-15 11:36   ` Andreas Vögele
  2004-09-19 13:40   ` Andreas Vögele
  2004-09-21  0:11   ` Marius Vollmer
  2 siblings, 0 replies; 7+ messages in thread
From: Andreas Vögele @ 2004-09-15 11:36 UTC (permalink / raw)
  Cc: bug-guile

Kevin Ryde writes:

> Andreas Vögele <voegelas@gmx.net> writes:
>>
>> Here's a patch that replaces the test for the pthread library with the
>> macro from
>> <http://www.gnu.org/software/ac-archive/htmldoc/acx_pthread.html>.
>
> Copyright paperwork will be needed before that can be used.

I've contacted Steven Johnson, the author of ACX_PTHREAD.  I don't know 
if the GNU Autoconf Macro Archive is an official part of the GNU 
Project.  If yes, it's likely that Steven has already signed the 
necessary papers.



_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: portable pthreads test
  2004-09-08 22:01 ` Kevin Ryde
  2004-09-15 11:36   ` Andreas Vögele
@ 2004-09-19 13:40   ` Andreas Vögele
  2004-09-21  0:11   ` Marius Vollmer
  2 siblings, 0 replies; 7+ messages in thread
From: Andreas Vögele @ 2004-09-19 13:40 UTC (permalink / raw)


Kevin Ryde writes:

> Andreas Vögele <voegelas@gmx.net> writes:
>>
>> Here's a patch that replaces the test for the pthread library with the
>> macro from
>> <http://www.gnu.org/software/ac-archive/htmldoc/acx_pthread.html>.
>
> Copyright paperwork will be needed before that can be used.

Steven Johnson would be willing to do the paperwork.  He writes:

  The autoconf macro repository does not require a copyright
  assignment.  I would be willing to sign one, though.

  It might be worth thinking about whether ACX_PTHREAD should be added
  to autoconf proper, since it seems to be a fairly widely used macro.
  (I have already signed a copyright assignment for my work on
  Autoconf.)

  Cordially,
  Steven G. Johnson


_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: portable pthreads test
  2004-09-08 22:01 ` Kevin Ryde
  2004-09-15 11:36   ` Andreas Vögele
  2004-09-19 13:40   ` Andreas Vögele
@ 2004-09-21  0:11   ` Marius Vollmer
  2004-09-21  1:17     ` Kevin Ryde
  2 siblings, 1 reply; 7+ messages in thread
From: Marius Vollmer @ 2004-09-21  0:11 UTC (permalink / raw)
  Cc: bug-guile

Kevin Ryde <user42@zip.com.au> writes:

> Andreas Vögele <voegelas@gmx.net> writes:
>>
>> Here's a patch that replaces the test for the pthread library with the
>> macro from
>> <http://www.gnu.org/software/ac-archive/htmldoc/acx_pthread.html>.
>
> Copyright paperwork will be needed before that can be used.

I say we don't need the papers for this.  The code is not part of
Guile, we only use it.  Like libltdl.

-- 
GPG: D5D4E405 - 2F9B BCCC 8527 692A 04E3  331E FAF8 226A D5D4 E405


_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: portable pthreads test
  2004-09-08  5:51 portable pthreads test Andreas Vögele
  2004-09-08 22:01 ` Kevin Ryde
@ 2004-09-21  0:11 ` Marius Vollmer
  1 sibling, 0 replies; 7+ messages in thread
From: Marius Vollmer @ 2004-09-21  0:11 UTC (permalink / raw)
  Cc: bug-guile

Andreas Vögele <voegelas@gmx.net> writes:

> Here's a patch

Applied, thanks!

-- 
GPG: D5D4E405 - 2F9B BCCC 8527 692A 04E3  331E FAF8 226A D5D4 E405


_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: portable pthreads test
  2004-09-21  0:11   ` Marius Vollmer
@ 2004-09-21  1:17     ` Kevin Ryde
  0 siblings, 0 replies; 7+ messages in thread
From: Kevin Ryde @ 2004-09-21  1:17 UTC (permalink / raw)
  Cc: bug-guile

Marius Vollmer <mvo@zagadka.de> writes:
>
> I say we don't need the papers for this.  The code is not part of
> Guile, we only use it.

Make sure the right copyright notice gets into configure, since that
file will be a joint work.


_______________________________________________
Bug-guile mailing list
Bug-guile@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-guile


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2004-09-21  1:17 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-09-08  5:51 portable pthreads test Andreas Vögele
2004-09-08 22:01 ` Kevin Ryde
2004-09-15 11:36   ` Andreas Vögele
2004-09-19 13:40   ` Andreas Vögele
2004-09-21  0:11   ` Marius Vollmer
2004-09-21  1:17     ` Kevin Ryde
2004-09-21  0:11 ` Marius Vollmer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).