Changeset 03134ee in trunk

Timestamp:

2010-02-20T06:13:13Z (15 years ago)

Author:

david-sarah <david-sarah@…>

Branches:

master

Children:

9741b96

Parents:

b7c0248

Message:

Improve behaviour of 'tahoe ls' for unknown objects, addressing kevan's comments

Location:

src/allmydata/scripts

Files:

• cli.py (modified) (1 diff)
• common.py (modified) (1 diff)
• tahoe_ls.py (modified) (4 diffs)

src/allmydata/scripts/cli.py ¶

@@ -86 +86 @@
         self.where = where
 
-    longdesc = """List the contents of some portion of the grid."""
+    longdesc = """
+    List the contents of some portion of the grid.
+
+    When the -l or --long option is used, each line is shown in the
+    following format:
+
+    drwx <size> <date/time> <name in this directory>
+
+    where each of the letters on the left may be replaced by '-'.
+    If 'd' is present, it indicates that the object is a directory.
+    If the 'd' is replaced by a '?', the object type is unknown.
+    'rwx' is a Unix-like permissions mask: if the mask includes 'w',
+    then the object is writable through its link in this directory.
+    The 'x' is a legacy of Unix filesystems. In Tahoe it is used
+    only to indicate that the contents of a directory can be listed.
+
+    Directories have no size, so their size field is shown as '-'.
+    Otherwise the size of the file, when known, is given in bytes.
+    The size of mutable files or unknown objects is shown as '?'.
+
+    The date/time shows when this link in the Tahoe filesystem was
+    last modified.
+    """
 
 class GetOptions(VDriveOptions):

src/allmydata/scripts/common.py ¶

@@ -140 +140 @@
     path = path.strip()
     if uri.has_uri_prefix(path):
-        # The only way to get a sub-path is to use URI:blah:./foo, and we
-        # strip out the :./ sequence.
+        # We used to require "URI:blah:./foo" in order to get a subpath,
+        # stripping out the ":./" sequence. We still allow that for compatibility,
+        # but now also allow just "URI:blah/foo".
         sep = path.find(":./")
         if sep != -1:
             return path[:sep], path[sep+3:]
+        sep = path.find("/")
+        if sep != -1:
+            return path[:sep], path[sep+1:]
         return path, ""
     colon = path.find(":")

src/allmydata/scripts/tahoe_ls.py ¶

@@ -56 +56 @@
     if nodetype == "dirnode":
         children = d['children']
-    elif nodetype == "filenode":
+    else:
         childname = path.split("/")[-1]
         children = {childname: (nodetype, d)}
@@ -68 +68 @@
     # variable-width ones.
     rows = []
+    has_unknowns = False
 
     for name in childnames:
@@ -102 +103 @@
         elif childtype == "filenode":
             t0 = "-"
-            size = str(child[1]['size'])
+            size = str(child[1].get("size", "?"))
             classify = ""
             if rw_uri:
                 classify = "*"
         else:
+            has_unknowns = True
             t0 = "?"
             size = "?"
@@ -162 +164 @@
         print >>stdout, (fmt % tuple(row)).rstrip()
 
+    if has_unknowns:
+        print >>stderr, "\nThis listing included unknown objects. Using a webapi server that supports" \
+                        "\na later version of Tahoe may help."
+
     return 0