[gentoo-commits] proj/portage:2.1.9 commit in: pym/repoman/, man/, bin/

[gentoo-commits] proj/portage:2.1.9 commit in: pym/repoman/, man/, bin/

[Zac Medico]

commit:     a8be8f23dc84e92b9163f152af9640c2a37bad0b
Author:     Zac Medico <zmedico <AT> gentoo <DOT> org>
AuthorDate: Sat Feb 26 00:19:01 2011 +0000
Commit:     Zac Medico <zmedico <AT> gentoo <DOT> org>
CommitDate: Tue Mar  1 20:52:31 2011 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/portage.git;a=commit;h=a8be8f23

repoman: add back portage.internal check

This check got removed for bug #262365, but now that EAPI 4 is released
we can add it back. Now it checks for ecompress/ecompressdir too (for
bug #218347).

---
 bin/repoman           |    2 ++
 man/repoman.1         |    3 +++
 pym/repoman/checks.py |   12 +++++++++++-
 3 files changed, 16 insertions(+), 1 deletions(-)

diff --git a/bin/repoman b/bin/repoman
index b694b6c..8b1efc5 100755
--- a/bin/repoman
+++ b/bin/repoman
@@ -364,6 +364,7 @@ qahelp={
 	"metadata.missing":"Missing metadata.xml files",
 	"metadata.bad":"Bad metadata.xml files",
 	"metadata.warning":"Warnings in metadata.xml files",
+	"portage.internal":"The ebuild uses an internal Portage function",
 	"virtual.versioned":"PROVIDE contains virtuals with versions",
 	"virtual.exists":"PROVIDE contains existing package names",
 	"virtual.unavailable":"PROVIDE contains a virtual which contains no profile default",
@@ -410,6 +411,7 @@ qawarnings = set((
 "java.eclassesnotused",
 "wxwidgets.eclassnotused",
 "metadata.warning",
+"portage.internal",
 "virtual.versioned",
 "virtual.exists",
 "virtual.unavailable",

diff --git a/man/repoman.1 b/man/repoman.1
index f9501bf..162b62b 100644
--- a/man/repoman.1
+++ b/man/repoman.1
@@ -330,6 +330,9 @@ Missing metadata.xml files
 .B metadata.warning
 Warnings in metadata.xml files
 .TP
+.B portage.internal
+The ebuild uses an internal Portage function
+.TP
 .B upstream.workaround
 The ebuild works around an upstream bug, an upstream bug should be filed and
 tracked in bugs.gentoo.org

diff --git a/pym/repoman/checks.py b/pym/repoman/checks.py
index 5bf039b..a05c128 100644
--- a/pym/repoman/checks.py
+++ b/pym/repoman/checks.py
@@ -623,6 +623,16 @@ class Eapi4GoneVars(LineCheck):
 			return ("variable '$%s'" % m.group(1)) + \
 				" is gone in EAPI=4 on line: %d"
 
+class PortageInternal(LineCheck):
+	repoman_check_name = 'portage.internal'
+	re = re.compile(r'[^#]*\b(ecompress|ecompressdir|prepalldocs)\b')
+
+	def check(self, num, line):
+		"""Run the check on line and return error if there is one"""
+		m = self.re.match(line)
+		if m is not None:
+			return ("'%s'" % m.group(1)) + " called on line: %d"
+
 _constant_checks = tuple((c() for c in (
 	EbuildHeader, EbuildWhitespace, EbuildBlankLine, EbuildQuote,
 	EbuildAssignment, Eapi3EbuildAssignment, EbuildUselessDodoc,
@@ -633,7 +643,7 @@ _constant_checks = tuple((c() for c in (
 	DeprecatedBindnowFlags, SrcUnpackPatches, WantAutoDefaultValue,
 	SrcCompileEconf, Eapi3DeprecatedFuncs, NoOffsetWithHelpers,
 	Eapi4IncompatibleFuncs, Eapi4GoneVars, BuiltWithUse,
-	PreserveOldLib, SandboxAddpredict)))
+	PreserveOldLib, SandboxAddpredict, PortageInternal)))
 
 _here_doc_re = re.compile(r'.*\s<<[-]?(\w+)$')