summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHelmut Grohne <helmut@subdivi.de>2021-12-29 21:05:58 +0100
committerHelmut Grohne <helmut@subdivi.de>2021-12-29 21:05:58 +0100
commitd9d757792682e98e006d93f6bcbb94688d3a0f3f (patch)
tree25af5ab11015d3dca345e4ce260a5a31d216da86
parent69a8861b704c969260ecb55110d8e41cd9aaf0a7 (diff)
downloaddebian-dedup-d9d757792682e98e006d93f6bcbb94688d3a0f3f.tar.gz
autoimport: avoid changing variable type
knownpkgvers is a dict while knownpkgs is a set. Separating them helps static type checkers.
-rwxr-xr-xautoimport.py9
1 files changed, 5 insertions, 4 deletions
diff --git a/autoimport.py b/autoimport.py
index d9ee0a3..eb610b4 100755
--- a/autoimport.py
+++ b/autoimport.py
@@ -104,14 +104,15 @@ def main():
print("reading database")
cur.execute("SELECT name, version FROM package;")
- knownpkgs = dict((row[0], row[1]) for row in cur.fetchall())
+ knownpkgvers = dict((row[0], row[1]) for row in cur.fetchall())
distpkgs = set(pkgs.keys())
if args.new:
for name in distpkgs:
- if name in knownpkgs and version_compare(pkgs[name]["version"],
- knownpkgs[name]) <= 0:
+ if name in knownpkgvers and \
+ version_compare(pkgs[name]["version"], knownpkgvers[name]) <= 0:
del pkgs[name]
- knownpkgs = set(knownpkgs)
+ knownpkgs = set(knownpkgvers)
+ del knownpkgvers
with e:
fs = {}