<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Hi Tudor,</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="color:rgb(0,0,0);font-family:garamond,&quot;new york&quot;,times,serif;font-size:16px;background-color:rgb(255,255,255)"><div id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20479" dir="ltr">Hi Jagadeesh,<span class="gmail-"><br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20596"><br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20597">&gt; The patch include below changes:<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20598"></span>...<span class="gmail-"><br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20599">&gt; 2. Some of the libunwind test cases such as run-check-namespace,<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20600">&gt;    run-coredump-unwind require debug symbols to be present in<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20601">&gt;    libunwind library. So set INHIBIT_PACKAGE_STRIP to &quot;1&quot;.<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20602"></span>&gt; --- a/meta/recipes-support/<wbr>libunwind/<a href="http://libunwind_git.bb" target="_blank">libunwind_git.bb</a><br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20603">&gt; +++ b/meta/recipes-support/<wbr>libunwind/<a href="http://libunwind_git.bb" target="_blank">libunwind_git.bb</a><br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20604">...<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20605">&gt; +INHIBIT_PACKAGE_STRIP = &quot;1&quot;<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20606">&gt; +<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20607"><br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20608"></div></div></div></blockquote><div>Thanks for reviewing the patch.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="color:rgb(0,0,0);font-family:garamond,&quot;new york&quot;,times,serif;font-size:16px;background-color:rgb(255,255,255)"><div id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20479" dir="ltr">The change above have the effect of inhibiting the package strip even if ptest  is not enabled at all. I think this have to be something specific to ptest.<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20611">You may consider using INSANE_SKIP_${PN}-ptest [1] and add unstripped binaries needed for ptest in {PN}-ptest (along with the original - stripped - ones)</div></div></div></blockquote><div> </div><div><font face="arial, helvetica, sans-serif">I didn&#39;t get how unstripped and stripped binaries can co-exist in a single package called ${PN}-test.</font></div><div><font face="arial, helvetica, sans-serif">The split_and_strip_files function keeps files containing debug symbols in a separate directory</font></div><div><font face="arial, helvetica, sans-serif">( like ../.debug/..). Do you mean to say include files containing debug symbols (i,e </font><span style="font-family:arial,helvetica,sans-serif">debugdirectory) in ${PN}-test package and set  INSANE_SKIP_${PN}-ptest += &quot;</span><span style="font-family:arial,helvetica,sans-serif;color:rgb(51,51,51);line-height:24px"><code class="gmail-filename">debug-files</code></span><span style="font-family:arial,helvetica,sans-serif">&quot;</span></div><div><font face="arial, helvetica, sans-serif">to prevent packaging .debug directories in non -dbg package.</font></div><div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif">This doesn&#39;t help,  </font><span style="font-family:arial,helvetica,sans-serif">as run-coredump-unwind requires unstripped &quot;crasher&quot;  program to be present at target, </span><span style="font-family:arial,helvetica,sans-serif">to get procedure names using libuwind APIs.</span></div><div><span style="font-family:arial,helvetica,sans-serif"><br></span></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="color:rgb(0,0,0);font-family:garamond,&quot;new york&quot;,times,serif;font-size:16px;background-color:rgb(255,255,255)"><div id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20479" dir="ltr">or simply consider the appropriate tests as XFAIL for openembedded.<br id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20613"></div></div></div></blockquote><div>Yes, for such test (e.g: run-coredump-unwind) XFAIL is fine, if INHIBIT_PACKAGE_STRIP = &quot;1&quot; is</div><div>not used.</div><div><br></div><div>Regards,</div><div>Jagadeesh</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="color:rgb(0,0,0);font-family:garamond,&quot;new york&quot;,times,serif;font-size:16px;background-color:rgb(255,255,255)"><div id="gmail-m_8849950624476238808yui_3_16_0_ym19_1_1477346198248_20728"><br></div></div></div></blockquote></div><br></div></div>