From daff4f017cbe9d14f147d0f7d46eef9ac776be49 Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Tue, 6 Oct 2009 06:10:07 +0000 Subject: 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 --- bin/repoman | 15 +++++++++++---- man/repoman.1 | 6 ++++++ 2 files changed, 17 insertions(+), 4 deletions(-) diff --git a/bin/repoman b/bin/repoman index 80939ca92..fd20b0778 100755 --- 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 b07427749..dbb214d31 100644 --- 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 -- cgit v1.2.3-1-g7c22