← Back to branch summary

~azzar1/unity/add-show-desktop-key

« back to all changes in this revision

Viewing changes to ivle/fileservice_lib/listing.py

  • Committer: Matt Giuca
  • Date: 2010-02-18 03:57:33 UTC
  • Revision ID: matt.giuca@gmail.com-20100218035733-d4v0iy4hq0wbjas6
https://launchpad.net/bugs/523592
ivle.fileservice_lib.listing: Previously assumed any SVN client error meant the directory was not versioned, and silently dropped SVN metadata. Now checks the error message, and for any unexpected errors, raises the exception rather than assuming unversioned. Fixes Launchpad Bug #523592.

Show diffs side-by-side

added added

removed removed

Lines of Context:
ivle/fileservice_lib/listing.py
263
263
            for status in status_list:
264
264
                filename, attrs = PysvnStatus_to_fileinfo(path, status)
265
265
                listing[filename.decode('utf-8')] = attrs
266
 
    except pysvn.ClientError:
267
 
        # Presumably the directory is not under version control.
 
266
    except pysvn.ClientError, e:
 
267
        # Could indicate a serious SVN error, or just that the directory is
 
268
        # not under version control (which is perfectly normal).
 
269
        # XXX Unfortunately, the only way to tell is to inspect the message
 
270
        if not str(e).endswith("is not a working copy"):
 
271
            # This is a serious error -- let it propagate upwards
 
272
            raise
 
273
        # The directory is not under version control.
268
274
        # Fallback to just an OS file listing.
269
275
        try:
270
276
            for filename in os.listdir(path):