commit: ea2d9c5a7da408437d36c3f14112088f905d57f1
Author: Andreas K. Huettel (dilfridge) <dilfridge <AT> gentoo <DOT> org>
AuthorDate: Mon Jul 21 20:22:36 2014 +0000
Commit: Alexander Berntsen <bernalex <AT> gentoo <DOT> org>
CommitDate: Fri Jul 25 16:11:24 2014 +0000
URL:
http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=ea2d9c5a
repoman: Warn if virtuals depend on perl-core, bug 516428
Reported-by: Andreas K. Hüttel <dilfridge <AT> gentoo.org>
Authored-by: Andreas K. Hüttel <dilfridge <AT> gentoo.org>
Reviewed-by: Alexander Berntsen <bernalex <AT> gentoo.org>
Signed-off-by: Alexander Berntsen <bernalex <AT> gentoo.org>
---
bin/repoman | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/bin/repoman b/bin/repoman
index c36ace1..9c5d720 100755
--- a/bin/repoman
+++ b/bin/repoman
@@ -299,6 +299,7 @@ qahelp = {
"dependency.badindev": "User-visible ebuilds with unsatisfied
dependencies (matched against *visible* ebuilds) in developing arch",
"dependency.badmaskedindev": "Masked ebuilds with unsatisfied
dependencies (matched against *all* ebuilds) in developing arch",
"dependency.badtilde": "Uses the ~ dep operator with a non-zero
revision part, which is useless (the revision is ignored)",
+ "dependency.perlcore": "This ebuild directly depends on a package in
perl-core; it should use the corresponding virtual instead.",
"dependency.syntax": "Syntax error in dependency string (usually an
extra/missing space/parenthesis)",
"dependency.unknown": "Ebuild has a dependency that refers to an
unknown package (which may be valid if it is a blocker for a renamed/removed
package, or is an alternative choice provided by an overlay)",
"file.executable": "Ebuilds, digests, metadata.xml, Manifest, and
ChangeLog do not need the executable bit",
@@ -386,6 +387,7 @@ qawarnings = set((
"dependency.badindev",
"dependency.badmaskedindev",
"dependency.badtilde",
+"dependency.perlcore",
"DESCRIPTION.punctuation",
"DESCRIPTION.toolong",
"EAPI.deprecated",
@@ -2058,6 +2060,13 @@ for x in effective_scanlist:
relative_path +
": %s: consider
using '%s' instead of '%s'" %
(mytype,
suspect_virtual[atom.cp], atom))
+ if not is_blocker and \
+
atom.cp.startswith("perl-core/"):
+
stats['dependency.perlcore'] += 1
+
fails['dependency.perlcore'].append(
+ relative_path +
+ ": %s: please
use '%s' instead of '%s'" %
+ (mytype,
atom.replace("perl-core/","virtual/perl-"), atom))
if buildtime and \
not is_blocker and \
