Adjust the EbuildNestedDie pattern so that it only matches when ( is surrounded

by whitespace. This filters out false positives triggered by sed expressions in
net-analyzer/amap-5.2-r1 and media-libs/libsndfile-1.0.17-r1. Thanks to Diego
Pettenò <flameeyes@g.o> for reporting. (trunk r12265)

svn path=/main/branches/2.1.6/; revision=12267

diff --git a/pym/repoman/checks.py b/pym/repoman/checks.py
index 6f2c076c32907785a52a4b95bf62fe3390f5928c..7565039f274af031be89299a7336ae4e8034e35a 100644 (file)
--- a/pym/repoman/checks.py
+++ b/pym/repoman/checks.py
@@ -177,7 +177,7 @@ class EbuildNestedDie(LineCheck):
        """Check ebuild for nested die statements (die statements in subshells"""
        
        repoman_check_name = 'ebuild.nesteddie'
-       nesteddie_re = re.compile(r'^[^#]*\([^)]*\bdie\b')
+       nesteddie_re = re.compile(r'^[^#]*\s\(\s[^)]*\bdie\b')
        
        def check(self, num, line):
                if self.nesteddie_re.match(line):