[OE-core] [PATCH] package.bbclass: Fix split_and_strip_files when file has single quote (')

[Chris Larson]

On Thu, Oct 31, 2013 at 8:11 PM, Felipe Ferreri Tonello <
eu at felipetonello.com> wrote:

> On 10/31/2013 07:25 PM, eu at felipetonello.com wrote:
>
>> From: "Felipe F. Tonello" <eu at felipetonello.com>
>>
>> Fix false error report when a file that has a single quote by escaping
>> the single quote. Some packages might install files with quotes, such
>> as music files and other types, that will cause the problem.
>>
>> Signed-off-by: Felipe F. Tonello <eu at felipetonello.com>
>> ---
>>   meta/classes/package.bbclass | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
>> index b0f44c7..a12b7b7 100644
>> --- a/meta/classes/package.bbclass
>> +++ b/meta/classes/package.bbclass
>> @@ -747,7 +747,7 @@ python split_and_strip_files () {
>>       # 16 - kernel module
>>       def isELF(path):
>>           type = 0
>> -        ret, result = oe.utils.getstatusoutput("file '%s'" % path)
>> +        ret, result = oe.utils.getstatusoutput("file '%s'" %
>> path.replace("'", "\\'"))
>>
>>           if ret:
>>               msg = "split_and_strip_files: 'file %s' failed" % path
>>
>>
>
> I just found a bug in this implementation. I will send a v2 shortly.


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 :)
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20131031/686bab96/attachment-0002.html>