]> arthur.barton.de Git - bup.git/commitdiff
Use Python 3 compatible "except ... as e" syntax
authorRob Browning <rlb@defaultvalue.org>
Tue, 29 Dec 2015 07:11:19 +0000 (01:11 -0600)
committerRob Browning <rlb@defaultvalue.org>
Fri, 1 Jan 2016 19:43:07 +0000 (13:43 -0600)
Signed-off-by: Rob Browning <rlb@defaultvalue.org>
Tested-by: Rob Browning <rlb@defaultvalue.org>
35 files changed:
cmd/cat-file-cmd.py
cmd/daemon-cmd.py
cmd/fsck-cmd.py
cmd/ftp-cmd.py
cmd/help-cmd.py
cmd/index-cmd.py
cmd/init-cmd.py
cmd/join-cmd.py
cmd/list-idx-cmd.py
cmd/memtest-cmd.py
cmd/midx-cmd.py
cmd/on-cmd.py
cmd/restore-cmd.py
cmd/save-cmd.py
cmd/server-cmd.py
cmd/split-cmd.py
cmd/tag-cmd.py
cmd/web-cmd.py
cmd/xstat-cmd.py
lib/bup/client.py
lib/bup/drecurse.py
lib/bup/git.py
lib/bup/helpers.py
lib/bup/hlinkdb.py
lib/bup/index.py
lib/bup/ls.py
lib/bup/metadata.py
lib/bup/options.py
lib/bup/t/tbloom.py
lib/bup/t/tgit.py
lib/bup/t/thelpers.py
lib/bup/t/tmetadata.py
lib/bup/xstat.py
main.py
wvtest.py

index 735a2398022b4e64d3c7d4a645a83fd311cab2b0..d8a3dc559cc553e6f11abf3d9067a383d50786c5 100755 (executable)
@@ -37,7 +37,7 @@ if not re.match(r'/*[^/]+/[^/]+', target):
 
 try:
     n = top.lresolve(target)
-except vfs.NodeError, e:
+except vfs.NodeError as e:
     o.fatal(e)
 
 if isinstance(n, vfs.FakeSymlink):
index 5149c1602bb166640baee39506e29abf29867f26..8386fc0e120983252da06dbfe80c8b79a98c4ddb 100755 (executable)
@@ -30,7 +30,7 @@ for res in socket.getaddrinfo(host, port, socket.AF_UNSPEC,
     af, socktype, proto, canonname, sa = res
     try:
         s = socket.socket(af, socktype, proto)
-    except socket.error, e:
+    except socket.error as e:
         continue
     try:
         if af == socket.AF_INET6:
@@ -41,7 +41,7 @@ for res in socket.getaddrinfo(host, port, socket.AF_UNSPEC,
         s.bind(sa)
         s.listen(1)
         fcntl.fcntl(s.fileno(), fcntl.F_SETFD, fcntl.FD_CLOEXEC)
-    except socket.error, e:
+    except socket.error as e:
         s.close()
         continue
     socks.append(s)
index a5ab754c131f1715f89e93a86111e10961718f9a..94eeb89f4ae52bb429a725a2d28fb080a59fb581 100755 (executable)
@@ -76,7 +76,7 @@ def git_verify(base):
     if opt.quick:
         try:
             quick_verify(base)
-        except Exception, e:
+        except Exception as e:
             debug('error: %s\n' % e)
             return 1
         return 0
@@ -201,7 +201,7 @@ for name in extra:
         else: # child
             try:
                 sys.exit(do_pack(base, last, par2_exists))
-            except Exception, e:
+            except Exception as e:
                 log('exception: %r\n' % e)
                 sys.exit(99)
                 
index 19352668ab28a42e9a0fddd4dcac28d62f2d8121..908932bfd49e9b3ac600153fcd1ad8adb5b15e6f 100755 (executable)
@@ -19,7 +19,7 @@ class OptionError(Exception):
 def do_ls(cmd_args):
     try:
         ls.do_ls(cmd_args, pwd, onabort=OptionError)
-    except OptionError, e:
+    except OptionError as e:
         return
 
 
@@ -49,7 +49,7 @@ def _completer_get_subs(line):
         n = pwd.resolve(dir)
         subs = list(filter(lambda x: x.name.startswith(name),
                            n.subs()))
-    except vfs.NoSuchFile, e:
+    except vfs.NoSuchFile as e:
         subs = []
     return (dir, name, qtype, lastword, subs)
 
@@ -115,12 +115,12 @@ def completer(text, state):
                 ret = shquote.what_to_add(qtype, lastword, fullname,
                                           terminate=True) + ' '
             return text + ret
-    except Exception, e:
+    except Exception as e:
         log('\n')
         try:
             import traceback
             traceback.print_tb(sys.exc_traceback)
-        except Exception, e2:
+        except Exception as e2:
             log('Error printing traceback: %s\n' % e2)
         log('\nError in completion: %s\n' % e)
 
@@ -198,7 +198,7 @@ for line in lines:
                             outf = open(n.name, 'wb')
                             write_to_file(inf, outf)
                             outf.close()
-                        except Exception, e:
+                        except Exception as e:
                             rv = 1
                             log('  error: %s\n' % e)
         elif cmd == 'help' or cmd == '?':
@@ -208,7 +208,7 @@ for line in lines:
         else:
             rv = 1
             raise Exception('no such command %r' % cmd)
-    except Exception, e:
+    except Exception as e:
         rv = 1
         log('error: %s\n' % e)
         #raise
index 2efc729a2e3eb41b10154880b0174352c504b1df..95155b6ad28d8dde6904367e484ffafb0edc086c 100755 (executable)
@@ -26,7 +26,7 @@ elif len(extra) == 1:
             os.execvp('man', ['man', '-l', g[0]])
         else:
             os.execvp('man', ['man', docname])
-    except OSError, e:
+    except OSError as e:
         sys.stderr.write('Unable to run man command: %s\n' % e)
         sys.exit(1)
 else:
index e416ac5c988baf04fc39b2be336b104589f5b061..73ef55820b5dd95e56e8b7245aea27868d140e14 100755 (executable)
@@ -62,7 +62,7 @@ def clear_index(indexfile):
             os.remove(path)
             if opt.verbose:
                 log('clear: removed %s\n' % path)
-        except OSError, e:
+        except OSError as e:
             if e.errno != errno.ENOENT:
                 raise
 
@@ -111,7 +111,7 @@ def update_index(top, excluded_paths, exclude_rxs):
         if rig.cur and rig.cur.name == path:    # paths that already existed
             try:
                 meta = metadata.from_path(path, statinfo=pst)
-            except (OSError, IOError), e:
+            except (OSError, IOError) as e:
                 add_error(e)
                 rig.next()
                 continue
@@ -146,7 +146,7 @@ def update_index(top, excluded_paths, exclude_rxs):
         else:  # new paths
             try:
                 meta = metadata.from_path(path, statinfo=pst)
-            except (OSError, IOError), e:
+            except (OSError, IOError) as e:
                 add_error(e)
                 continue
             # See same assignment to 0, above, for rationale.
index e6cc6b51001867aac79494b1501230068a5e1c8d..fabd98c2d1711affba816b7742f3ff3ca30c93db 100755 (executable)
@@ -24,7 +24,7 @@ if extra:
 
 try:
     git.init_repo()  # local repo
-except git.GitError, e:
+except git.GitError as e:
     log("bup: error: could not init repository: %s" % e)
     sys.exit(1)
 
index 347206da86f5fde814b86e7e2c3f07b7e50ad56d..8ad322d151d6ad3565eef38fd8c23753aa7e837b 100755 (executable)
@@ -41,7 +41,7 @@ for id in extra:
     try:
         for blob in cat(id):
             outfile.write(blob)
-    except KeyError, e:
+    except KeyError as e:
         outfile.flush()
         log('error: %s\n' % e)
         ret = 1
index 9eb92c177971e474a9fc0307a845a5b8d2b080f6..3bab6b3fc3fee580df45cc57b97dcfec5a833119 100755 (executable)
@@ -40,7 +40,7 @@ count = 0
 for name in extra:
     try:
         ix = git.open_idx(name)
-    except git.GitError, e:
+    except git.GitError as e:
         add_error('%s: %s' % (name, e))
         continue
     if len(opt.find) == 40:
index 65778b73de374829c1a8eb30e7b74a0c61bac60a..245bfc99fb89ff4ecb95c0932032a537ceba8c63 100755 (executable)
@@ -17,7 +17,7 @@ def linux_memstat():
     d = {}
     try:
         f = open('/proc/self/status')
-    except IOError, e:
+    except IOError as e:
         if not _linux_warned:
             log('Warning: %s\n' % e)
             _linux_warned = 1
index eb558f88ca8c0defc932bc2c5ddd8531a600e172..d867ad6064f72ffd2484a59c1bffdf10ef463fbf 100755 (executable)
@@ -46,7 +46,7 @@ def check_midx(name):
     log('Checking %s.\n' % nicename)
     try:
         ix = git.open_idx(name)
-    except git.GitError, e:
+    except git.GitError as e:
         add_error('%s: %s' % (name, e))
         return
     for count,subname in enumerate(ix.idxnames):
index df0fdb3158faaf6091bf8977dd054764b66313e6..f76ad3e1e06a71fd07de06db586f1827d92d1a97 100755 (executable)
@@ -63,11 +63,11 @@ try:
                 if sp:
                     sp.wait()
                 break
-            except SigException, e:
+            except SigException as e:
                 log('\nbup on: %s\n' % e)
                 os.kill(p.pid, e.signum)
                 ret = 84
-except SigException, e:
+except SigException as e:
     if ret == 0:
         ret = 99
     log('\nbup on: %s\n' % e)
index 49f51761b6ae2911665618dc51ff0af089e7fe90..afcc3a444940e266a4312744f1336b043e9be64b 100755 (executable)
@@ -151,7 +151,7 @@ def hardlink_if_possible(fullname, node, meta):
                 try:
                     os.link(target_path, fullname)
                     return True
-                except OSError, e:
+                except OSError as e:
                     if e.errno != errno.EXDEV:
                         raise
     else:
@@ -319,7 +319,7 @@ for d in extra:
     path,name = os.path.split(d)
     try:
         n = top.lresolve(d)
-    except vfs.NodeError, e:
+    except vfs.NodeError as e:
         add_error(e)
         continue
     isdir = stat.S_ISDIR(n.mode)
index 664b580176a66c888be1634067abc769255b320e..e48170a667c1629e222d32e59b5a76b97b3a98f3 100755 (executable)
@@ -81,7 +81,7 @@ refname = opt.name and 'refs/heads/%s' % opt.name or None
 if opt.remote or is_reverse:
     try:
         cli = client.Client(opt.remote)
-    except client.ClientError, e:
+    except client.ClientError as e:
         log('error: %s' % e)
         sys.exit(1)
     oldref = refname and cli.read_ref(refname) or None
@@ -213,7 +213,7 @@ indexfile = opt.indexfile or git.repo('bupindex')
 r = index.Reader(indexfile)
 try:
     msr = index.MetaStoreReader(indexfile + '.meta')
-except IOError, ex:
+except IOError as ex:
     if ex.errno != EACCES:
         raise
     log('error: cannot access %r; have you run bup index?' % indexfile)
@@ -342,7 +342,7 @@ for (transname,ent) in r.filter(extra, wantrecurse=wantrecurse_during):
         # Not indexed, so just grab the FS metadata or use empty metadata.
         try:
            meta = metadata.from_path(fs_path) if fs_path else metadata.Metadata()
-        except (OSError, IOError), e:
+        except (OSError, IOError) as e:
             add_error(e)
             lastskip_name = dir_name
             meta = metadata.Metadata()
@@ -381,7 +381,7 @@ for (transname,ent) in r.filter(extra, wantrecurse=wantrecurse_during):
         if stat.S_ISREG(ent.mode):
             try:
                 f = hashsplit.open_noatime(ent.name)
-            except (IOError, OSError), e:
+            except (IOError, OSError) as e:
                 add_error(e)
                 lastskip_name = ent.name
             else:
@@ -389,7 +389,7 @@ for (transname,ent) in r.filter(extra, wantrecurse=wantrecurse_during):
                     (mode, id) = hashsplit.split_to_blob_or_tree(
                                             w.new_blob, w.new_tree, [f],
                                             keep_boundaries=False)
-                except (IOError, OSError), e:
+                except (IOError, OSError) as e:
                     add_error('%s: %s' % (ent.name, e))
                     lastskip_name = ent.name
         else:
@@ -398,7 +398,7 @@ for (transname,ent) in r.filter(extra, wantrecurse=wantrecurse_during):
             elif stat.S_ISLNK(ent.mode):
                 try:
                     rl = os.readlink(ent.name)
-                except (OSError, IOError), e:
+                except (OSError, IOError) as e:
                     add_error(e)
                     lastskip_name = ent.name
                 else:
@@ -419,7 +419,7 @@ for (transname,ent) in r.filter(extra, wantrecurse=wantrecurse_during):
             hlink = find_hardlink_target(hlink_db, ent)
             try:
                 meta = metadata.from_path(ent.name, hardlink_target=hlink)
-            except (OSError, IOError), e:
+            except (OSError, IOError) as e:
                 add_error(e)
                 lastskip_name = ent.name
             else:
index cf7be261c343ab28f720a9feee193ea842a7b3fa..520ea649585ab63d3318fd86fd290848c4072e41 100755 (executable)
@@ -158,7 +158,7 @@ def cat(conn, id):
         for blob in cat_pipe.join(id):
             conn.write(struct.pack('!I', len(blob)))
             conn.write(blob)
-    except KeyError, e:
+    except KeyError as e:
         log('server: error: %s\n' % e)
         conn.write('\0\0\0\0')
         conn.error(e)
index b7c7c86e0e00d24aec141ef2193df76eb43a2838..2ad5721853eff6530102b45dca8e13fc258064c5 100755 (executable)
@@ -125,7 +125,7 @@ if opt.git_ids:
             try:
                 it = cp.get(line.strip())
                 next(it, None)  # skip the file type
-            except KeyError, e:
+            except KeyError as e:
                 add_error('error: %s' % e)
                 continue
             yield IterToFile(it)
index f12ea4e473a4349cf13b24f19c087a30be8ca2a5..f065904d78397c3c66e34a988f2bf560b866f621 100755 (executable)
@@ -64,7 +64,7 @@ if tag_name.startswith('.'):
 
 try:
     hash = git.rev_parse(commit)
-except git.GitError, e:
+except git.GitError as e:
     log("bup: error: %s" % e)
     sys.exit(2)
 
@@ -80,7 +80,7 @@ if not pL.exists(hash):
 tag_file = git.repo('refs/tags/%s' % tag_name)
 try:
     tag = file(tag_file, 'w')
-except OSError, e:
+except OSError as e:
     log("bup: error: could not create tag '%s': %s" % (tag_name, e))
     sys.exit(3)
 
index acbbb802a5da1c191caf89353af51370b3665dae..640e4ae8de3126983b414602b7d3504e5020aad0 100755 (executable)
@@ -110,7 +110,7 @@ class BupRequestHandler(tornado.web.RequestHandler):
 
         try:
             show_hidden = int(self.request.arguments.get('hidden', [0])[-1])
-        except ValueError, e:
+        except ValueError as e:
             show_hidden = False
 
         self.render(
@@ -229,7 +229,7 @@ if __name__ == "__main__":
 
     try:
         sock = http_server._socket # tornado < 2.0
-    except AttributeError, e:
+    except AttributeError as e:
         sock = http_server._sockets.values()[0]
 
     print "Serving HTTP on %s:%d..." % sock.getsockname()
index c977d1c62ee5c8dc5fc56501e587456c66030eea..f299cbca7bf5a1ce12e745ad98a54f515c4fdd59 100755 (executable)
@@ -18,7 +18,7 @@ def parse_timestamp_arg(field, value):
     res = str(value) # Undo autoconversion.
     try:
         res = parse_timestamp(res)
-    except ValueError, ex:
+    except ValueError as ex:
         if ex.args:
             o.fatal('unable to parse %s resolution "%s" (%s)'
                     % (field, value, ex))
@@ -82,7 +82,7 @@ first_path = True
 for path in remainder:
     try:
         m = metadata.from_path(path, archive_path = path)
-    except (OSError,IOError), e:
+    except (OSError,IOError) as e:
         if e.errno == errno.ENOENT:
             add_error(e)
             continue
index 54fe9d59d3d318ce14a0a7285f069dcf7affcd9e..d819e701f77629e2609399c1509d023babf9ec1f 100644 (file)
@@ -75,7 +75,7 @@ class Client:
                     self.pout = self.p.stdout
                     self.pin = self.p.stdin
                     self.conn = Conn(self.pout, self.pin)
-                except OSError, e:
+                except OSError as e:
                     raise ClientError, 'connect: %s' % e, sys.exc_info()[2]
             elif self.protocol == 'bup':
                 self.sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
@@ -94,7 +94,7 @@ class Client:
     def __del__(self):
         try:
             self.close()
-        except IOError, e:
+        except IOError as e:
             if e.errno == errno.EPIPE:
                 pass
             else:
@@ -130,7 +130,7 @@ class Client:
                                   % rv)
         try:
             return self.conn.check_ok()
-        except Exception, e:
+        except Exception as e:
             raise ClientError, e, sys.exc_info()[2]
 
     def check_busy(self):
@@ -332,7 +332,7 @@ class PackWriter_Remote(git.PackWriter):
         try:
             (self._bwcount, self._bwtime) = _raw_write_bwlimit(
                     self.file, outbuf, self._bwcount, self._bwtime)
-        except IOError, e:
+        except IOError as e:
             raise ClientError, e, sys.exc_info()[2]
         self.outbytes += len(data)
         self.count += 1
index 6be7fa2ff38f8ff7bd6de1d87e9536372c9448fe..de8e5aa704888bb4c5d2e120d53f4af69d5d371f 100644 (file)
@@ -39,7 +39,7 @@ def _dirlist():
     for n in os.listdir('.'):
         try:
             st = xstat.lstat(n)
-        except OSError, e:
+        except OSError as e:
             add_error(Exception('%s: %s' % (realpath(n), str(e))))
             continue
         if (st.st_mode & _IFMT) == stat.S_IFDIR:
@@ -70,7 +70,7 @@ def _recursive_dirlist(prepend, xdev, bup_dir=None,
             else:
                 try:
                     OsFile(name).fchdir()
-                except OSError, e:
+                except OSError as e:
                     add_error('%s: %s' % (prepend, e))
                 else:
                     for i in _recursive_dirlist(prepend=prepend+name, xdev=xdev,
@@ -93,12 +93,12 @@ def recursive_dirlist(paths, xdev, bup_dir=None, excluded_paths=None,
                 if stat.S_ISLNK(pst.st_mode):
                     yield (path, pst)
                     continue
-            except OSError, e:
+            except OSError as e:
                 add_error('recursive_dirlist: %s' % e)
                 continue
             try:
                 pfile = OsFile(path)
-            except OSError, e:
+            except OSError as e:
                 add_error(e)
                 continue
             pst = pfile.stat()
index dbbcbffea63b4c599b3206310fdc72159e1aa229..a79b9bd9f8f676fe05e3befdf18ce33162dba775 100644 (file)
@@ -129,7 +129,7 @@ def auto_midx(objdir):
     args = [path.exe(), 'midx', '--auto', '--dir', objdir]
     try:
         rv = subprocess.call(args, stdout=open('/dev/null', 'w'))
-    except OSError, e:
+    except OSError as e:
         # make sure 'args' gets printed to help with debugging
         add_error('%r: exception: %s' % (args, e))
         raise
@@ -139,7 +139,7 @@ def auto_midx(objdir):
     args = [path.exe(), 'bloom', '--dir', objdir]
     try:
         rv = subprocess.call(args, stdout=open('/dev/null', 'w'))
-    except OSError, e:
+    except OSError as e:
         # make sure 'args' gets printed to help with debugging
         add_error('%r: exception: %s' % (args, e))
         raise
@@ -502,7 +502,7 @@ class PackIdxList:
                 if not d.get(full):
                     try:
                         ix = open_idx(full)
-                    except GitError, e:
+                    except GitError as e:
                         add_error(e)
                         continue
                     d[full] = ix
@@ -594,7 +594,7 @@ class PackWriter:
         oneblob = ''.join(datalist)
         try:
             f.write(oneblob)
-        except IOError, e:
+        except IOError as e:
             raise GitError, e, sys.exc_info()[2]
         nw = len(oneblob)
         crc = zlib.crc32(oneblob) & 0xffffffff
@@ -953,7 +953,7 @@ def check_repo_or_die(path=None):
     guess_repo(path)
     try:
         os.stat(repo('objects/pack/.'))
-    except OSError, e:
+    except OSError as e:
         if e.errno == errno.ENOENT:
             log('error: %r is not a bup repository; run "bup init"\n'
                 % repo())
@@ -1015,7 +1015,7 @@ class _AbortableIter:
     def next(self):
         try:
             return self.it.next()
-        except StopIteration, e:
+        except StopIteration as e:
             self.done = True
             raise
         except:
@@ -1100,7 +1100,7 @@ class CatPipe:
             readline_result = self.p.stdout.readline()
             assert(readline_result == '\n')
             self.inprogress = None
-        except Exception, e:
+        except Exception as e:
             it.abort()
             raise
 
index 3c4cdb20b16099b602f09e7a4272fd7940aadcd9..5bae5dee943d477a97f6dd65c73cc154eec089f8 100644 (file)
@@ -59,7 +59,7 @@ def _hard_write(fd, buf):
             raise IOError('select(fd) returned without being writable')
         try:
             sz = os.write(fd, buf)
-        except OSError, e:
+        except OSError as e:
             if e.errno != errno.EAGAIN:
                 raise
         assert(sz >= 0)
@@ -129,7 +129,7 @@ def mkdirp(d, mode=None):
             os.makedirs(d, mode)
         else:
             os.makedirs(d)
-    except OSError, e:
+    except OSError as e:
         if e.errno == errno.EEXIST:
             pass
         else:
@@ -193,7 +193,7 @@ def unlink(f):
     """
     try:
         os.unlink(f)
-    except OSError, e:
+    except OSError as e:
         if e.errno != errno.ENOENT:
             raise
 
@@ -753,7 +753,7 @@ if _mincore:
             msize = min(_fmincore_chunk_size, st.st_size - pos)
             try:
                 m = mmap.mmap(fd, msize, mmap.MAP_PRIVATE, 0, 0, pos)
-            except mmap.error, ex:
+            except mmap.error as ex:
                 if ex.errno == errno.EINVAL or ex.errno == errno.ENODEV:
                     # Perhaps the file was a pipe, i.e. "... | bup split ..."
                     return None
@@ -878,7 +878,7 @@ def parse_date_or_fatal(str, fatal):
     For now we expect a string that contains a float."""
     try:
         date = float(str)
-    except ValueError, e:
+    except ValueError as e:
         raise fatal('invalid date format (should be a float): %r' % e)
     else:
         return date
@@ -895,7 +895,7 @@ def parse_excludes(options, fatal):
         elif option == '--exclude-from':
             try:
                 f = open(realpath(parameter))
-            except IOError, e:
+            except IOError as e:
                 raise fatal("couldn't read %s" % parameter)
             for exclude_path in f.readlines():
                 # FIXME: perhaps this should be rstrip('\n')
@@ -915,12 +915,12 @@ def parse_rx_excludes(options, fatal):
         if option == '--exclude-rx':
             try:
                 excluded_patterns.append(re.compile(parameter))
-            except re.error, ex:
+            except re.error as ex:
                 fatal('invalid --exclude-rx pattern (%s): %s' % (parameter, ex))
         elif option == '--exclude-rx-from':
             try:
                 f = open(realpath(parameter))
-            except IOError, e:
+            except IOError as e:
                 raise fatal("couldn't read %s" % parameter)
             for pattern in f.readlines():
                 spattern = pattern.rstrip('\n')
@@ -928,7 +928,7 @@ def parse_rx_excludes(options, fatal):
                     continue
                 try:
                     excluded_patterns.append(re.compile(spattern))
-                except re.error, ex:
+                except re.error as ex:
                     fatal('invalid --exclude-rx pattern (%s): %s' % (spattern, ex))
     return excluded_patterns
 
index 5edb0a627961bd9ca50da249ec214fc791d562fd..70f3213f8b061805533fda066b196ce2f14b2b32 100644 (file)
@@ -15,7 +15,7 @@ class HLinkDB:
         f = None
         try:
             f = open(filename, 'r')
-        except IOError, e:
+        except IOError as e:
             if e.errno == errno.ENOENT:
                 pass
             else:
@@ -67,7 +67,7 @@ class HLinkDB:
         else: # No data -- delete _filename if it exists.
             try:
                 os.unlink(self._filename)
-            except OSError, e:
+            except OSError as e:
                 if e.errno == errno.ENOENT:
                     pass
                 else:
index 31f6c115ff3dc5a2561c291bef69794df450f2d2..fdf304380942a2a6802620598193542660e824e2 100644 (file)
@@ -174,7 +174,7 @@ class Entry:
                                self.gitmode, self.sha, self.flags,
                                self.children_ofs, self.children_n,
                                self.meta_ofs)
-        except (DeprecationWarning, struct.error), e:
+        except (DeprecationWarning, struct.error) as e:
             log('pack error: %s (%r)\n' % (e, self))
             raise
 
@@ -353,7 +353,7 @@ class Reader:
         f = None
         try:
             f = open(filename, 'r+')
-        except IOError, e:
+        except IOError as e:
             if e.errno == errno.ENOENT:
                 pass
             else:
@@ -536,7 +536,7 @@ def reduce_paths(paths):
                 rp = slashappend(rp)
                 p = slashappend(p)
             xpaths.append((rp, p))
-        except OSError, e:
+        except OSError as e:
             add_error('reduce_paths: %s' % e)
     xpaths.sort()
 
index 72f8b9b67b3a5dbcbe6a5a48d6ec49d635b30233..df299e58787b040f930a8938d65cbb6c4ad5bbf4 100644 (file)
@@ -120,7 +120,7 @@ def do_ls(args, pwd, default='.', onabort=None, spec_prefix=''):
                         output_node_info(sub, name)
             else:
                 output_node_info(n, os.path.normpath(path))
-        except vfs.NodeError, e:
+        except vfs.NodeError as e:
             log('error: %s\n' % e)
             ret = 1
 
index 046634519b2095707e0f0368e32175b25f5c0682..224393676fcfb2ba72e8d445baaed79990ca823b 100644 (file)
@@ -306,14 +306,14 @@ class Metadata:
         st = None
         try:
             st = xstat.lstat(path)
-        except OSError, e:
+        except OSError as e:
             if e.errno != errno.ENOENT:
                 raise
         if st:
             if stat.S_ISDIR(st.st_mode):
                 try:
                     os.rmdir(path)
-                except OSError, e:
+                except OSError as e:
                     if e.errno in (errno.ENOTEMPTY, errno.EEXIST):
                         msg = 'refusing to overwrite non-empty dir ' + path
                         raise Exception(msg)
@@ -340,7 +340,7 @@ class Metadata:
         elif stat.S_ISSOCK(self.mode):
             try:
                 os.mknod(path, 0o600 | stat.S_IFSOCK)
-            except OSError, e:
+            except OSError as e:
                 if e.errno in (errno.EINVAL, errno.EPERM):
                     s = socket.socket(socket.AF_UNIX, socket.SOCK_STREAM)
                     s.bind(path)
@@ -372,7 +372,7 @@ class Metadata:
         if lutime and stat.S_ISLNK(self.mode):
             try:
                 lutime(path, (self.atime, self.mtime))
-            except OSError, e:
+            except OSError as e:
                 if e.errno == errno.EACCES:
                     raise ApplyError('lutime: %s' % e)
                 else:
@@ -380,7 +380,7 @@ class Metadata:
         else:
             try:
                 utime(path, (self.atime, self.mtime))
-            except OSError, e:
+            except OSError as e:
                 if e.errno == errno.EACCES:
                     raise ApplyError('utime: %s' % e)
                 else:
@@ -414,7 +414,7 @@ class Metadata:
         if uid != -1 or gid != -1:
             try:
                 os.lchown(path, uid, gid)
-            except OSError, e:
+            except OSError as e:
                 if e.errno == errno.EPERM:
                     add_error('lchown: %s' %  e)
                 elif sys.platform.startswith('cygwin') \
@@ -451,7 +451,7 @@ class Metadata:
         try:
             if stat.S_ISLNK(st.st_mode):
                 self.symlink_target = os.readlink(path)
-        except OSError, e:
+        except OSError as e:
             add_error('readlink: %s' % e)
 
     def _encode_symlink_target(self):
@@ -498,7 +498,7 @@ class Metadata:
                     if stat.S_ISDIR(st.st_mode):
                         def_acl = posix1e.ACL(filedef=path)
                         def_acls = [def_acl, def_acl]
-            except EnvironmentError, e:
+            except EnvironmentError as e:
                 if e.errno not in (errno.EOPNOTSUPP, errno.ENOSYS):
                     raise
             if acls:
@@ -535,7 +535,7 @@ class Metadata:
         def apply_acl(acl_rep, kind):
             try:
                 acl = posix1e.ACL(text = acl_rep)
-            except IOError, e:
+            except IOError as e:
                 if e.errno == 0:
                     # pylibacl appears to return an IOError with errno
                     # set to 0 if a group referred to by the ACL rep
@@ -546,7 +546,7 @@ class Metadata:
                     raise
             try:
                 acl.applyto(path, kind)
-            except IOError, e:
+            except IOError as e:
                 if e.errno == errno.EPERM or e.errno == errno.EOPNOTSUPP:
                     raise ApplyError('POSIX1e ACL applyto: %s' % e)
                 else:
@@ -580,7 +580,7 @@ class Metadata:
                 attr = get_linux_file_attr(path)
                 if attr != 0:
                     self.linux_attr = attr
-            except OSError, e:
+            except OSError as e:
                 if e.errno == errno.EACCES:
                     add_error('read Linux attr: %s' % e)
                 elif e.errno in (errno.ENOTTY, errno.ENOSYS, errno.EOPNOTSUPP):
@@ -612,7 +612,7 @@ class Metadata:
                 return
             try:
                 set_linux_file_attr(path, self.linux_attr)
-            except OSError, e:
+            except OSError as e:
                 if e.errno in (errno.ENOTTY, errno.EOPNOTSUPP, errno.ENOSYS,
                                errno.EACCES):
                     raise ApplyError('Linux chattr: %s (0x%s)'
@@ -627,7 +627,7 @@ class Metadata:
         if not xattr: return
         try:
             self.linux_xattr = xattr.get_all(path, nofollow=True)
-        except EnvironmentError, e:
+        except EnvironmentError as e:
             if e.errno != errno.EOPNOTSUPP:
                 raise
 
@@ -664,7 +664,7 @@ class Metadata:
             return
         try:
             existing_xattrs = set(xattr.list(path, nofollow=True))
-        except IOError, e:
+        except IOError as e:
             if e.errno == errno.EACCES:
                 raise ApplyError('xattr.set %r: %s' % (path, e))
             else:
@@ -674,7 +674,7 @@ class Metadata:
                     or v != xattr.get(path, k, nofollow=True):
                 try:
                     xattr.set(path, k, v, nofollow=True)
-                except IOError, e:
+                except IOError as e:
                     if e.errno == errno.EPERM \
                             or e.errno == errno.EOPNOTSUPP:
                         raise ApplyError('xattr.set %r: %s' % (path, e))
@@ -684,7 +684,7 @@ class Metadata:
         for k in existing_xattrs:
             try:
                 xattr.remove(path, k, nofollow=True)
-            except IOError, e:
+            except IOError as e:
                 if e.errno == errno.EPERM:
                     raise ApplyError('xattr.remove %r: %s' % (path, e))
                 else:
@@ -810,7 +810,7 @@ class Metadata:
                                self._apply_linux_xattr_rec):
             try:
                 apply_metadata(path, restore_numeric_ids=num_ids)
-            except ApplyError, e:
+            except ApplyError as e:
                 add_error(e)
 
     def same_file(self, other):
index a15220a92da5680d66bde2b854c8dc4696ecfaf5..d73aac2eb126484aa57c1a31887e32a96338e0d9 100644 (file)
@@ -247,7 +247,7 @@ class Options:
         """
         try:
             (flags,extra) = self.optfunc(args, self._shortopts, self._longopts)
-        except getopt.GetoptError, e:
+        except getopt.GetoptError as e:
             self.fatal(e)
 
         opt = OptDict(aliases=self._aliases)
index 467a1ccdcb1626dd70c11646438b5480853d63f2..25ef66105335d6e9b1578f2a5ae631a7d440154b 100644 (file)
@@ -45,7 +45,7 @@ def test_bloom():
     skip_test = False
     try:
         b = bloom.create('bup.bloom', f=tf, expected=2**28, delaywrite=False)
-    except EnvironmentError, ex:
+    except EnvironmentError as ex:
         (ptr_width, linkage) = platform.architecture()
         if ptr_width == '32bit' and ex.errno == errno.ENOMEM:
             WVMSG('skipping large bloom filter test (mmap probably failed) '
index 79010b417159c3162564a10253ac18513e2f78d5..c57e8cd75e4b866bbe1c0fe5ed384f10c9f8c389 100644 (file)
@@ -193,7 +193,7 @@ def test_check_repo_or_die():
         open(bupdir + '/objects/pack', 'w').close()
         try:
             git.check_repo_or_die()
-        except SystemExit, e:
+        except SystemExit as e:
             WVPASSEQ(e.code, 14)
         else:
             WVFAIL()
@@ -202,7 +202,7 @@ def test_check_repo_or_die():
 
         try:
             git.check_repo_or_die('nonexistantbup.tmp')
-        except SystemExit, e:
+        except SystemExit as e:
             WVPASSEQ(e.code, 15)
         else:
             WVFAIL()
index 10ebce1b2c6bdb4f3f5ceca1c5075afeefbe733e..ee8df5103e6238c71f11832ca2fec3fd47ee19bf 100644 (file)
@@ -121,7 +121,7 @@ def test_readpipe():
     WVPASSEQ(x, '42\n')
     try:
         readpipe(['bash', '-c', 'exit 42'])
-    except Exception, ex:
+    except Exception as ex:
         WVPASSEQ(str(ex), "subprocess 'bash -c exit 42' failed with status 42")
 
 
@@ -135,7 +135,7 @@ def test_batchpipe():
     WVPASSEQ(out, '42\n')
     try:
         batchpipe(['bash', '-c'], ['exit 42'])
-    except Exception, ex:
+    except Exception as ex:
         WVPASSEQ(str(ex), "subprocess 'bash -c exit 42' failed with status 42")
     args = [str(x) for x in range(6)]
     # Force batchpipe to break the args into batches of 3.  This
index dbd5fe21028bc68c9726c2214c14a5b99a70ec3b..4aa87cc1b17f8ff6c9eeebe6cc35b361f3708923 100644 (file)
@@ -23,7 +23,7 @@ def ex(*cmd):
         elif rc > 0:
             print >> sys.stderr, 'returned exit status', rc
             sys.exit(1)
-    except OSError, e:
+    except OSError as e:
         print >> sys.stderr, 'subprocess call failed:', e
         sys.exit(1)
 
@@ -187,7 +187,7 @@ def _linux_attr_supported(path):
         return False
     try:
         metadata.get_linux_file_attr(path)
-    except OSError, e:
+    except OSError as e:
         if e.errno in (errno.ENOTTY, errno.ENOSYS, errno.EOPNOTSUPP):
             return False
         else:
index 7408414ce167bfed075cb02d0578ce3f6b5221c4..37f90be704fcba5b3f6f433dbf885ee1fca9f947 100644 (file)
@@ -5,17 +5,17 @@ from bup import _helpers
 
 try:
     _bup_utimensat = _helpers.bup_utimensat
-except AttributeError, e:
+except AttributeError as e:
     _bup_utimensat = False
 
 try:
     _bup_utimes = _helpers.bup_utimes
-except AttributeError, e:
+except AttributeError as e:
     _bup_utimes = False
 
 try:
     _bup_lutimes = _helpers.bup_lutimes
-except AttributeError, e:
+except AttributeError as e:
     _bup_lutimes = False
 
 
diff --git a/main.py b/main.py
index ef229571714926cfb41b09f0448d94e595e628e9..b149bbad76cbe49643792a6cf50be58864eb492e 100755 (executable)
--- a/main.py
+++ b/main.py
@@ -80,7 +80,7 @@ if len(argv) < 2:
 try:
     optspec = ['help', 'version', 'debug', 'profile', 'bup-dir=']
     global_args, subcmd = getopt.getopt(argv[1:], '?VDd:', optspec)
-except getopt.GetoptError, ex:
+except getopt.GetoptError as ex:
     usage('error: %s' % ex.msg)
 
 help_requested = None
@@ -195,7 +195,7 @@ try:
             ret = p.wait()
             forward_signals = False
             break
-    except OSError, e:
+    except OSError as e:
         log('%s: %s\n' % (subcmd[0], e))
         ret = 98
 finally:
index ef03381bd0328bd8ae23476152b9bcfaf1e01b0f..fdb4822dc3d1b90d69fe7f2060d1d46d610fa5c5 100755 (executable)
--- a/wvtest.py
+++ b/wvtest.py
@@ -122,7 +122,7 @@ if __name__ != '__main__':   # we're imported as a module
         '''
         try:
             func(*args, **kwargs)
-        except etype, e:
+        except etype as e:
             return _check(True, 'EXCEPT(%s)' % _code())
         except:
             _check(False, 'EXCEPT(%s)' % _code())
@@ -186,7 +186,7 @@ def _runtest(fname, f):
     sys.stdout.flush()
     try:
         _run_in_chdir(os.path.split(mod.__file__)[0], f)
-    except Exception, e:
+    except Exception as e:
         print
         print traceback.format_exc()
         tb = sys.exc_info()[2]