[vlc-devel] Re: [PATCH] add THREAD_LIB to configure test for x264

[Jean-Paul Saman]

Applied thank you for your contribution.

Greetings,
Jean-Paul Saman.

Anssi Hannula wrote:
> Hi!
> 
> x264 needs to be linked with -lpthread in vlc's configure script if x264
> is built with pthread support.
> 
> The attached patch adds $THREAD_LIB to LDFLAGS for the configure test of
> x264. It doesn't affect non-pthread builds of x264.
> Please apply.
> 
> Please CC me on replies, as I'm not subscribed.
> 
> 
> 
> ------------------------------------------------------------------------
> 
> --- configure.ac	2005-11-04 18:31:33.000000000 +0200
> +++ configure.ac.new	2005-11-11 19:46:15.000000000 +0200
> @@ -2879,7 +2879,7 @@ if test "${enable_x264}" != "no"; then
>        AC_MSG_RESULT(yes)
>        VLC_ADD_CPPFLAGS([x264],[-I${real_x264_tree}])
>        VLC_ADD_LDFLAGS([x264],[-L${real_x264_tree}])
> -      LDFLAGS="${LDFLAGS_save} ${LDFLAGS_x264}"
> +      LDFLAGS="${LDFLAGS_save} ${LDFLAGS_x264} ${THREAD_LIB}"
>        AC_CHECK_LIB(x264, x264_encoder_open, [
>          VLC_ADD_BUILTINS([x264])
>          VLC_ADD_LDFLAGS([x264],[-lx264])
> @@ -2892,7 +2892,7 @@ if test "${enable_x264}" != "no"; then
>        AC_MSG_ERROR([the specified tree doesn't have x264.h])
>      fi
>    else
> -    LDFLAGS="${LDFLAGS_save} ${LDFLAGS_x264}"
> +    LDFLAGS="${LDFLAGS_save} ${LDFLAGS_x264} ${THREAD_LIB}"
>      AC_CHECK_HEADERS(x264.h, [
>        AC_CHECK_LIB(x264, x264_encoder_open, [
>          VLC_ADD_PLUGINS([x264])

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html