[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [PATCH 1/3] Ignore several false positives and import errors while running pylint.



On Wed, 10 Nov 2010, Chris Lumens wrote:

(1) I don't know why it tries to do this, but it always fails and causes the
following:

   F0002:  1: <type 'exceptions.RuntimeError'>: could not open display

(2) Also, this keeps coming up:

   E1101: 65:randomIname: Module 'hashlib' has no 'md5' member

But I do not see how it's a valid bug:

In [1]: import hashlib
In [2]: m = hashlib.md5()
In [3]:

(3) Finally, there's this from several places:

   E0611: 26: No name '_isys' in module 'pyanaconda'

This is not a bug because _isys.so is in .libs/, which will work just fine
during runtime but is only a problem for pylint due to our "from pyanaconda
import _isys" lines.
---
tests/pylint/pylint-false-positives |    4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/tests/pylint/pylint-false-positives b/tests/pylint/pylint-false-positives
index da6ae9a..e4d04ff 100644
--- a/tests/pylint/pylint-false-positives
+++ b/tests/pylint/pylint-false-positives
@@ -1,3 +1,4 @@
+^F0002:[ 0-9]*: <type 'exceptions.RuntimeError'>: could not open display$
^F0401:[ 0-9]*: Unable to import 'gtk.gdk'$
^F0401:[ 0-9]*: Unable to import 'repomd.mdErrors'$
^F0401:[ 0-9]*: Unable to import 'volume_key'$
@@ -5,6 +6,8 @@
^W0122:[ 0-9]*:InstallInterface\.run: Use of the exec statement$
^E0602:[ 0-9]*:InstallInterface\.run: Undefined variable 'nextWin'$
^E0602:[ 0-9]*:sparcBootloaderInfo\.writeSilo: Undefined variable 'butil'$
+^E0611:[ 0-9]*: No name '_isys' in module 'pyanaconda'$
+^E0611:[ 0-9]*:.*: No name '_isys' in module 'pyanaconda'$
^E1101:[ 0-9]*:.*: Class 'vbox' has no 'pack_start' member$
^E1101:[ 0-9]*:.*: Instance of 'WideCheckList' has no 'get_model' member$
^E1101:[ 0-9]*:.*: Instance of 'WideCheckList' has no 'get_column' member$
@@ -15,6 +18,7 @@
^E1101:[ 0-9]*:AnacondaTZMap\.selectionChanged: Instance of 'Enum' has no 'ENTRY' member$
^E1101:[ 0-9]*:DeviceAction\..*: Instance of 'DeviceAction' has no 'dir' member$
^E1101:[ 0-9]*:VolumeGroupEditor\.editLogicalVolume: Class 'format' has no 'mountable' member$
+^E1101:[ 0-9]*:randomIname: Module 'hashlib' has no 'md5' member$
^E1101:[ 0-9]*:rpm.*: Instance of 'TransactionSet' has no 'dbMatch' member$
^E1101:[ 0-9]*:EntryWindow.__init__: Class 'child' has no 'pack_start' member$
^E1101:[ 0-9]*:.*: Instance of 'dict' has no 'Get' member$


Seems like a pylint bug that we should file.  I'm ok with making these
changes, but perhaps we should get someone from pylint to take a look at
what's happening.

--
David Cantrell <dcantrell redhat com>
Red Hat / Honolulu, HI


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]