diff options
| author | Zac Medico <zmedico@gentoo.org> | 2008-03-28 12:41:23 +0000 |
|---|---|---|
| committer | Zac Medico <zmedico@gentoo.org> | 2008-03-28 12:41:23 +0000 |
| commit | df269fa89f00eacabf855b88ce0dcf5add6b8fd6 (patch) | |
| tree | 2b65fa63b1e9e9e0202f6b5b85a8dd32a9000ae9 /bin/repoman | |
| parent | 486628dd25d0731808d10b4832afefad90ee8959 (diff) | |
| download | portage-df269fa89f00eacabf855b88ce0dcf5add6b8fd6.tar.gz portage-df269fa89f00eacabf855b88ce0dcf5add6b8fd6.tar.bz2 portage-df269fa89f00eacabf855b88ce0dcf5add6b8fd6.zip | |
Bug #212621 - Make the EbuildQuote check do quoting validation for misc
variables from games.eclass. (trunk r9507)
svn path=/main/branches/2.1.2/; revision=9569
Diffstat (limited to 'bin/repoman')
| -rwxr-xr-x | bin/repoman | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/bin/repoman b/bin/repoman index db1dbd90c..f927fc221 100755 --- a/bin/repoman +++ b/bin/repoman @@ -939,7 +939,14 @@ class EbuildQuote(LineCheck): repoman_check_name = 'ebuild.minorsyn' ignore_line = re.compile(r'(^$)|(^\s*#.*)|(^\s*\w+=.*)|(^\s*(local|export)\s+)') - var_names = r'(D|DISTDIR|FILESDIR|S|T|ROOT|WORKDIR)' + var_names = ["D", "DISTDIR", "FILESDIR", "S", "T", "ROOT", "WORKDIR"] + + # variables for games.eclass + var_names += ["Ddir", "dir", "GAMES_PREFIX_OPT", "GAMES_DATADIR", + "GAMES_DATADIR_BASE", "GAMES_SYSCONFDIR", "GAMES_STATEDIR", + "GAMES_LOGDIR", "GAMES_BINDIR"] + + var_names = "(%s)" % "|".join(var_names) var_reference = re.compile(r'\$(\{'+var_names+'\}|' + \ var_names + '\W)') missing_quotes = re.compile(r'(\s|^)[^"\'\s]*\$\{?' + var_names + \ |