Thomas Waldmann 1 year ago
parent
commit
6151b369c4
4 changed files with 17 additions and 18 deletions
  1. 3 4
      setup.cfg
  2. 2 2
      src/borg/archiver/__init__.py
  3. 2 2
      src/borg/helpers/parseformat.py
  4. 10 10
      src/borg/testsuite/crypto.py

+ 3 - 4
setup.cfg

@@ -78,7 +78,6 @@ markers =
 # for reference ...
 # for reference ...
 #   E402 module level import not at top
 #   E402 module level import not at top
 #   E501 line too long
 #   E501 line too long
-#   E741 ambiguous variable name
 #   F401 import unused
 #   F401 import unused
 #   F405 undefined or defined from star imports
 #   F405 undefined or defined from star imports
 #   F811 redef of unused var
 #   F811 redef of unused var
@@ -96,7 +95,7 @@ ignore = W503, E203, F405
 # flake8 failures that appear with your change.
 # flake8 failures that appear with your change.
 per_file_ignores =
 per_file_ignores =
     src/borg/archive.py:E402,E501
     src/borg/archive.py:E402,E501
-    src/borg/archiver/__init__.py:E402,E741
+    src/borg/archiver/__init__.py:E402
     src/borg/archiver/_common.py:E501
     src/borg/archiver/_common.py:E501
     src/borg/archiver/help_cmd.py:E501
     src/borg/archiver/help_cmd.py:E501
     src/borg/archiver/rlist_cmd.py:E501
     src/borg/archiver/rlist_cmd.py:E501
@@ -109,7 +108,7 @@ per_file_ignores =
     src/borg/crypto/key.py:E402
     src/borg/crypto/key.py:E402
     src/borg/helpers/__init__.py:F401
     src/borg/helpers/__init__.py:F401
     src/borg/helpers/misc.py:E402
     src/borg/helpers/misc.py:E402
-    src/borg/helpers/parseformat.py:E402,E501,E741
+    src/borg/helpers/parseformat.py:E402,E501
     src/borg/helpers/process.py:E402
     src/borg/helpers/process.py:E402
     src/borg/helpers/shellpattern.py:E501
     src/borg/helpers/shellpattern.py:E501
     src/borg/platform/__init__.py:F401
     src/borg/platform/__init__.py:F401
@@ -117,7 +116,7 @@ per_file_ignores =
     src/borg/testsuite/archiver/disk_full.py:F811
     src/borg/testsuite/archiver/disk_full.py:F811
     src/borg/testsuite/archiver/return_codes.py:F811
     src/borg/testsuite/archiver/return_codes.py:F811
     src/borg/testsuite/benchmark.py:F811
     src/borg/testsuite/benchmark.py:F811
-    src/borg/testsuite/crypto.py:E501,E741
+    src/borg/testsuite/crypto.py:E501
     src/borg/testsuite/helpers.py:E501
     src/borg/testsuite/helpers.py:E501
     src/borg/testsuite/locking.py:E501
     src/borg/testsuite/locking.py:E501
     src/borg/testsuite/platform.py:E501,F811
     src/borg/testsuite/platform.py:E501,F811

+ 2 - 2
src/borg/archiver/__init__.py

@@ -570,8 +570,8 @@ def format_tb(exc):
     remote = isinstance(exc, RemoteRepository.RPCError)
     remote = isinstance(exc, RemoteRepository.RPCError)
     if remote:
     if remote:
         prefix = "Borg server: "
         prefix = "Borg server: "
-        trace_back = "\n".join(prefix + l for l in exc.exception_full.splitlines())
-        sys_info = "\n".join(prefix + l for l in exc.sysinfo.splitlines())
+        trace_back = "\n".join(prefix + line for line in exc.exception_full.splitlines())
+        sys_info = "\n".join(prefix + line for line in exc.sysinfo.splitlines())
     else:
     else:
         trace_back = traceback.format_exc()
         trace_back = traceback.format_exc()
         sys_info = sysinfo()
         sys_info = sysinfo()

+ 2 - 2
src/borg/helpers/parseformat.py

@@ -373,8 +373,8 @@ def format_archive(archive):
 
 
 
 
 def parse_stringified_list(s):
 def parse_stringified_list(s):
-    l = re.split(" *, *", s)
-    return [item for item in l if item != ""]
+    items = re.split(" *, *", s)
+    return [item for item in items if item != ""]
 
 
 
 
 class Location:
 class Location:

+ 10 - 10
src/borg/testsuite/crypto.py

@@ -202,9 +202,9 @@ class CryptoTestCase(BaseTestCase):
         ikm = b"\x0b" * 22
         ikm = b"\x0b" * 22
         salt = bytes.fromhex("000102030405060708090a0b0c")
         salt = bytes.fromhex("000102030405060708090a0b0c")
         info = bytes.fromhex("f0f1f2f3f4f5f6f7f8f9")
         info = bytes.fromhex("f0f1f2f3f4f5f6f7f8f9")
-        l = 42
+        length = 42
 
 
-        okm = hkdf_hmac_sha512(ikm, salt, info, l)
+        okm = hkdf_hmac_sha512(ikm, salt, info, length)
         assert okm == bytes.fromhex(
         assert okm == bytes.fromhex(
             "832390086cda71fb47625bb5ceb168e4c8e26a1a16ed34d9fc7fe92c1481579338da362cb8d9f925d7cb"
             "832390086cda71fb47625bb5ceb168e4c8e26a1a16ed34d9fc7fe92c1481579338da362cb8d9f925d7cb"
         )
         )
@@ -222,9 +222,9 @@ class CryptoTestCase(BaseTestCase):
             "b0b1b2b3b4b5b6b7b8b9babbbcbdbebfc0c1c2c3c4c5c6c7c8c9cacbcccdcecfd0d1d2d3d4d5d6d7"
             "b0b1b2b3b4b5b6b7b8b9babbbcbdbebfc0c1c2c3c4c5c6c7c8c9cacbcccdcecfd0d1d2d3d4d5d6d7"
             "d8d9dadbdcdddedfe0e1e2e3e4e5e6e7e8e9eaebecedeeeff0f1f2f3f4f5f6f7f8f9fafbfcfdfeff"
             "d8d9dadbdcdddedfe0e1e2e3e4e5e6e7e8e9eaebecedeeeff0f1f2f3f4f5f6f7f8f9fafbfcfdfeff"
         )
         )
-        l = 82
+        length = 82
 
 
-        okm = hkdf_hmac_sha512(ikm, salt, info, l)
+        okm = hkdf_hmac_sha512(ikm, salt, info, length)
         assert okm == bytes.fromhex(
         assert okm == bytes.fromhex(
             "ce6c97192805b346e6161e821ed165673b84f400a2b514b2fe23d84cd189ddf1b695b48cbd1c838844"
             "ce6c97192805b346e6161e821ed165673b84f400a2b514b2fe23d84cd189ddf1b695b48cbd1c838844"
             "1137b3ce28f16aa64ba33ba466b24df6cfcb021ecff235f6a2056ce3af1de44d572097a8505d9e7a93"
             "1137b3ce28f16aa64ba33ba466b24df6cfcb021ecff235f6a2056ce3af1de44d572097a8505d9e7a93"
@@ -234,9 +234,9 @@ class CryptoTestCase(BaseTestCase):
         ikm = bytes.fromhex("0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b")
         ikm = bytes.fromhex("0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b0b")
         salt = None
         salt = None
         info = b""
         info = b""
-        l = 42
+        length = 42
 
 
-        okm = hkdf_hmac_sha512(ikm, salt, info, l)
+        okm = hkdf_hmac_sha512(ikm, salt, info, length)
         assert okm == bytes.fromhex(
         assert okm == bytes.fromhex(
             "f5fa02b18298a72a8c23898a8703472c6eb179dc204c03425c970e3b164bf90fff22d04836d0e2343bac"
             "f5fa02b18298a72a8c23898a8703472c6eb179dc204c03425c970e3b164bf90fff22d04836d0e2343bac"
         )
         )
@@ -245,9 +245,9 @@ class CryptoTestCase(BaseTestCase):
         ikm = bytes.fromhex("0b0b0b0b0b0b0b0b0b0b0b")
         ikm = bytes.fromhex("0b0b0b0b0b0b0b0b0b0b0b")
         salt = bytes.fromhex("000102030405060708090a0b0c")
         salt = bytes.fromhex("000102030405060708090a0b0c")
         info = bytes.fromhex("f0f1f2f3f4f5f6f7f8f9")
         info = bytes.fromhex("f0f1f2f3f4f5f6f7f8f9")
-        l = 42
+        length = 42
 
 
-        okm = hkdf_hmac_sha512(ikm, salt, info, l)
+        okm = hkdf_hmac_sha512(ikm, salt, info, length)
         assert okm == bytes.fromhex(
         assert okm == bytes.fromhex(
             "7413e8997e020610fbf6823f2ce14bff01875db1ca55f68cfcf3954dc8aff53559bd5e3028b080f7c068"
             "7413e8997e020610fbf6823f2ce14bff01875db1ca55f68cfcf3954dc8aff53559bd5e3028b080f7c068"
         )
         )
@@ -256,9 +256,9 @@ class CryptoTestCase(BaseTestCase):
         ikm = bytes.fromhex("0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c")
         ikm = bytes.fromhex("0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c0c")
         salt = None
         salt = None
         info = b""
         info = b""
-        l = 42
+        length = 42
 
 
-        okm = hkdf_hmac_sha512(ikm, salt, info, l)
+        okm = hkdf_hmac_sha512(ikm, salt, info, length)
         assert okm == bytes.fromhex(
         assert okm == bytes.fromhex(
             "1407d46013d98bc6decefcfee55f0f90b0c7f63d68eb1a80eaf07e953cfc0a3a5240a155d6e4daa965bb"
             "1407d46013d98bc6decefcfee55f0f90b0c7f63d68eb1a80eaf07e953cfc0a3a5240a155d6e4daa965bb"
         )
         )