Mesa (master): configure.ac: require pthread-stubs only where available

Emil Velikov evelikov at kemper.freedesktop.org
Mon Mar 13 11:34:08 UTC 2017


Module: Mesa
Branch: master
Commit: b82bd31c54cc91b972a027492fa75b13cd9c7267
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=b82bd31c54cc91b972a027492fa75b13cd9c7267

Author: Emil Velikov <emil.velikov at collabora.com>
Date:   Thu Mar  2 19:02:44 2017 +0000

configure.ac: require pthread-stubs only where available

The project is a thing only for BSD platforms. Or in other words - for
any other platforms building/installing pthread-stubs results only in a
pthread-stub.pc file.

And even where it provides a DSO, there's a fundamental design issue
with it - see the pthread-stubs mailing list for the specifics.

v2: Update comment above the switch statement (Jon Turney).

Reviewed-by: Jeremy Huddleston Sequoia <jeremyhu at apple.com>
Acked-by: Gary Wong <gtw at gnu.org>
Tested-by: Eric Engestrom <eric.engestrom at imgtec.com>
Acked-by: Randy Fishel <randy.fishel at oracle.com>
Cc: Niveditha Rau <niveditha.rau at oracle.com>
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>

---

 configure.ac | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/configure.ac b/configure.ac
index aa5cb75..bc9a9e4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -801,9 +801,10 @@ if test "x$android" = xno; then
     test -z "$PTHREAD_LIBS" && PTHREAD_LIBS="-lpthread"
 fi
 
-dnl pthread-stubs is mandatory on targets where it exists
+dnl pthread-stubs is mandatory on BSD platforms, due to the nature of the
+dnl project. Even then there's a notable issue as described in the project README
 case "$host_os" in
-cygwin* )
+linux* | cygwin* | darwin* | solaris* | gnu*)
     pthread_stubs_possible="no"
     ;;
 * )




More information about the mesa-commit mailing list