[patch] implement a new RDEPEND.suspect check

suspect_rdepend.patch (text/plain), 2.82 KB, created by Zac Medico on 2008-05-14 18:01:50 UTC

Index: bin/repoman
===================================================================
--- bin/repoman	(revision 10333)
+++ bin/repoman	(working copy)
@@ -207,6 +207,7 @@
 	"IUSE.invalid":"This ebuild has a variable in IUSE that is not in the use.desc or use.local.desc file",
 	"LICENSE.invalid":"This ebuild is listing a license that doesnt exist in portages license/ dir.",
 	"KEYWORDS.invalid":"This ebuild contains KEYWORDS that are not listed in profiles/arch.list or for which no valid profile was found",
+	"RDEPEND.suspect":"RDEPEND contains a package that usually only belongs in DEPEND.",
 	"RESTRICT.invalid":"This ebuild contains invalid RESTRICT values.",
 	"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)",
@@ -242,6 +243,7 @@
 "KEYWORDS.dropped",
 "KEYWORDS.stupid",
 "KEYWORDS.missing",
+"RDEPEND.suspect",
 "RESTRICT.invalid",
 "ebuild.minorsyn",
 "ebuild.badheader",
@@ -272,6 +274,40 @@
 	"fetch", "installsources", "mirror",
 	"primaryuri", "strip", "test", "userpriv"])
 
+suspect_rdepend = frozenset([
+	"app-arch/cabextract",
+	"app-arch/rpm2targz",
+	"app-doc/doxygen",
+	"dev-lang/nasm",
+	"dev-lang/swig",
+	"dev-lang/yasm",
+	"dev-perl/extutils-pkgconfig",
+	"dev-python/setuptools",
+	"dev-util/byacc",
+	"dev-util/cmake",
+	"dev-util/gtk-doc",
+	"dev-util/gtk-doc-am",
+	"dev-util/intltool",
+	"dev-util/jam",
+	"dev-util/pkgconfig",
+	"dev-util/scons",
+	"dev-util/unifdef",
+	"dev-util/yacc",
+	"media-gfx/ebdftopcf",
+	"sys-apps/help2man",
+	"sys-devel/autoconf",
+	"sys-devel/automake",
+	"sys-devel/bin86",
+	"sys-devel/bison",
+	"sys-devel/dev86",
+	"sys-devel/flex",
+	"sys-devel/libtool",
+	"sys-devel/m4",
+	"sys-devel/pmake",
+	"x11-misc/bdftopcf",
+	"x11-misc/imake",
+])
+
 # file.executable
 no_exec = frozenset(["Manifest","ChangeLog","metadata.xml"])
 
@@ -1678,6 +1714,11 @@
 							portage.dep_getkey(atom) == "virtual/jdk":
 							stats['java.eclassesnotused'] += 1
 							fails['java.eclassesnotused'].append(relative_path)
+						elif mytype == "RDEPEND":
+							if portage.dep_getkey(atom) in suspect_rdepend:
+								stats['RDEPEND.suspect'] += 1
+								fails['RDEPEND.suspect'].append(
+									relative_path + ": '%s'" % atom)
 						if eapi == "0":
 							if portage_dep.dep_getslot(atom):
 								stats['EAPI.incompatible'] += 1
Index: man/repoman.1
===================================================================
--- man/repoman.1	(revision 10333)
+++ man/repoman.1	(working copy)
@@ -171,6 +171,9 @@
 .B RDEPEND.badmaskedindev
 Masked ebuilds with RDEPEND settings (matched against *all* ebuilds) in developing arch
 .TP
+.B RDEPEND.suspect
+RDEPEND contains a package that usually only belongs in DEPEND
+.TP
 .B RDEPEND.syntax
 Syntax error in RDEPEND (usually an extra/missing space/parenthesis)
 .TP