<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 31, 2013 at 8:11 PM, Felipe Ferreri Tonello <span dir="ltr">&lt;<a href="mailto:eu@felipetonello.com" target="_blank">eu@felipetonello.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 10/31/2013 07:25 PM, <a href="mailto:eu@felipetonello.com" target="_blank">eu@felipetonello.com</a> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
From: &quot;Felipe F. Tonello&quot; &lt;<a href="mailto:eu@felipetonello.com" target="_blank">eu@felipetonello.com</a>&gt;<br>
<br>
Fix false error report when a file that has a single quote by escaping<br>
the single quote. Some packages might install files with quotes, such<br>
as music files and other types, that will cause the problem.<br>
<br>
Signed-off-by: Felipe F. Tonello &lt;<a href="mailto:eu@felipetonello.com" target="_blank">eu@felipetonello.com</a>&gt;<br>
---<br>
  meta/classes/package.<span class="il">bbclass</span> | 2 +-<br>
  1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/meta/classes/package.<span class="il">bbclass</span> b/meta/classes/package.<span class="il">bbclass</span><br>
index b0f44c7..a12b7b7 100644<br>
--- a/meta/classes/package.<span class="il">bbclass</span><br>
+++ b/meta/classes/package.<span class="il">bbclass</span><br>
@@ -747,7 +747,7 @@ python split_and_strip_files () {<br>
      # 16 - kernel module<br>
      def isELF(path):<br>
          type = 0<br>
-        ret, result = oe.utils.getstatusoutput(&quot;file &#39;%s&#39;&quot; % path)<br>
+        ret, result = oe.utils.getstatusoutput(&quot;file &#39;%s&#39;&quot; % path.replace(&quot;&#39;&quot;, &quot;\\&#39;&quot;))<br>
<br>
          if ret:<br>
              msg = &quot;split_and_strip_files: &#39;file %s&#39; failed&quot; % path<br>
<br>
</blockquote>
<br>
<br></div>
I just found a bug in this implementation. I will send a v2 shortly.</blockquote></div><br>This should really be fixed to use a better method. In this case we’re just running a single binary with one argument, there’s no need to call out to a shell at all, much less have to mess around with escaping arguments :)<br>

-- <br>Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics
</div></div>