Currently the pylint script shows false positives for source files that use python packages which are not installed.
Make it so that it prints a warning about the missing packages.
Improvement proposal: there was an idea on the meeting today that there should be a ignore list for files/directories we don't want to scan. This would apply for example for dist directory or entitle.py - if we decide that we don't want to install the python package python-rhsm with FreeIPA.
dist
entitle.py
Ignore list is already in, it was there from the beginning.
Yes but dist needs to be added to the list.
And you may be able to dynamically add to it based on failed imports. Right now there are only a couple of optional components so you can do something like:
# convert IGNORE_* to lists instead of tuples try: from rhsm.certificate import EntitlementCertificate except ImportError: IGNORE_FILES.append('ipalib/plugins/entitle.py')
master: 7e0d453
ipa-2-0: 8522388
Metadata Update from @jcholast: - Issue assigned to jcholast - Issue set to the milestone: FreeIPA 2.1 - 2011/08 (Final)
Log in to comment on this ticket.