After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 585039 - rtpool-test can't find pthreads
rtpool-test can't find pthreads
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Windows
: Normal trivial
: 0.10.24
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-06-07 03:02 UTC by LRN
Modified: 2009-06-07 20:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
A crude patch that fixes the problem. At least - on Windows. (787 bytes, patch)
2009-06-07 03:04 UTC, LRN
none Details | Review

Description LRN 2009-06-07 03:02:37 UTC
Because -lpthread argument to the linker is missing. I don't know where it comes from on Linux, but on Windows it isn't there (and why would you want to use pthreads instead of gthread anyway?).
Comment 1 LRN 2009-06-07 03:04:10 UTC
Created attachment 136075 [details] [review]
A crude patch that fixes the problem. At least - on Windows.
Comment 2 Wim Taymans 2009-06-07 11:43:19 UTC
Can't we just disable this example on windows?

We are using pthreads here as an example how you can substitute the threading with custom threads (in this case realtime threads for which no GThread API exists)
Comment 3 LRN 2009-06-07 18:07:20 UTC
Why would you? Just add pthreads as a requirement. With the patch it builds just fine (don't really know whether it actually works or not).

gst-plugins-bad does it like this:

AC_CHECK_HEADERS([pthread.h], HAVE_PTHREAD_H=yes)
AM_CONDITIONAL(HAVE_PTHREAD_H, test "x$HAVE_PTHREAD_H" = "xyes")

if test "x$HAVE_PTHREAD_H" != "xyes"; then
  GST_PLUGINS_SELECTED=`echo $GST_PLUGINS_SELECTED | $SED -e s/dccp//`
fi

I don't see why you can't do the same in core.
Comment 4 Wim Taymans 2009-06-07 20:22:33 UTC
Something like this (please reopen if it does not work)

commit fbaa49faf4a1122cf2d3fb4f4425a4e028b20460
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Sun Jun 7 22:20:54 2009 +0200

    tests: conditionally compile the streams example
    
    Detect pthreads.h in configure.ac
    Only compile the streams example when pthreads.h is present.
    
    Fixes #585039