<div dir="ltr">Dear Thomas, <div><br></div><div>To be honest, I haven't even though about other libc. But you are right, that is probably a better idea. </div><div><br></div><div>What about adding those lines to <a href="http://configure.ac" target="_blank">configure.ac</a> ( modified from <a href="http://configure.ac" target="_blank">configure.ac</a> found in lttng-tools ) </div>

<div><br></div><div># Check for pthread</div><div>AC_CHECK_LIB([pthread], [pthread_create],</div><div><span style="white-space:pre-wrap">       </span>[AM_CONDITIONAL(LIBC_INCLUDES_PTHREAD, false)],</div><div><span style="white-space:pre-wrap">  </span>[AC_CHECK_LIB([c], [pthread_create],</div>

<div><span style="white-space:pre-wrap">          </span>[AM_CONDITIONAL(LIBC_INCLUDES_PTHREAD, true)],</div><div><span style="white-space:pre-wrap">           </span>[AC_MSG_ERROR([Cannot find libpthread. Use [LDFLAGS]=-Ldir to specify its location.])]</div>

<div><span style="white-space:pre-wrap">  </span>)]</div><div>)</div><div><br></div><div>and changing the if in the patch to "if !LIBC_INCLUDES_PTHREAD"</div><div><br></div><div><br></div><div>Thanks for the comment,</div>
<div>Charles</div><div><br></div><div><br></div><div><br></div><div>Mathieu, </div><div><br></div><div>   if I am to submit a patch for that, should it be intended to replace this one, or on top of HEAD as a Fix ?</div><div>
<br></div><div><br></div><div>Thanks,</div><div>Charles</div><div> </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 28, 2013 at 1:30 AM, Thomas Petazzoni <span dir="ltr"><<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">thomas.petazzoni@free-electrons.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Dear Charles Briere,<br>
<div class="im"><br>
On Wed, 27 Nov 2013 07:48:30 -0800, Charles Briere wrote:<br>
<br>
>  #Add the -version-info directly here since we are only building<br>
>  # library that use the version-info<br>
> -AM_LDFLAGS=-lpthread -version-info $(URCU_LIBRARY_VERSION)<br>
> +AM_LDFLAGS=-version-info $(URCU_LIBRARY_VERSION)<br>
> +if !TARGET_IS_ANDROID<br>
> +AM_LDFLAGS += -lpthread<br>
> +endif<br>
<br>
</div>Instead of making that conditional on Android, wouldn't it make more<br>
sense to make it conditional on whether linking with pthread is<br>
necessary or not?<br>
<br>
I can at least think of one other C library than Bionic that has<br>
everything in libc instead of things split in several libraries: the<br>
Musl C library (<a href="http://www.musl-libc.org" target="_blank">http://www.musl-libc.org</a>).<br>
<br>
Best regards,<br>
<br>
Thomas<br>
<span class="HOEnZb"><font color="#888888">--<br>
Thomas Petazzoni, CTO, Free Electrons<br>
Embedded Linux, Kernel and Android engineering<br>
<a href="http://free-electrons.com" target="_blank">http://free-electrons.com</a><br>
</font></span></blockquote></div><br></div>