Bug #285191 - Remove the RDEPEND.implicit check because it is invalid.
authorZac Medico <zmedico@gentoo.org>
Thu, 1 Oct 2009 19:29:24 +0000 (19:29 -0000)
committerZac Medico <zmedico@gentoo.org>
Thu, 1 Oct 2009 19:29:24 +0000 (19:29 -0000)
svn path=/main/trunk/; revision=14471

man/repoman.1
pym/repoman/checks.py

index 52ef02a1e3d6c10b21584b4e1efd465353338a06..b074277496991f301dfb59a51b152e3fca80afc5 100644 (file)
@@ -186,10 +186,6 @@ Masked ebuilds with RDEPEND settings (matched against *all* ebuilds)
 .B RDEPEND.badmaskedindev
 Masked ebuilds with RDEPEND settings (matched against *all* ebuilds) in developing arch
 .TP
-.B RDEPEND.implicit
-RDEPEND is unset in the ebuild which triggers implicit RDEPEND=$DEPEND
-assignment
-.TP
 .B RDEPEND.suspect
 RDEPEND contains a package that usually only belongs in DEPEND
 .TP
index bfb2625450a0b901b5ea623b53bae1f31c6e63f5..5058ec77e0de14ab2967df67577a9b3a7472a1cd 100644 (file)
@@ -268,37 +268,6 @@ class EbuildQuotedA(LineCheck):
                if match:
                        return "Quoted \"${A}\" on line: %d"
 
-class ImplicitRuntimeDeps(LineCheck):
-       """
-       Detect the case where DEPEND is set and RDEPEND is unset in the ebuild,
-       since this triggers implicit RDEPEND=$DEPEND assignment.
-       """
-
-       _assignment_re = re.compile(r'^\s*(R?DEPEND)=')
-
-       def new(self, pkg):
-               # RDEPEND=DEPEND is no longer available in EAPI=3
-               if pkg.metadata['EAPI'] in ('0', '1', '2'):
-                       self.repoman_check_name = 'RDEPEND.implicit'
-               else:
-                       self.repoman_check_name = 'EAPI.incompatible'
-               self._rdepend = False
-               self._depend = False
-
-       def check(self, num, line):
-               if not self._rdepend:
-                       m = self._assignment_re.match(line)
-                       if m is None:
-                               pass
-                       elif m.group(1) == "RDEPEND":
-                               self._rdepend = True
-                       elif m.group(1) == "DEPEND":
-                               self._depend = True
-
-       def end(self):
-               if self._depend and not self._rdepend:
-                       yield 'RDEPEND is not explicitly assigned'
-
 class InheritAutotools(LineCheck):
        """
        Make sure appropriate functions are called in
@@ -493,7 +462,7 @@ _constant_checks = tuple((c() for c in (
        EbuildAssignment, EbuildUselessDodoc,
        EbuildUselessCdS, EbuildNestedDie,
        EbuildPatches, EbuildQuotedA, EapiDefinition,
-       IUseUndefined, ImplicitRuntimeDeps, InheritAutotools,
+       IUseUndefined, InheritAutotools,
        EMakeParallelDisabled, EMakeParallelDisabledViaMAKEOPTS,
        DeprecatedBindnowFlags, SrcUnpackPatches, WantAutoDefaultValue,
        SrcCompileEconf, Eapi3IncompatibleFuncs, Eapi3GoneVars)))