]> err.no Git - dak/commitdiff
do not check for superfluous checksums-* entries for now
authorPhilipp Kern <pkern@debian.org>
Sun, 7 Sep 2008 21:46:51 +0000 (21:46 +0000)
committerPhilipp Kern <pkern@debian.org>
Sun, 7 Sep 2008 21:46:51 +0000 (21:46 +0000)
The current check for additional entries in checksums-* which are not
covered by entries in files is flawed because files could be modified
by autobyhand or sometimes when the file's already in the pool.

We need to rework this to look into the original files dict instead.

Signed-off-by: Philipp Kern <pkern@debian.org>
daklib/utils.py

index 075e4a16ab8e190a4a865cc496a0fbaf1bbdcda0..86e51a2f5c852f5b8c53b42741db14fcb96265ef 100755 (executable)
@@ -394,12 +394,15 @@ def parse_checksums(where, files, manifest, hashname):
         if not line:
             break
         hash, size, file = line.strip().split(' ')
-        if not files.has_key(file):
-            rejmsg.append("%s: not present in files but in checksums-%s in %s" %
-                (file, hashname, where))
+        # TODO: check for the file's entry in the original files dict, not
+        # the one modified by (auto)byhand and other weird stuff
+        #if not files.has_key(file):
+        #    rejmsg.append("%s: not present in files but in checksums-%s in %s" %
+        #        (file, hashname, where))
         if not files[file]["size"] == size:
             rejmsg.append("%s: size differs for files and checksums-%s entry "\
                 "in %s" % (file, hashname, where))
+            continue
         files[file][hash_key(hashname)] = hash
     for f in files.keys():
         if not files[f].has_key(hash_key(hashname)):