diff options
Diffstat (limited to 'rdiff-backup')
-rw-r--r-- | rdiff-backup/CHANGELOG | 3 | ||||
-rw-r--r-- | rdiff-backup/rdiff_backup/eas_acls.py | 27 |
2 files changed, 19 insertions, 11 deletions
diff --git a/rdiff-backup/CHANGELOG b/rdiff-backup/CHANGELOG index c8d1dfe..fc7c0ad 100644 --- a/rdiff-backup/CHANGELOG +++ b/rdiff-backup/CHANGELOG @@ -1,6 +1,9 @@ New in v1.1.13 (????/??/??) --------------------------- +Don't throw an error when clearing extended attributes if they are not +supported on the file. (Andrew Ferguson) + New in v1.1.12 (2007/07/12) --------------------------- diff --git a/rdiff-backup/rdiff_backup/eas_acls.py b/rdiff-backup/rdiff_backup/eas_acls.py index 71878db..0b37898 100644 --- a/rdiff-backup/rdiff_backup/eas_acls.py +++ b/rdiff-backup/rdiff_backup/eas_acls.py @@ -81,17 +81,22 @@ class ExtendedAttributes: def clear_rp(self, rp): """Delete all the extended attributes in rpath""" - for name in rp.conn.xattr.listxattr(rp.path): - try: - rp.conn.xattr.removexattr(rp.path, name) - except IOError, exc: - # SELinux attributes cannot be removed, and we don't want - # to bail out or be too noisy at low log levels. - if exc[0] == errno.EACCES: - log.Log("Warning: unable to remove xattr %s from %s" - % (name, rp.path), 7) - continue - else: raise + try: + for name in rp.conn.xattr.listxattr(rp.path): + try: + rp.conn.xattr.removexattr(rp.path, name) + except IOError, exc: + # SELinux attributes cannot be removed, and we don't want + # to bail out or be too noisy at low log levels. + if exc[0] == errno.EACCES: + log.Log("Warning: unable to remove xattr %s from %s" + % (name, rp.path), 7) + continue + else: raise + except IOError, exc: + if exc[0] == errno.EOPNOTSUPP or exc[0] == errno.EPERM: + return # if not supported, consider empty + else: raise def write_to_rp(self, rp): """Write extended attributes to rpath rp""" |