diff --git a/scripts/glibc_check.py b/scripts/glibc_check.py index 02be4aac4..eb198ed56 100755 --- a/scripts/glibc_check.py +++ b/scripts/glibc_check.py @@ -41,7 +41,7 @@ def main(): overall_versions.add(requires_glibc) if verbose: print("%s %s" % (filename, format_version(requires_glibc))) - except subprocess.CalledProcessError as e: + except subprocess.CalledProcessError: if verbose: print("%s errored." % filename) diff --git a/setup.cfg b/setup.cfg index 40ca52dba..c283992b3 100644 --- a/setup.cfg +++ b/setup.cfg @@ -5,7 +5,9 @@ python_files = testsuite/*.py # please note that the values are adjusted so that they do not cause failures # with existing code. if you want to change them, you should first fix all # flake8 failures that appear with your change. -ignore = E122,E123,E125,E126,E127,E128,E226,E402,E722,E731,E741,F401,F405,F811 +ignore = E122,E123,E125,E126,E127,E128,E226,E402,E722,E731,E741, + F401,F405,F811, + W504 # line length long term target: 120 max-line-length = 255 exclude = build,dist,.git,.idea,.cache,.tox,docs/conf.py diff --git a/src/borg/archive.py b/src/borg/archive.py index c22a655f7..cf8976428 100644 --- a/src/borg/archive.py +++ b/src/borg/archive.py @@ -758,7 +758,7 @@ Utilization of max. archive size: {csize_max:.0%} def fetch_async_response(wait=True): try: return self.repository.async_response(wait=wait) - except Repository.ObjectNotFound as e: + except Repository.ObjectNotFound: nonlocal error # object not in repo - strange, but we wanted to delete it anyway. if forced == 0: @@ -1627,7 +1627,7 @@ class ArchiveChecker: yield Item(internal_dict=item) else: report('Did not get expected metadata dict when unpacking item metadata (%s)' % reason, chunk_id, i) - except msgpack.UnpackException as err: + except msgpack.UnpackException: report('Unpacker crashed while unpacking item metadata, trying to resync...', chunk_id, i) unpacker.resync() except Exception: diff --git a/src/borg/archiver.py b/src/borg/archiver.py index 2a775fe5a..1afd0a197 100644 --- a/src/borg/archiver.py +++ b/src/borg/archiver.py @@ -1978,7 +1978,7 @@ class Archiver: `Fnmatch `_, selector `fm:` This is the default style for ``--exclude`` and ``--exclude-from``. - These patterns use a variant of shell pattern syntax, with '\*' matching + These patterns use a variant of shell pattern syntax, with '\\*' matching any number of characters, '?' matching any single character, '[...]' matching any single character specified, including ranges, and '[!...]' matching any character not specified. For the purpose of these patterns, @@ -1989,7 +1989,7 @@ class Archiver: from the start of the full path to just before a path separator. Except for the root path, paths will never end in the path separator when matching is attempted. Thus, if a given pattern ends in a path - separator, a '\*' is appended before matching is attempted. + separator, a '\\*' is appended before matching is attempted. Shell-style patterns, selector `sh:` This is the default style for ``--pattern`` and ``--patterns-from``. @@ -2064,7 +2064,7 @@ class Archiver: # The contents of directories in '/home' are not backed up when their name # ends in '.tmp' - $ borg create --exclude 're:^/home/[^/]+\.tmp/' backup / + $ borg create --exclude 're:^/home/[^/]+\\.tmp/' backup / # Load exclusions from file $ cat >exclude.txt < 100: # Limit file names to some reasonable length. Most file systems # limit them to 255 [unit of choice]; due to variations in unicode diff --git a/src/borg/testsuite/__init__.py b/src/borg/testsuite/__init__.py index 465e24fa1..caa7d2a22 100644 --- a/src/borg/testsuite/__init__.py +++ b/src/borg/testsuite/__init__.py @@ -111,7 +111,7 @@ def is_utime_fully_supported(): new_stats = os.stat(filepath, follow_symlinks=False) if new_stats.st_atime == 1000 and new_stats.st_mtime == 2000: return True - except OSError as err: + except OSError: pass return False @@ -133,7 +133,7 @@ def is_birthtime_fully_supported(): new_stats = os.stat(filepath, follow_symlinks=False) if new_stats.st_birthtime == birthtime and new_stats.st_mtime == mtime and new_stats.st_atime == atime: return True - except OSError as err: + except OSError: pass return False diff --git a/src/borg/testsuite/key.py b/src/borg/testsuite/key.py index 1046e1ad1..1d6281056 100644 --- a/src/borg/testsuite/key.py +++ b/src/borg/testsuite/key.py @@ -35,7 +35,7 @@ class TestKey: /cXJq7jrqmrJ1phd6dg4SHAM/i+hubadZoS6m25OQzYAW09wZD/phG8OVa698Z5ed3HTaT SmrtgJL3EoOKgUI9d6BLE4dJdBqntifo""".strip() - keyfile2_cdata = unhexlify(re.sub('\W', '', """ + keyfile2_cdata = unhexlify(re.sub(r'\W', '', """ 0055f161493fcfc16276e8c31493c4641e1eb19a79d0326fad0291e5a9c98e5933 00000000000003e8d21eaf9b86c297a8cd56432e1915bb """)) diff --git a/src/borg/testsuite/patterns.py b/src/borg/testsuite/patterns.py index ff447888f..5806ff69b 100644 --- a/src/borg/testsuite/patterns.py +++ b/src/borg/testsuite/patterns.py @@ -229,13 +229,13 @@ def test_invalid_unicode_pattern(pattern): "", "# EOF"], ["/more/data", "/home", " #/wsfoobar"]), - (["re:.*"], []), - (["re:\s"], ["/data/something00.txt", "/more/data", "/home"]), + ([r"re:.*"], []), + ([r"re:\s"], ["/data/something00.txt", "/more/data", "/home"]), ([r"re:(.)(\1)"], ["/more/data", "/home", "\tstart/whitespace", "/whitespace/end\t"]), (["", "", "", "# This is a test with mixed pattern styles", # Case-insensitive pattern - "re:(?i)BAR|ME$", + r"re:(?i)BAR|ME$", "", "*whitespace*", "fm:*/something00*"],