Add a 'digest.missing' check, for cases like mythtv where the eclass has

modified the SRC_URI and now the Manifest has missing digests. Thanks to
Brian Harring for the suggestion.

svn path=/main/trunk/; revision=14500

diff --git a/bin/repoman b/bin/repoman
index 80939ca92b178354c7d5657a7f87beb8d93e59af..fd20b0778cceccf11c1010f590ab5d81f2add6a9 100755 (executable)
--- a/bin/repoman
+++ b/bin/repoman
@@ -314,7 +314,9 @@ qahelp={
        "RDEPEND.implicit":"RDEPEND is unset in the ebuild which triggers implicit RDEPEND=$DEPEND assignment",
        "RDEPEND.suspect":"RDEPEND contains a package that usually only belongs in DEPEND.",
        "RESTRICT.invalid":"This ebuild contains invalid RESTRICT values.",
-       "digestentry.unused":"Some files listed in the Manifest aren't referenced in SRC_URI",
+       "digest.assumed":"Existing digest must be assumed correct (Package level only)",
+       "digest.missing":"Some files listed in SRC_URI aren't referenced in the Manifest",
+       "digest.unused":"Some files listed in the Manifest aren't referenced in SRC_URI",
        "ebuild.nostable":"There are no ebuilds that are marked as stable for your ARCH",
        "ebuild.allmasked":"All ebuilds are masked for this package (Package level only)",
        "ebuild.majorsyn":"This ebuild has a major syntax error that may cause the ebuild to fail partially or fully",
@@ -336,7 +338,8 @@ qacats.sort()
 qawarnings = set((
 "changelog.missing",
 "changelog.notadded",
-"digestentry.unused",
+"digest.assumed",
+"digest.unused",
 "ebuild.notadded",
 "ebuild.nostable",
 "ebuild.allmasked",
@@ -1094,8 +1097,12 @@ for x in scanlist:
                myfiles_all = set(myfiles_all)
                for entry in mydigests:
                        if entry not in myfiles_all:
-                               stats["digestentry.unused"] += 1
-                               fails["digestentry.unused"].append(checkdir+"::"+entry)
+                               stats["digest.unused"] += 1
+                               fails["digest.unused"].append(checkdir+"::"+entry)
+               for entry in myfiles_all:
+                       if entry not in mydigests:
+                               stats["digest.missing"] += 1
+                               fails["digest.missing"].append(checkdir+"::"+entry)
        del myfiles_all
 
        if os.path.exists(checkdir+"/files"):

diff --git a/man/repoman.1 b/man/repoman.1
index b074277496991f301dfb59a51b152e3fca80afc5..dbb214d318574c18d3c316c5f5a5579e2b273d28 100644 (file)
--- a/man/repoman.1
+++ b/man/repoman.1
@@ -216,6 +216,12 @@ ChangeLogs that exist but have not been added to cvs
 .B digest.assumed
 Existing digest must be assumed correct (Package level only)
 .TP
+.B digest.missing
+Some files listed in SRC_URI aren't referenced in the Manifest
+.TP
+.B digest.unused
+Some files listed in the Manifest aren't referenced in SRC_URI
+.TP
 .B ebuild.allmasked
 All ebuilds are masked for this package (Package level only)
 .TP