| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957 |
- # Copyright (c) 2003-2010 Sylvain Thenault (thenault@gmail.com).
- # Copyright (c) 2003-2012 LOGILAB S.A. (Paris, FRANCE).
- # http://www.logilab.fr/ -- mailto:contact@logilab.fr
- #
- # This program is free software; you can redistribute it and/or modify it under
- # the terms of the GNU General Public License as published by the Free Software
- # Foundation; either version 2 of the License, or (at your option) any later
- # version.
- #
- # This program is distributed in the hope that it will be useful, but WITHOUT
- # ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
- # FOR A PARTICULAR PURPOSE. See the GNU General Public License for more details
- #
- # You should have received a copy of the GNU General Public License along with
- # this program; if not, write to the Free Software Foundation, Inc.,
- # 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
- """ %prog [options] module_or_package
- Check that a module satisfy a coding standard (and more !).
- %prog --help
- Display this help message and exit.
- %prog --help-msg <msg-id>[,<msg-id>]
- Display help messages about given message identifiers and exit.
- """
- # import this first to avoid builtin namespace pollution
- from pylint.checkers import utils
- import sys
- import os
- import re
- import tokenize
- from warnings import warn
- from logilab.common.configuration import UnsupportedAction, OptionsManagerMixIn
- from logilab.common.optik_ext import check_csv
- from logilab.common.modutils import load_module_from_name
- from logilab.common.interface import implements
- from logilab.common.textutils import splitstrip
- from logilab.common.ureports import Table, Text, Section
- from logilab.common.__pkginfo__ import version as common_version
- from logilab.astng import MANAGER, nodes, ASTNGBuildingException
- from logilab.astng.__pkginfo__ import version as astng_version
- from pylint.utils import PyLintASTWalker, UnknownMessage, MessagesHandlerMixIn,\
- ReportsHandlerMixIn, MSG_TYPES, expand_modules
- from pylint.interfaces import ILinter, IRawChecker, IASTNGChecker
- from pylint.checkers import BaseRawChecker, EmptyReport, \
- table_lines_from_stats
- from pylint.reporters.text import TextReporter, ParseableTextReporter, \
- VSTextReporter, ColorizedTextReporter
- from pylint.reporters.html import HTMLReporter
- from pylint import config
- from pylint.__pkginfo__ import version
- OPTION_RGX = re.compile('\s*#*\s*pylint:(.*)')
- REPORTER_OPT_MAP = {'text': TextReporter,
- 'parseable': ParseableTextReporter,
- 'msvs': VSTextReporter,
- 'colorized': ColorizedTextReporter,
- 'html': HTMLReporter,}
- def _get_python_path(filepath):
- dirname = os.path.dirname(os.path.realpath(
- os.path.expanduser(filepath)))
- while True:
- if not os.path.exists(os.path.join(dirname, "__init__.py")):
- return dirname
- old_dirname = dirname
- dirname = os.path.dirname(dirname)
- if old_dirname == dirname:
- return os.getcwd()
- # Python Linter class #########################################################
- MSGS = {
- 'F0001': ('%s',
- 'Used when an error occurred preventing the analysis of a \
- module (unable to find it for instance).'),
- 'F0002': ('%s: %s',
- 'Used when an unexpected error occurred while building the ASTNG \
- representation. This is usually accompanied by a traceback. \
- Please report such errors !'),
- 'F0003': ('ignored builtin module %s',
- 'Used to indicate that the user asked to analyze a builtin module\
- which has been skipped.'),
- 'F0004': ('unexpected inferred value %s',
- 'Used to indicate that some value of an unexpected type has been \
- inferred.'),
- 'F0010': ('error while code parsing: %s',
- 'Used when an exception occured while building the ASTNG \
- representation which could be handled by astng.'),
- 'I0001': ('Unable to run raw checkers on built-in module %s',
- 'Used to inform that a built-in module has not been checked \
- using the raw checkers.'),
- 'I0010': ('Unable to consider inline option %r',
- 'Used when an inline option is either badly formatted or can\'t \
- be used inside modules.'),
- 'I0011': ('Locally disabling %s',
- 'Used when an inline option disables a message or a messages \
- category.'),
- 'I0012': ('Locally enabling %s',
- 'Used when an inline option enables a message or a messages \
- category.'),
- 'I0013': ('Ignoring entire file',
- 'Used to inform that the file will not be checked'),
- 'E0001': ('%s',
- 'Used when a syntax error is raised for a module.'),
- 'E0011': ('Unrecognized file option %r',
- 'Used when an unknown inline option is encountered.'),
- 'E0012': ('Bad option value %r',
- 'Used when a bad value for an inline option is encountered.'),
- }
- class PyLinter(OptionsManagerMixIn, MessagesHandlerMixIn, ReportsHandlerMixIn,
- BaseRawChecker):
- """lint Python modules using external checkers.
- This is the main checker controlling the other ones and the reports
- generation. It is itself both a raw checker and an astng checker in order
- to:
- * handle message activation / deactivation at the module level
- * handle some basic but necessary stats'data (number of classes, methods...)
- IDE plugins developpers: you may have to call
- `logilab.astng.builder.MANAGER.astng_cache.clear()` accross run if you want
- to ensure the latest code version is actually checked.
- """
- __implements__ = (ILinter, IRawChecker)
- name = 'master'
- priority = 0
- level = 0
- msgs = MSGS
- may_be_disabled = False
- @staticmethod
- def make_options():
- return (('ignore',
- {'type' : 'csv', 'metavar' : '<file>[,<file>...]',
- 'dest' : 'black_list', 'default' : ('CVS',),
- 'help' : 'Add files or directories to the blacklist. \
- They should be base names, not paths.'}),
- ('persistent',
- {'default': True, 'type' : 'yn', 'metavar' : '<y_or_n>',
- 'level': 1,
- 'help' : 'Pickle collected data for later comparisons.'}),
- ('load-plugins',
- {'type' : 'csv', 'metavar' : '<modules>', 'default' : (),
- 'level': 1,
- 'help' : 'List of plugins (as comma separated values of \
- python modules names) to load, usually to register additional checkers.'}),
- ('output-format',
- {'default': 'text', 'type': 'choice', 'metavar' : '<format>',
- 'choices': REPORTER_OPT_MAP.keys(),
- 'short': 'f',
- 'group': 'Reports',
- 'help' : 'Set the output format. Available formats are text,\
- parseable, colorized, msvs (visual studio) and html'}),
- ('include-ids',
- {'type' : 'yn', 'metavar' : '<y_or_n>', 'default' : 0,
- 'short': 'i',
- 'group': 'Reports',
- 'help' : 'Include message\'s id in output'}),
- ('files-output',
- {'default': 0, 'type' : 'yn', 'metavar' : '<y_or_n>',
- 'group': 'Reports', 'level': 1,
- 'help' : 'Put messages in a separate file for each module / \
- package specified on the command line instead of printing them on stdout. \
- Reports (if any) will be written in a file name "pylint_global.[txt|html]".'}),
- ('reports',
- {'default': 1, 'type' : 'yn', 'metavar' : '<y_or_n>',
- 'short': 'r',
- 'group': 'Reports',
- 'help' : 'Tells whether to display a full report or only the\
- messages'}),
- ('evaluation',
- {'type' : 'string', 'metavar' : '<python_expression>',
- 'group': 'Reports', 'level': 1,
- 'default': '10.0 - ((float(5 * error + warning + refactor + \
- convention) / statement) * 10)',
- 'help' : 'Python expression which should return a note less \
- than 10 (10 is the highest note). You have access to the variables errors \
- warning, statement which respectively contain the number of errors / warnings\
- messages and the total number of statements analyzed. This is used by the \
- global evaluation report (RP0004).'}),
- ('comment',
- {'default': 0, 'type' : 'yn', 'metavar' : '<y_or_n>',
- 'group': 'Reports', 'level': 1,
- 'help' : 'Add a comment according to your evaluation note. \
- This is used by the global evaluation report (RP0004).'}),
- ('enable',
- {'type' : 'csv', 'metavar': '<msg ids>',
- 'short': 'e',
- 'group': 'Messages control',
- 'help' : 'Enable the message, report, category or checker with the '
- 'given id(s). You can either give multiple identifier '
- 'separated by comma (,) or put this option multiple time.'}),
- ('disable',
- {'type' : 'csv', 'metavar': '<msg ids>',
- 'short': 'd',
- 'group': 'Messages control',
- 'help' : 'Disable the message, report, category or checker '
- 'with the given id(s). You can either give multiple identifier'
- ' separated by comma (,) or put this option multiple time '
- '(only on the command line, not in the configuration file '
- 'where it should appear only once).'}),
- )
- option_groups = (
- ('Messages control', 'Options controling analysis messages'),
- ('Reports', 'Options related to output formating and reporting'),
- )
- def __init__(self, options=(), reporter=None, option_groups=(),
- pylintrc=None):
- # some stuff has to be done before ancestors initialization...
- #
- # checkers / reporter / astng manager
- self.reporter = None
- self._checkers = {}
- self._ignore_file = False
- # visit variables
- self.base_name = None
- self.base_file = None
- self.current_name = None
- self.current_file = None
- self.stats = None
- # init options
- self.options = options + PyLinter.make_options()
- self.option_groups = option_groups + PyLinter.option_groups
- self._options_methods = {
- 'enable': self.enable,
- 'disable': self.disable}
- self._bw_options_methods = {'disable-msg': self.disable,
- 'enable-msg': self.enable}
- full_version = '%%prog %s, \nastng %s, common %s\nPython %s' % (
- version, astng_version, common_version, sys.version)
- OptionsManagerMixIn.__init__(self, usage=__doc__,
- version=full_version,
- config_file=pylintrc or config.PYLINTRC)
- MessagesHandlerMixIn.__init__(self)
- ReportsHandlerMixIn.__init__(self)
- BaseRawChecker.__init__(self)
- # provided reports
- self.reports = (('RP0001', 'Messages by category',
- report_total_messages_stats),
- ('RP0002', '% errors / warnings by module',
- report_messages_by_module_stats),
- ('RP0003', 'Messages',
- report_messages_stats),
- ('RP0004', 'Global evaluation',
- self.report_evaluation),
- )
- self.register_checker(self)
- self._dynamic_plugins = []
- self.load_provider_defaults()
- self.set_reporter(reporter or TextReporter(sys.stdout))
- def load_default_plugins(self):
- from pylint import checkers
- checkers.initialize(self)
- def load_plugin_modules(self, modnames):
- """take a list of module names which are pylint plugins and load
- and register them
- """
- for modname in modnames:
- if modname in self._dynamic_plugins:
- continue
- self._dynamic_plugins.append(modname)
- module = load_module_from_name(modname)
- module.register(self)
- def set_reporter(self, reporter):
- """set the reporter used to display messages and reports"""
- self.reporter = reporter
- reporter.linter = self
- def set_option(self, optname, value, action=None, optdict=None):
- """overridden from configuration.OptionsProviderMixin to handle some
- special options
- """
- if optname in self._options_methods or optname in self._bw_options_methods:
- if value:
- try:
- meth = self._options_methods[optname]
- except KeyError:
- meth = self._bw_options_methods[optname]
- warn('%s is deprecated, replace it by %s' % (
- optname, optname.split('-')[0]), DeprecationWarning)
- value = check_csv(None, optname, value)
- if isinstance(value, (list, tuple)):
- for _id in value :
- meth(_id)
- else :
- meth(value)
- elif optname == 'output-format':
- self.set_reporter(REPORTER_OPT_MAP[value.lower()]())
- try:
- BaseRawChecker.set_option(self, optname, value, action, optdict)
- except UnsupportedAction:
- print >> sys.stderr, 'option %s can\'t be read from config file' % \
- optname
- # checkers manipulation methods ############################################
- def register_checker(self, checker):
- """register a new checker
- checker is an object implementing IRawChecker or / and IASTNGChecker
- """
- assert checker.priority <= 0, 'checker priority can\'t be >= 0'
- self._checkers.setdefault(checker.name, []).append(checker)
- for r_id, r_title, r_cb in checker.reports:
- self.register_report(r_id, r_title, r_cb, checker)
- self.register_options_provider(checker)
- if hasattr(checker, 'msgs'):
- self.register_messages(checker)
- checker.load_defaults()
- def disable_noerror_messages(self):
- for msgcat, msgids in self._msgs_by_category.iteritems():
- if msgcat == 'E':
- for msgid in msgids:
- self.enable(msgid)
- else:
- for msgid in msgids:
- self.disable(msgid)
- def disable_reporters(self):
- """disable all reporters"""
- for reporters in self._reports.values():
- for report_id, _title, _cb in reporters:
- self.disable_report(report_id)
- def error_mode(self):
- """error mode: enable only errors; no reports, no persistent"""
- self.disable_noerror_messages()
- self.disable('miscellaneous')
- self.set_option('reports', False)
- self.set_option('persistent', False)
- # block level option handling #############################################
- #
- # see func_block_disable_msg.py test case for expected behaviour
- def process_tokens(self, tokens):
- """process tokens from the current module to search for module/block
- level options
- """
- comment = tokenize.COMMENT
- newline = tokenize.NEWLINE
- for (tok_type, _, start, _, line) in tokens:
- if tok_type not in (comment, newline):
- continue
- match = OPTION_RGX.search(line)
- if match is None:
- continue
- if match.group(1).strip() == "disable-all":
- self.add_message('I0013', line=start[0])
- self._ignore_file = True
- return
- try:
- opt, value = match.group(1).split('=', 1)
- except ValueError:
- self.add_message('I0010', args=match.group(1).strip(),
- line=start[0])
- continue
- opt = opt.strip()
- if opt in self._options_methods or opt in self._bw_options_methods:
- try:
- meth = self._options_methods[opt]
- except KeyError:
- meth = self._bw_options_methods[opt]
- warn('%s is deprecated, replace it by %s (%s, line %s)' % (
- opt, opt.split('-')[0], self.current_file, line),
- DeprecationWarning)
- for msgid in splitstrip(value):
- try:
- meth(msgid, 'module', start[0])
- except UnknownMessage:
- self.add_message('E0012', args=msgid, line=start[0])
- else:
- self.add_message('E0011', args=opt, line=start[0])
- def collect_block_lines(self, node, msg_state):
- """walk ast to collect block level options line numbers"""
- # recurse on children (depth first)
- for child in node.get_children():
- self.collect_block_lines(child, msg_state)
- first = node.fromlineno
- last = node.tolineno
- # first child line number used to distinguish between disable
- # which are the first child of scoped node with those defined later.
- # For instance in the code below:
- #
- # 1. def meth8(self):
- # 2. """test late disabling"""
- # 3. # pylint: disable=E1102
- # 4. print self.blip
- # 5. # pylint: disable=E1101
- # 6. print self.bla
- #
- # E1102 should be disabled from line 1 to 6 while E1101 from line 5 to 6
- #
- # this is necessary to disable locally messages applying to class /
- # function using their fromlineno
- if isinstance(node, (nodes.Module, nodes.Class, nodes.Function)) and node.body:
- firstchildlineno = node.body[0].fromlineno
- else:
- firstchildlineno = last
- for msgid, lines in msg_state.iteritems():
- for lineno, state in lines.items():
- if first <= lineno <= last:
- if lineno > firstchildlineno:
- state = True
- # set state for all lines for this block
- first, last = node.block_range(lineno)
- for line in xrange(first, last+1):
- # do not override existing entries
- if not line in self._module_msgs_state.get(msgid, ()):
- if line in lines: # state change in the same block
- state = lines[line]
- try:
- self._module_msgs_state[msgid][line] = state
- except KeyError:
- self._module_msgs_state[msgid] = {line: state}
- del lines[lineno]
- # code checking methods ###################################################
- def get_checkers(self):
- """return all available checkers as a list"""
- return [self] + [c for checkers in self._checkers.values()
- for c in checkers if c is not self]
- def prepare_checkers(self):
- """return checkers needed for activated messages and reports"""
- if not self.config.reports:
- self.disable_reporters()
- # get needed checkers
- neededcheckers = [self]
- for checker in self.get_checkers()[1:]:
- messages = set(msg for msg in checker.msgs
- if self.is_message_enabled(msg))
- if (messages or
- any(self.report_is_enabled(r[0]) for r in checker.reports)):
- neededcheckers.append(checker)
- checker.active_msgs = messages
- return neededcheckers
- def check(self, files_or_modules):
- """main checking entry: check a list of files or modules from their
- name.
- """
- self.reporter.include_ids = self.config.include_ids
- if not isinstance(files_or_modules, (list, tuple)):
- files_or_modules = (files_or_modules,)
- walker = PyLintASTWalker(self)
- checkers = self.prepare_checkers()
- rawcheckers = [c for c in checkers if implements(c, IRawChecker)
- and c is not self]
- # notify global begin
- for checker in checkers:
- checker.open()
- if implements(checker, IASTNGChecker):
- walker.add_checker(checker)
- # build ast and check modules or packages
- for descr in self.expand_files(files_or_modules):
- modname, filepath = descr['name'], descr['path']
- self.set_current_module(modname, filepath)
- # get the module representation
- astng = self.get_astng(filepath, modname)
- if astng is None:
- continue
- self.base_name = descr['basename']
- self.base_file = descr['basepath']
- if self.config.files_output:
- reportfile = 'pylint_%s.%s' % (modname, self.reporter.extension)
- self.reporter.set_output(open(reportfile, 'w'))
- self._ignore_file = False
- # fix the current file (if the source file was not available or
- # if it's actually a c extension)
- self.current_file = astng.file
- self.check_astng_module(astng, walker, rawcheckers)
- # notify global end
- self.set_current_module('')
- self.stats['statement'] = walker.nbstatements
- checkers.reverse()
- for checker in checkers:
- checker.close()
- def expand_files(self, modules):
- """get modules and errors from a list of modules and handle errors
- """
- result, errors = expand_modules(modules, self.config.black_list)
- for error in errors:
- message = modname = error["mod"]
- key = error["key"]
- self.set_current_module(modname)
- if key == "F0001":
- message = str(error["ex"]).replace(os.getcwd() + os.sep, '')
- self.add_message(key, args=message)
- return result
- def set_current_module(self, modname, filepath=None):
- """set the name of the currently analyzed module and
- init statistics for it
- """
- if not modname and filepath is None:
- return
- self.reporter.on_set_current_module(modname, filepath)
- self.current_name = modname
- self.current_file = filepath or modname
- self.stats['by_module'][modname] = {}
- self.stats['by_module'][modname]['statement'] = 0
- for msg_cat in MSG_TYPES.values():
- self.stats['by_module'][modname][msg_cat] = 0
- # XXX hack, to be correct we need to keep module_msgs_state
- # for every analyzed module (the problem stands with localized
- # messages which are only detected in the .close step)
- if modname:
- self._module_msgs_state = {}
- self._module_msg_cats_state = {}
- def get_astng(self, filepath, modname):
- """return a astng representation for a module"""
- try:
- return MANAGER.astng_from_file(filepath, modname, source=True)
- except SyntaxError, ex:
- self.add_message('E0001', line=ex.lineno, args=ex.msg)
- except ASTNGBuildingException, ex:
- self.add_message('F0010', args=ex)
- except Exception, ex:
- import traceback
- traceback.print_exc()
- self.add_message('F0002', args=(ex.__class__, ex))
- def check_astng_module(self, astng, walker, rawcheckers):
- """check a module from its astng representation, real work"""
- # call raw checkers if possible
- if not astng.pure_python:
- self.add_message('I0001', args=astng.name)
- else:
- #assert astng.file.endswith('.py')
- # invoke IRawChecker interface on self to fetch module/block
- # level options
- self.process_module(astng)
- if self._ignore_file:
- return False
- # walk ast to collect line numbers
- orig_state = self._module_msgs_state.copy()
- self._module_msgs_state = {}
- self.collect_block_lines(astng, orig_state)
- for checker in rawcheckers:
- checker.process_module(astng)
- # generate events to astng checkers
- walker.walk(astng)
- return True
- # IASTNGChecker interface #################################################
- def open(self):
- """initialize counters"""
- self.stats = { 'by_module' : {},
- 'by_msg' : {},
- }
- for msg_cat in MSG_TYPES.values():
- self.stats[msg_cat] = 0
- def close(self):
- """close the whole package /module, it's time to make reports !
- if persistent run, pickle results for later comparison
- """
- if self.base_name is not None:
- # load previous results if any
- previous_stats = config.load_results(self.base_name)
- # XXX code below needs refactoring to be more reporter agnostic
- self.reporter.on_close(self.stats, previous_stats)
- if self.config.reports:
- sect = self.make_reports(self.stats, previous_stats)
- if self.config.files_output:
- filename = 'pylint_global.' + self.reporter.extension
- self.reporter.set_output(open(filename, 'w'))
- else:
- sect = Section()
- if self.config.reports or self.config.output_format == 'html':
- self.reporter.display_results(sect)
- # save results if persistent run
- if self.config.persistent:
- config.save_results(self.stats, self.base_name)
- # specific reports ########################################################
- def report_evaluation(self, sect, stats, previous_stats):
- """make the global evaluation report"""
- # check with at least check 1 statements (usually 0 when there is a
- # syntax error preventing pylint from further processing)
- if stats['statement'] == 0:
- raise EmptyReport()
- # get a global note for the code
- evaluation = self.config.evaluation
- try:
- note = eval(evaluation, {}, self.stats)
- except Exception, ex:
- msg = 'An exception occurred while rating: %s' % ex
- else:
- stats['global_note'] = note
- msg = 'Your code has been rated at %.2f/10' % note
- if 'global_note' in previous_stats:
- msg += ' (previous run: %.2f/10)' % previous_stats['global_note']
- if self.config.comment:
- msg = '%s\n%s' % (msg, config.get_note_message(note))
- sect.append(Text(msg))
- # some reporting functions ####################################################
- def report_total_messages_stats(sect, stats, previous_stats):
- """make total errors / warnings report"""
- lines = ['type', 'number', 'previous', 'difference']
- lines += table_lines_from_stats(stats, previous_stats,
- ('convention', 'refactor',
- 'warning', 'error'))
- sect.append(Table(children=lines, cols=4, rheaders=1))
- def report_messages_stats(sect, stats, _):
- """make messages type report"""
- if not stats['by_msg']:
- # don't print this report when we didn't detected any errors
- raise EmptyReport()
- in_order = sorted([(value, msg_id)
- for msg_id, value in stats['by_msg'].items()
- if not msg_id.startswith('I')])
- in_order.reverse()
- lines = ('message id', 'occurrences')
- for value, msg_id in in_order:
- lines += (msg_id, str(value))
- sect.append(Table(children=lines, cols=2, rheaders=1))
- def report_messages_by_module_stats(sect, stats, _):
- """make errors / warnings by modules report"""
- if len(stats['by_module']) == 1:
- # don't print this report when we are analysing a single module
- raise EmptyReport()
- by_mod = {}
- for m_type in ('fatal', 'error', 'warning', 'refactor', 'convention'):
- total = stats[m_type]
- for module in stats['by_module'].keys():
- mod_total = stats['by_module'][module][m_type]
- if total == 0:
- percent = 0
- else:
- percent = float((mod_total)*100) / total
- by_mod.setdefault(module, {})[m_type] = percent
- sorted_result = []
- for module, mod_info in by_mod.items():
- sorted_result.append((mod_info['error'],
- mod_info['warning'],
- mod_info['refactor'],
- mod_info['convention'],
- module))
- sorted_result.sort()
- sorted_result.reverse()
- lines = ['module', 'error', 'warning', 'refactor', 'convention']
- for line in sorted_result:
- if line[0] == 0 and line[1] == 0:
- break
- lines.append(line[-1])
- for val in line[:-1]:
- lines.append('%.2f' % val)
- if len(lines) == 5:
- raise EmptyReport()
- sect.append(Table(children=lines, cols=5, rheaders=1))
- # utilities ###################################################################
- # this may help to import modules using gettext
- try:
- __builtins__._ = str
- except AttributeError:
- __builtins__['_'] = str
- class ArgumentPreprocessingError(Exception):
- """Raised if an error occurs during argument preprocessing."""
- def preprocess_options(args, search_for):
- """look for some options (keys of <search_for>) which have to be processed
- before others
- values of <search_for> are callback functions to call when the option is
- found
- """
- i = 0
- while i < len(args):
- arg = args[i]
- if arg.startswith('--'):
- try:
- option, val = arg[2:].split('=', 1)
- except ValueError:
- option, val = arg[2:], None
- try:
- cb, takearg = search_for[option]
- del args[i]
- if takearg and val is None:
- if i >= len(args) or args[i].startswith('-'):
- raise ArgumentPreprocessingError(arg)
- val = args[i]
- del args[i]
- cb(option, val)
- except KeyError:
- i += 1
- else:
- i += 1
- class Run:
- """helper class to use as main for pylint :
- run(*sys.argv[1:])
- """
- LinterClass = PyLinter
- option_groups = (
- ('Commands', 'Options which are actually commands. Options in this \
- group are mutually exclusive.'),
- )
- def __init__(self, args, reporter=None, exit=True):
- self._rcfile = None
- self._plugins = []
- try:
- preprocess_options(args, {
- # option: (callback, takearg)
- 'rcfile': (self.cb_set_rcfile, True),
- 'load-plugins': (self.cb_add_plugins, True),
- })
- except ArgumentPreprocessingError, e:
- print >> sys.stderr, 'Argument %s expects a value.' % (e.args[0],)
- sys.exit(32)
- self.linter = linter = self.LinterClass((
- ('rcfile',
- {'action' : 'callback', 'callback' : lambda *args: 1,
- 'type': 'string', 'metavar': '<file>',
- 'help' : 'Specify a configuration file.'}),
- ('init-hook',
- {'action' : 'callback', 'type' : 'string', 'metavar': '<code>',
- 'callback' : cb_init_hook, 'level': 1,
- 'help' : 'Python code to execute, usually for sys.path \
- manipulation such as pygtk.require().'}),
- ('help-msg',
- {'action' : 'callback', 'type' : 'string', 'metavar': '<msg-id>',
- 'callback' : self.cb_help_message,
- 'group': 'Commands',
- 'help' : '''Display a help message for the given message id and \
- exit. The value may be a comma separated list of message ids.'''}),
- ('list-msgs',
- {'action' : 'callback', 'metavar': '<msg-id>',
- 'callback' : self.cb_list_messages,
- 'group': 'Commands', 'level': 1,
- 'help' : "Generate pylint's messages."}),
- ('full-documentation',
- {'action' : 'callback', 'metavar': '<msg-id>',
- 'callback' : self.cb_full_documentation,
- 'group': 'Commands', 'level': 1,
- 'help' : "Generate pylint's full documentation."}),
- ('generate-rcfile',
- {'action' : 'callback', 'callback' : self.cb_generate_config,
- 'group': 'Commands',
- 'help' : '''Generate a sample configuration file according to \
- the current configuration. You can put other options before this one to get \
- them in the generated configuration.'''}),
- ('generate-man',
- {'action' : 'callback', 'callback' : self.cb_generate_manpage,
- 'group': 'Commands',
- 'help' : "Generate pylint's man page.",'hide': True}),
- ('errors-only',
- {'action' : 'callback', 'callback' : self.cb_error_mode,
- 'short': 'E',
- 'help' : '''In error mode, checkers without error messages are \
- disabled and for others, only the ERROR messages are displayed, and no reports \
- are done by default'''}),
- ('profile',
- {'type' : 'yn', 'metavar' : '<y_or_n>',
- 'default': False, 'hide': True,
- 'help' : 'Profiled execution.'}),
- ), option_groups=self.option_groups,
- reporter=reporter, pylintrc=self._rcfile)
- # register standard checkers
- linter.load_default_plugins()
- # load command line plugins
- linter.load_plugin_modules(self._plugins)
- # add some help section
- linter.add_help_section('Environment variables', config.ENV_HELP, level=1)
- linter.add_help_section('Output', '''
- Using the default text output, the message format is :
-
- MESSAGE_TYPE: LINE_NUM:[OBJECT:] MESSAGE
-
- There are 5 kind of message types :
- * (C) convention, for programming standard violation
- * (R) refactor, for bad code smell
- * (W) warning, for python specific problems
- * (E) error, for probable bugs in the code
- * (F) fatal, if an error occurred which prevented pylint from doing further
- processing.
- ''', level=1)
- linter.add_help_section('Output status code', '''
- Pylint should leave with following status code:
- * 0 if everything went fine
- * 1 if a fatal message was issued
- * 2 if an error message was issued
- * 4 if a warning message was issued
- * 8 if a refactor message was issued
- * 16 if a convention message was issued
- * 32 on usage error
-
- status 1 to 16 will be bit-ORed so you can know which different categories has
- been issued by analysing pylint output status code
- ''', level=1)
- # read configuration
- linter.disable('W0704')
- linter.read_config_file()
- # is there some additional plugins in the file configuration, in
- config_parser = linter.cfgfile_parser
- if config_parser.has_option('MASTER', 'load-plugins'):
- plugins = splitstrip(config_parser.get('MASTER', 'load-plugins'))
- linter.load_plugin_modules(plugins)
- # now we can load file config and command line, plugins (which can
- # provide options) have been registered
- linter.load_config_file()
- if reporter:
- # if a custom reporter is provided as argument, it may be overridden
- # by file parameters, so re-set it here, but before command line
- # parsing so it's still overrideable by command line option
- linter.set_reporter(reporter)
- try:
- args = linter.load_command_line_configuration(args)
- except SystemExit, exc:
- if exc.code == 2: # bad options
- exc.code = 32
- raise
- if not args:
- print linter.help()
- sys.exit(32)
- # insert current working directory to the python path to have a correct
- # behaviour
- if len(args) == 1:
- sys.path.insert(0, _get_python_path(args[0]))
- else:
- sys.path.insert(0, os.getcwd())
- if self.linter.config.profile:
- print >> sys.stderr, '** profiled run'
- import cProfile, pstats
- cProfile.runctx('linter.check(%r)' % args, globals(), locals(), 'stones.prof' )
- data = pstats.Stats('stones.prof')
- data.strip_dirs()
- data.sort_stats('time', 'calls')
- data.print_stats(30)
- else:
- linter.check(args)
- sys.path.pop(0)
- if exit:
- sys.exit(self.linter.msg_status)
- def cb_set_rcfile(self, name, value):
- """callback for option preprocessing (i.e. before optik parsing)"""
- self._rcfile = value
- def cb_add_plugins(self, name, value):
- """callback for option preprocessing (i.e. before optik parsing)"""
- self._plugins.extend(splitstrip(value))
- def cb_error_mode(self, *args, **kwargs):
- """error mode:
- * disable all but error messages
- * disable the 'miscellaneous' checker which can be safely deactivated in
- debug
- * disable reports
- * do not save execution information
- """
- self.linter.error_mode()
- def cb_generate_config(self, *args, **kwargs):
- """optik callback for sample config file generation"""
- self.linter.generate_config(skipsections=('COMMANDS',))
- sys.exit(0)
- def cb_generate_manpage(self, *args, **kwargs):
- """optik callback for sample config file generation"""
- from pylint import __pkginfo__
- self.linter.generate_manpage(__pkginfo__)
- sys.exit(0)
- def cb_help_message(self, option, optname, value, parser):
- """optik callback for printing some help about a particular message"""
- self.linter.help_message(splitstrip(value))
- sys.exit(0)
- def cb_full_documentation(self, option, optname, value, parser):
- """optik callback for printing full documentation"""
- self.linter.print_full_documentation()
- sys.exit(0)
- def cb_list_messages(self, option, optname, value, parser): # FIXME
- """optik callback for printing available messages"""
- self.linter.list_messages()
- sys.exit(0)
- def cb_init_hook(option, optname, value, parser):
- """exec arbitrary code to set sys.path for instance"""
- exec value
- if __name__ == '__main__':
- Run(sys.argv[1:])
|