#1184 Warn about missing python packages in pylint
Closed: Fixed None Opened 14 years ago by jcholast.

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.

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')

Metadata Update from @jcholast:
- Issue assigned to jcholast
- Issue set to the milestone: FreeIPA 2.1 - 2011/08 (Final)

8 years ago

Log in to comment on this ticket.

Metadata