[PATCH] lib/oe/unpack.py: unpack rar archives

[PATCH] lib/oe/unpack.py: unpack rar archives

[Andreas Oberritter]

Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
---
 lib/oe/unpack.py |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/lib/oe/unpack.py b/lib/oe/unpack.py
index 8e8bf36..fa517d8 100644
--- a/lib/oe/unpack.py
+++ b/lib/oe/unpack.py
@@ -96,6 +96,8 @@ def unpack_file(file, destdir, parameters, env=None):
             if dos:
                 cmd = '%s -a' % cmd
             cmd = "%s '%s'" % (cmd, file)
+        elif file.endswith('.rar'):
+            cmd = 'unrar x %s' % file
 
     if not unpack or not cmd:
         if os.path.isdir(file):
-- 
1.7.2.5

Re: [PATCH] lib/oe/unpack.py: unpack rar archives

[Paul Menzel]

[-- Attachment #1: Type: text/plain, Size: 942 bytes --]

Am Montag, den 16.05.2011, 11:41 +0000 schrieb Andreas Oberritter:
> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
> ---
>  lib/oe/unpack.py |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/lib/oe/unpack.py b/lib/oe/unpack.py
> index 8e8bf36..fa517d8 100644
> --- a/lib/oe/unpack.py
> +++ b/lib/oe/unpack.py
> @@ -96,6 +96,8 @@ def unpack_file(file, destdir, parameters, env=None):
>              if dos:
>                  cmd = '%s -a' % cmd
>              cmd = "%s '%s'" % (cmd, file)
> +        elif file.endswith('.rar'):
> +            cmd = 'unrar x %s' % file
>  
>      if not unpack or not cmd:
>          if os.path.isdir(file):

1. What program needs this? I am just asking to be able to test this.
2. Will `unrar-native` automatically be build when such an archive is
encountered?

Acked-by: Paul Menzel <paulepanter@users.sourceforge.net>


Thanks,

Paul

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH] lib/oe/unpack.py: unpack rar archives

[Andreas Oberritter]

On 05/16/2011 03:03 PM, Paul Menzel wrote:
> Am Montag, den 16.05.2011, 11:41 +0000 schrieb Andreas Oberritter:
>> Signed-off-by: Andreas Oberritter <obi@opendreambox.org>
>> ---
>>  lib/oe/unpack.py |    2 ++
>>  1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/lib/oe/unpack.py b/lib/oe/unpack.py
>> index 8e8bf36..fa517d8 100644
>> --- a/lib/oe/unpack.py
>> +++ b/lib/oe/unpack.py
>> @@ -96,6 +96,8 @@ def unpack_file(file, destdir, parameters, env=None):
>>              if dos:
>>                  cmd = '%s -a' % cmd
>>              cmd = "%s '%s'" % (cmd, file)
>> +        elif file.endswith('.rar'):
>> +            cmd = 'unrar x %s' % file
>>  
>>      if not unpack or not cmd:
>>          if os.path.isdir(file):
> 
> 1. What program needs this? I am just asking to be able to test this.

A recipe in a private layer.

> 2. Will `unrar-native` automatically be build when such an archive is
> encountered?

No. IIRC, adding that dependency created a non-trivial dependency
problem. I figured it would be preferrable to use the host's unrar.

> Acked-by: Paul Menzel <paulepanter@users.sourceforge.net>

Thanks,
Andreas