<div dir="ltr"><br>Hi,<div><br></div><div>I sent this patch to the list a couple of days ago following the instructions given previously. What&#39;s the state of the patch?</div><div>There were some discussions regarding wether it should be separated in two commits or not.</div>
<div><br></div><div>Could anybody inform me if the patch has been accepted?</div><div><br></div><div>Regards,</div><div><br></div><div>Víctor.<br><div class="gmail_quote"><br></div><div class="gmail_quote"><br><br>From: victor &lt;<a href="mailto:v.mayoralv@gmail.com">v.mayoralv@gmail.com</a>&gt;<br>

<br>
By default apr_cv_mutex_recursive in apr is set to &quot;no&quot; and this leads<br>
to the APRENOTIMPL return value of apr_thread_mutex_create in thread_mutex.c when<br>
APR_THREAD_MUTEX_NESTED is requested via flags which leads to a deadlock in applications<br>
using apr, as observed in an application with log4cxx.<br>
<br>
This commit adds CACHED_CONFIGUREVARS += &quot;apr_cv_mutex_recursive=yes&quot; to <a href="http://apr_1.4.6.bb" target="_blank">apr_1.4.6.bb</a><br>
and removes this variable in powerpc32-linux to address this issue.<br>
<br>
We detected this issue while working at the meta-ros project <a href="https://github.com/bmwcarit/meta-ros" target="_blank">https://github.com/bmwcarit/meta-ros</a><br>
(<a href="https://github.com/bmwcarit/meta-ros/issues/68" target="_blank">https://github.com/bmwcarit/meta-ros/issues/68</a>)<br>
<br>
Signed-off-by: Víctor Mayoral Vilches &lt;<a href="mailto:v.mayoralv@gmail.com">v.mayoralv@gmail.com</a>&gt;<br>
---<br>
 meta/recipes-support/apr/<a href="http://apr_1.4.6.bb" target="_blank">apr_1.4.6.bb</a> |    3 +++<br>
 meta/site/powerpc32-linux             |    1 -<br>
 2 files changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/meta/recipes-support/apr/<a href="http://apr_1.4.6.bb" target="_blank">apr_1.4.6.bb</a> b/meta/recipes-support/apr/<a href="http://apr_1.4.6.bb" target="_blank">apr_1.4.6.bb</a><br>
index 896f79f..ba59639 100644<br>
--- a/meta/recipes-support/apr/<a href="http://apr_1.4.6.bb" target="_blank">apr_1.4.6.bb</a><br>
+++ b/meta/recipes-support/apr/<a href="http://apr_1.4.6.bb" target="_blank">apr_1.4.6.bb</a><br>
@@ -23,6 +23,9 @@ inherit autotools lib_package binconfig multilib_header<br>
<br>
 OE_BINCONFIG_EXTRA_MANGLE = &quot; -e &#39;s:location=source:location=installed:&#39;&quot;<br>
<br>
+# Added to fix some issues with cmake. Refer to <a href="https://github.com/bmwcarit/meta-ros/issues/68#issuecomment-19896928" target="_blank">https://github.com/bmwcarit/meta-ros/issues/68#issuecomment-19896928</a><br>

+CACHED_CONFIGUREVARS += &quot;apr_cv_mutex_recursive=yes&quot;<br>
+<br>
 do_configure_prepend() {<br>
        cd ${S}<br>
        ./buildconf<br>
diff --git a/meta/site/powerpc32-linux b/meta/site/powerpc32-linux<br>
index 4550df3..b3973c9 100644<br>
--- a/meta/site/powerpc32-linux<br>
+++ b/meta/site/powerpc32-linux<br>
@@ -203,7 +203,6 @@ apr_cv_use_lfs64=${apr_cv_use_lfs64=yes}<br>
 apr_cv_epoll=${apr_cv_epoll=yes}<br>
 apr_cv_pthreads_cflags=${apr_cv_pthreads_cflags=-pthread}<br>
 apr_cv_pthreads_lib=${apr_cv_pthreads_lib=-lpthread}<br>
-apr_cv_mutex_recursive=${apr_cv_mutex_recursive=yes}<br>
 ac_cv_func_mmap=${ac_cv_func_mmap=yes}<br>
 ac_cv_file__dev_zero=${ac_cv_file__dev_zero=yes}<br>
 ac_cv_sizeof_off_t=${ac_cv_sizeof_off_t=4}<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.9.5<br>
<br>
</font></span></div><br></div></div>