diff options
author | Zac Medico <zmedico@gentoo.org> | 2007-11-01 17:49:49 +0000 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2007-11-01 17:49:49 +0000 |
commit | 3a95784f6ba9ebc2b79a96d1220d23497bc1a360 (patch) | |
tree | 8f850443b4ed21032fbfbeeb80d1dc0ca26b9185 | |
parent | dd012f0e9388b829d740763f094a5a9977cb9c4f (diff) | |
download | portage-3a95784f6ba9ebc2b79a96d1220d23497bc1a360.tar.gz portage-3a95784f6ba9ebc2b79a96d1220d23497bc1a360.tar.bz2 portage-3a95784f6ba9ebc2b79a96d1220d23497bc1a360.zip |
Make the EbuildUselessCdS check for both quoted and unquoted ${S}
so the user doesn't have to fix quoting before this check will
recognize the violation.
svn path=/main/trunk/; revision=8353
-rw-r--r-- | pym/repoman/checks.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pym/repoman/checks.py b/pym/repoman/checks.py index eea7ba40f..d6fcc1318 100644 --- a/pym/repoman/checks.py +++ b/pym/repoman/checks.py @@ -248,7 +248,7 @@ class EbuildUselessCdS(ContentCheck): """Check for redundant cd ${S} statements""" repoman_check_name = 'ebuild.minorsyn' method_re = re.compile(r'^\s*src_(compile|install|test)\s*\(\)') - cds_re = re.compile(r'^\s*cd\s+"\$(\{S\}|S)"\s') + cds_re = re.compile(r'^\s*cd\s+("\$(\{S\}|S)"|\$(\{S\}|S))\s') def __init__(self, contents): ContentCheck.__init__(self, contents) |