Command fixes

fs/base.py

@@ -649,6 +649,8 @@ class FS(object):
         :rtype: An FS object
         """
         
+        if path in ('', '/'):
+            return self
         from fs.wrapfs.subfs import SubFS
         if not self.exists(path):
             raise ResourceNotFoundError(path)

fs/commands/fsmkdir.py

@@ -15,6 +15,7 @@ Make a directory"""
                 
         for fs_url in args:                    
             fs, path = self.open_fs(fs_url, create=True)
+            print fs, path                  
     
 def run():
     return FSMkdir().run()

fs/commands/fsmv.py

@@ -2,7 +2,7 @@ from fs.utils import movefile, contains_files
 from fs.commands import fscp
 import sys
 
-class FSMove(fscp.FSCopy):
+class FSMove(fscp.FScp):
     
     usage = """fsmv [OPTION]... [SOURCE] [DESTINATION]
 Move files from SOURCE to DESTINATION"""

fs/commands/fsserve.py

@@ -30,6 +30,13 @@ Serves the contents of PATH with one of a number of methods"""
             
         fs, path = self.open_fs(fs_url)
         
+        if fs.isdir(path):
+            fs = fs.opendir(path)
+            path = '/'
+        
+        if options.verbose:
+            print "Serving \"%s\" in %s" % (path, fs)
+                    
         port = options.port
             
         try:

fs/commands/fstree.py

@@ -13,7 +13,7 @@ Recursively display the contents of PATH in an ascii tree"""
     
     def get_optparse(self):
         optparse = super(FSTree, self).get_optparse()        
-        optparse.add_option('-L', '--level', dest='depth', type="int", default=5,
+        optparse.add_option('-l', '--level', dest='depth', type="int", default=5,
                             help="Descend only LEVEL directories deep", metavar="LEVEL")
         optparse.add_option('-a', '--all', dest='all', action='store_true', default=False,
                             help="do not hide dot files")        

fs/commands/runner.py

@@ -222,12 +222,10 @@ class Command(object):
             if self.is_terminal():
                 self.output("\n")
             return 0        
-        #except ValueError:
-        #    pass
         except SystemExit:
             return 0        
         except Exception, e:            
-            self.error(self.wrap_error('Internal Error - %s\n' % unicode(e)))
+            self.error(self.wrap_error('Error - %s\n' % unicode(e)))
             return 1
         
         

fs/opener.py

@@ -79,7 +79,7 @@ class OpenerRegistry(object):
         for name in opener.names:
             self.registry[name] = index
     
-    def parse(self, fs_url, default_fs_name=None, writeable=False, create=False):
+    def parse(self, fs_url, default_fs_name=None, open_dir=True, writeable=False, create=False):
                    
         orig_url = fs_url     
         match = self.split_segments(fs_url)
@@ -110,24 +110,24 @@ class OpenerRegistry(object):
         if fs_url is None:
             raise OpenerError("Unable to parse '%s'" % orig_url)
         
-        #wildcard = None
-        #if iswildcard(fs_url):
-        #    fs_url, wildcard = pathsplit(fs_url)
         
         fs, fs_path = opener.get_fs(self, fs_name, fs_name_params, fs_url, writeable, create)
         
+        if fs_path and iswildcard(fs_path):
             pathname, resourcename = pathsplit(fs_path or '')
             if pathname:
                 fs = fs.opendir(pathname)
-            fs_path = resourcename
+            return fs, resourcename
             
-        if fs_path and iswildcard(fs_path):
-            return fs, fs_path     
+        #pathname, resourcename = pathsplit(fs_path or '')
+        #if pathname and resourcename:
+        #    fs = fs.opendir(pathname)
+        #    fs_path = resourcename
                 
         fs_path = join(fs_path, path)
                 
         pathname, resourcename = pathsplit(fs_path or '')        
-        if pathname:
+        if pathname and resourcename:
             fs = fs.opendir(pathname)
             fs_path = resourcename
                        
@@ -180,17 +180,12 @@ class OSFSOpener(Opener):
         from fs.osfs import OSFS 
                                             
         path = _expand_syspath(fs_path)
-        if create:
+        if create and not os.path.exists(path):
             from fs.osfs import _os_makedirs                    
-            _os_makedirs(fs_path)
-        if os.path.isdir(path):
-            osfs = OSFS(path)
-            filepath = None
-        else:
-            path, filepath = pathsplit(path)
-            osfs = OSFS(path, create=create)
-        return osfs, filepath
-        
+            _os_makedirs(path)            
+        dirname, resourcename = pathsplit(fs_path)
+        osfs = OSFS(dirname)
+        return osfs, resourcename                
         
 class ZipOpener(Opener):
     names = ['zip', 'zip64']
@@ -212,7 +207,6 @@ class ZipOpener(Opener):
                     
         zip_file = zip_fs.open(zip_path, mode=open_mode)                                            
                         
-                        
         username, password, fs_path = registry.parse_credentials(fs_path)
         
         from fs.zipfs import ZipFS
@@ -387,7 +381,9 @@ opener = OpenerRegistry([OSFSOpener,
 
 def main():
     
-    fs, path = opener.parse('zip:zip://~/zips.zip!t.zip!')
+    #fs, path = opener.parse('zip:zip://~/zips.zip!t.zip!')
+    fs, path = opener.parse('rpc://127.0.0.1/a/*.JPG')
+    
     print fs, path
     
 if __name__ == "__main__":

fs/rpcfs.py

@@ -34,14 +34,11 @@ def re_raise_faults(func):
             bits = " ".join(bits[1:]).split(">:")
             cls = bits[0]            
             msg = ">:".join(bits[1:])            
-            while cls[0] in ["'",'"']:
-                cls = cls[1:]
-            while cls[-1] in ["'",'"']:
-                cls = cls[:-1]
+            cls = cls.strip('\'')        
             cls = _object_by_name(cls)
             # Re-raise using the remainder of the fault code as message
             if cls:                            
-                raise cls(msg)
+                raise cls(msg=msg)
             raise f
         except socket.error, e:
             raise RemoteConnectionError(str(e), details=e)

fs/utils.py

@@ -56,9 +56,6 @@ def copyfile(src_fs, src_path, dst_fs, dst_path, overwrite=True, chunk_size=64*1
     src = None        
     try:
         # Chunk copy
-        if src_fs.getsize(src_path) < chunk_size:
-            src = src_fs.getcontents(src_path)
-        else:
         src = src_fs.open(src_path, 'rb')        
         dst_fs.setcontents(dst_path, src, chunk_size=chunk_size)
     finally:
@@ -92,19 +89,15 @@ def movefile(src_fs, src_path, dst_fs, dst_path, overwrite=True, chunk_size=64*1
         FS._shutil_movefile(src_syspath, dst_syspath)        
         return
 
-
     src = None        
     try:
         # Chunk copy
-        if src_fs.getsize(src_path) < chunk_size:
-            src = src_fs.getcontents(src_path)
-        else:
         src = src_fs.open(src_path, 'rb')        
         dst_fs.setcontents(dst_path, src, chunk_size=chunk_size)            
-        src_fs.remove(src_path)
     finally:
         if src is not None and hasattr(src, 'close'):
             src.close()
+    src_fs.remove(src_path)
 
 def movedir(fs1, fs2, overwrite=False, ignore_errors=False, chunk_size=64*1024):
     """Moves contents of a directory from one filesystem to another.

fs/wrapfs/__init__.py

@@ -164,7 +164,12 @@ class WrapFS(FS):
         return self.wrapped_fs.isfile(self._encode(path))
 
     @rewrite_errors
-    def listdir(self, path="", **kwds):        
+    def listdir(self, path="", wildcard=None, full=False, absolute=False, dirs_only=False, files_only=False):
+        kwds = dict(wildcard=wildcard,
+                    full=full,
+                    absolute=absolute,
+                    dirs_only=dirs_only,
+                    files_only=files_only)        
         full = kwds.pop("full",False)
         absolute = kwds.pop("absolute",False)
         wildcard = kwds.pop("wildcard",None)

fs/wrapfs/subfs.py

@@ -30,16 +30,16 @@ class SubFS(WrapFS):
         return abspath(normpath(path))[len(self.sub_dir):]
 
     def __str__(self):
-        return "<SubFS: %s in %s>" % (self.sub_dir, self.wrapped_fs)
+        return "%s/%s" % (self.wrapped_fs, self.sub_dir.lstrip('/'))
 
     def __unicode__(self):
-        return u"<SubFS: %s in %s>" % (self.sub_dir, self.wrapped_fs)
+        return u"%s/%s" % (self.wrapped_fs, self.sub_dir.lstrip('/'))
 
     def __repr__(self):
         return str(self)
 
     def desc(self, path):        
-        desc = "%s in sub dir %s of %s" % (path, self.sub_dir, str(self.wrapped_fs))
+        desc = "%s!%s" % (str(self), path)
         return desc
         
     def setcontents(self, path, data, chunk_size=64*1024):