summaryrefslogtreecommitdiffstats
path: root/pym/repoman
Commit message (Collapse)AuthorAgeFilesLines
* Make the EbuildQuote.missing_quotes regex accept singleZac Medico2007-11-101-2/+2
| | | | | | | | quotes where it accepts double quotes in order to eliminate some false positives. Thanks to Krzysiek Pawlik <nelchael@gentoo.org> for reporting. svn path=/main/trunk/; revision=8485
* Convert all the checks in repoman.checks to use a new LineCheckZac Medico2007-11-101-179/+106
| | | | | | | | | | | | | | | | | interface that takes a single line as an argument. This has at least a few of benefits: * Eliminates lots of redundant code * Error messages are ordered by line number across all checks The performance is slightly worse due to the increased number of method calls, but it's not really noticeable in comparison to the time consumed by dependency checks. Thanks to Petteri Räty <betelgeuse@gentoo.org> for the initial patch which I only made a few minor modifications to. svn path=/main/trunk/; revision=8483
* Fix 'TypeError: not all arguments converted during string formatting'Zac Medico2007-11-051-1/+1
| | | | | | issues with the EbuildNestedDie check. svn path=/main/trunk/; revision=8429
* Properly escape { and } characters in regular expressions evenZac Medico2007-11-011-3/+3
| | | | | | though it seems to work either way. svn path=/main/trunk/; revision=8355
* Remove an extra \$ from the EbuildQuote.var_reference regexZac Medico2007-11-011-1/+1
| | | | | | | so that it will properly match things like variable references without braces, like $S. svn path=/main/trunk/; revision=8354
* Make the EbuildUselessCdS check for both quoted and unquoted ${S}Zac Medico2007-11-011-1/+1
| | | | | | | so the user doesn't have to fix quoting before this check will recognize the violation. svn path=/main/trunk/; revision=8353
* Add a check for redundant cd "${S}" statements on the firstZac Medico2007-11-012-1/+23
| | | | | | | line of src_(compile|install|test) ebuild methods. Thanks to Petteri Räty <betelgeuse@gentoo.org> for this patch. svn path=/main/trunk/; revision=8351
* Bug #196652 - Check for useless ABOUT-NLS|COPYING|LICENSEZac Medico2007-10-231-0/+19
| | | | | | files in dodoc arguments. svn path=/main/trunk/; revision=8236
* Make the EbuildQuote check ignore lines beginningZac Medico2007-10-051-1/+1
| | | | | | | with local or export builtins since unquoted assignments are okay there. svn path=/main/trunk/; revision=7942
* In the EbuildQuote check, combine the var_assignmentZac Medico2007-10-031-4/+1
| | | | | | | | regex together with the ignore_line regex. This also fixes an issue with the var_assignment regex allowing violations like `make DESTDIR=${D}` to slip through. svn path=/main/trunk/; revision=7904
* Filter some false positives out of the EbuildQuote check.Zac Medico2007-10-021-13/+48
| | | | | | | It might work well enough now so that we don't have to degrade it to a warning. svn path=/main/trunk/; revision=7901
* Rename checks as the naming was overkill (they are all obviously checks ↵Alec Warner2007-10-012-6/+24
| | | | | | being in the check module), add nesteddie check svn path=/main/trunk/; revision=7898
* keep old repoman names so we don't confuse peopleAlec Warner2007-10-011-1/+1
| | | | svn path=/main/trunk/; revision=7896
* Add new repoman check classes using StringIO; possibly need some testing, ↵Alec Warner2007-10-013-0/+189
obviously the old code was faster (1 iteration over the file), here we do one iteration per check, StringIO was to try and negate this by doing the checks in memory...how much of a price do we pay here? svn path=/main/trunk/; revision=7894