duplicity-team team mailing list archive

[gliptak <gliptak@xxxxxxxxxxxxxxxxxxxxx>]

gliptak has proposed merging lp:~gliptak/duplicity/415619 into lp:duplicity.

Requested reviews:
  duplicity-team (duplicity-team)
Related bugs:
  Bug #415619 in Duplicity: "incomplete chown() error msgs"
  https://bugs.launchpad.net/duplicity/+bug/415619

For more details, see:
https://code.launchpad.net/~gliptak/duplicity/415619/+merge/189473
-- 
https://code.launchpad.net/~gliptak/duplicity/415619/+merge/189473
Your team duplicity-team is requested to review the proposed merge of lp:~gliptak/duplicity/415619 into lp:duplicity.

=== modified file 'duplicity/tarfile.py'
--- duplicity/tarfile.py	2012-11-09 03:21:40 +0000
+++ duplicity/tarfile.py	2013-10-05 15:16:42 +0000
@@ -2282,7 +2282,7 @@
                     if sys.platform != "os2emx":
                         os.chown(targetpath, u, g)
             except EnvironmentError, e:
-                raise ExtractError("could not change owner")
+                raise ExtractError("could not change owner to %d:%d" % (u, g))
 
     def chmod(self, tarinfo, targetpath):
         """Set file permissions of targetpath according to tarinfo.