<div dir="ltr"><div>I'm confused what the desired cleanup is.  Do you want  </div><div><br></div><div>1) PKG_CHECK_MODULES(PTHREAD, [pthread]) in <a href="http://configure.ac" target="_blank">configure.ac</a> </div><div>or</div><div>2) All references to PTHREAD_LIBS in the <a href="http://configure.ac" target="_blank">configure.ac</a> and the makefiles to be removed.  In the cases where this breaks the build, PTHREAD_CFLAGS should be added to the AM_LDFLAGS. </div><div> <br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 28, 2016 at 11:59 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On 28 October 2016 at 19:35, Gurchetan Singh<br>
<span><<a href="mailto:gurchetansingh@chromium.org" target="_blank">gurchetansingh@chromium.org</a>> wrote:<br>
> Removing the entire hunk causes build failures on CrOS mesa.  Refer to this<br>
> for more information:<br>
><br>
> <a href="http://stackoverflow.com/questions/17055279/autotools-for-pthreads-not-setting-correct-linker-flags" rel="noreferrer" target="_blank">http://stackoverflow.com/quest<wbr>ions/17055279/autotools-for-<wbr>pthreads-not-setting-correct-<wbr>linker-flags</a><br>
><br>
> We can do one of the following:<br>
><br>
> (1) Keep as is, with special case for android<br>
> (2) Set PTHREAD_LIBS=$PTHREAD_CFLAGS in <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
> (3) Do a giant sed to convert PTHREAD_LIBS to PTHREAD_CFLAGS in the entire<br>
> tree<br>
><br>
> Any strong preferences???<br>
><br>
</span>As mentioned before - I'm pretty sure we have places where we've<br>
forgot the cflags/libs.<br>
<br>
It's an issue on our end, so instead of working around it (by using<br>
PTHREAD_CFLAGS as PTHREAD_LIBS) we could fix it.<br>
Can you please give it a try ?<br>
<br>
Thanks<br>
<span class="gmail-m_2393458105227287803HOEnZb"><font color="#888888">Emil<br>
</font></span></blockquote></div><br></div></div>