From: Maciej Tronowski <mtro@man.poznan.pl>
Date: Tue, 14 Apr 2015 09:51:32 +0000 (+0200)
Subject: fix handling zip archives
X-Git-Tag: v1.0~71
X-Git-Url: http://mmka.chem.univ.gda.pl/gitweb/?a=commitdiff_plain;h=67ec532d7a614cd39cf1e4d36a4ba963c9e166ee;hp=273c0546c3f3154497c6554efeba805d9bbde526;p=qcg-portal.git

fix handling zip archives
---

diff --git a/filex/ftp.py b/filex/ftp.py
index 39ee807..49f470f 100644
--- a/filex/ftp.py
+++ b/filex/ftp.py
@@ -1,5 +1,6 @@
 from datetime import datetime
 from Queue import Queue, Empty
+from itertools import chain
 import os
 import re
 from threading import Event
@@ -167,7 +168,8 @@ class FTPOperation:
         elif self.match_ext(archive, '.tar.bz2', '.tbz'):
             cmd, args = 'tar', ['cvjf', os.path.join(path, archive), '-C', path] + files
         elif self.match_ext(archive, '.zip'):
-            cmd, args = 'zip', ['-r', os.path.join(path, archive)] + [os.path.join(path, f) for f in files]
+            cmd, args = 'jar', (['cvMf', os.path.join(path, archive)] +
+                                list(chain.from_iterable(('-C', path, f) for f in files)))
         else:
             raise ValueError('Unknown archive type: {}'.format(archive))
 
@@ -178,15 +180,15 @@ class FTPOperation:
 
     def extract(self, server, archive, dst):
         if self.match_ext(archive, '.tar.gz', '.tgz'):
-            cmd, args = 'tar', ['xvzf', archive, '-C', dst]
+            cmd, args = 'tar', ('xvzf', archive, '-C', dst)
         elif self.match_ext(archive, '.tar.bz2', '.tbz'):
-            cmd, args = 'tar', ['xvjf', archive, '-C', dst]
+            cmd, args = 'tar', ('xvjf', archive, '-C', dst)
         elif self.match_ext(archive, '.zip'):
-            cmd, args = 'unzip', [archive, '-d', dst]
+            cmd, args = 'unzip', (archive, '-d', dst)
         else:
             raise ValueError('Unknown archive type: {}'.format(archive))
 
         # FIXME handling filename with #
-        self.op_attr.set_disk_stack('#'.join(["popen:argv=", cmd] + args))
+        self.op_attr.set_disk_stack('#'.join(("popen:argv=", cmd) + args))
 
         return self.get(server)