InheritEclass: avoid false positive in func regex

diff --git a/pym/repoman/checks.py b/pym/repoman/checks.py
index 2864d675dccb6be292412001d2f6bb077f52327d..92a1e5a61eb62c77d4f026f16bd7f99b1903ce49 100644 (file)
--- a/pym/repoman/checks.py
+++ b/pym/repoman/checks.py
@@ -469,10 +469,11 @@ class InheritEclass(LineCheck):
                self._ignore_missing = ignore_missing
                inherit_re = eclass
                self._inherit_re = re.compile(r'^(\s*|.*[|&]\s*)\binherit\s(.*\s)?%s(\s|$)' % inherit_re)
-               # Match when the function is preceded only by leading whitespace or a shell
-               # operator such as (, {, |, ||, or &&. This prevents false postives in
-               # things like elog messages, as reported in bug #413285.
-               self._func_re = re.compile(r'(^|[|&{(])\s*\b(' + '|'.join(funcs) + r')\b')
+               # Match when the function is preceded only by leading whitespace, a
+               # shell operator such as (, {, |, ||, or &&, or optional variable
+               # setting(s). This prevents false postives in things like elog
+               # messages, as reported in bug #413285.
+               self._func_re = re.compile(r'(^|[|&{(])\s*(\w+=.*)?\b(' + '|'.join(funcs) + r')\b')
 
        def new(self, pkg):
                self.repoman_check_name = 'inherit.missing'
@@ -496,7 +497,7 @@ class InheritEclass(LineCheck):
                        if s:
                                self._func_call = True
                                return '%s.eclass is not inherited, but "%s" found at line: %s' % \
-                                       (self._eclass, s.group(2), '%d')
+                                       (self._eclass, s.group(3), '%d')
                elif not self._func_call:
                        self._func_call = self._func_re.search(line)