[PATCH] pylint up, pychecker down.

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

 



---
 Makefile.am               |   12 +--
 pychecker-false-positives |   26 -----
 pycheckrc-for-anaconda    |  223 ---------------------------------------------
 runpychecker.sh           |   59 ------------
 4 files changed, 2 insertions(+), 318 deletions(-)
 delete mode 100644 pychecker-false-positives
 delete mode 100644 pycheckrc-for-anaconda
 delete mode 100755 runpychecker.sh

diff --git a/Makefile.am b/Makefile.am
index c1c6e08..c9955c6 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -23,8 +23,8 @@ SUBDIRS = isys po loader booty storage textw utils scripts bootdisk \
           installclasses iw command-stubs ui docs pixmaps tests fonts \
           gptsync liveinst
 
-EXTRA_DIST = config.rpath COPYING pychecker-false-positives \
-             pycheckrc-for-anaconda runpychecker.sh lang-table \
+EXTRA_DIST = config.rpath COPYING \
+             lang-table \
              anaconda.spec.in
 
 MAINTAINERCLEANFILES = Makefile.in config.guess config.h.in config.sub \
@@ -58,8 +58,6 @@ xutils_la_SOURCES     = xutils.c
 anacondadir     = $(pkgpyexecdir)
 anaconda_PYTHON = *.py
 
-PYCHECKERPATH = isys:textw:iw:installclasses:/usr/share/system-config-date
-PYCHECKEROPTS = -F pycheckrc-for-anaconda
 ARCHIVE_TAG   = $(PACKAGE_NAME)-$(PACKAGE_VERSION)-$(PACKAGE_RELEASE)
 
 sed_verbose = $(sed_verbose_$(V))
@@ -97,12 +95,6 @@ src: archive
 	@rpmbuild -ts --nodeps $(PACKAGE_NAME)-$(PACKAGE_VERSION).tar.bz2 || exit 1
 	@rm -f $(PACKAGE_NAME)-$(PACKAGE_VERSION).tar.bz2
 
-pycheck:
-	PYTHONPATH=$(PYCHECKERPATH) pychecker $(PYCHECKEROPTS) *.py textw/*.py iw/*.py installclasses/*.py storage/*.py | grep -v "__init__() not called"
-
-pycheck-file:
-	PYTHONPATH=.:$(PYCHECKERPATH) pychecker $(PYCHECKEROPTS) $(CHECK) | grep -v "__init__() not called"
-
 api:
 	doxygen docs/api.cfg
 
diff --git a/pychecker-false-positives b/pychecker-false-positives
deleted file mode 100644
index 69f7cf2..0000000
--- a/pychecker-false-positives
+++ /dev/null
@@ -1,26 +0,0 @@
-deleted
-filter
-setattr
-^$
-^dm.c: [0-9]*: not running as root returning empty list$
-^Warnings...$
-^anaconda\.py:[0-9]*: No global .* found$
-^cmdline\.py:[0-9]*: No global \(nextWin\) found$
-^fsset\.py:[0-9]*: Object \(bestprep\) has no attribute \(format\)$
-^partitions.py:[0-9]*: Object \(bestprep\) has no attribute \(getPreExisting\)$
-^partitions.py:[0-9]*: Object \(bestreq\) has no attribute \(format\)$
-^iscsi.py:[0-9]*: Invalid arguments to \(addTarget\), got 0, expected between 1 and 7$
-^(.*isys/isys.py|network.py):[0-9]*: Object \(.*_iface\) has no attribute \(Get.*\)$
-^network.py:[0-9]*: Object \(.*props\) has no attribute \(Get\)$
-^kickstart.py:[0-9]*: No global \(fileSystemTypeGet.*\) found$
-^kickstart.py:[0-9]*: Object \(kwargs\) has no attribute \(update\)$
-^upgrade.py:[0-9]*: No global \(getUsableLinuxFs\) found$
-^upgrade.py:[0-9]*: No global \(SwapFileDevice\) found$
-^upgrade.py:[0-9]*: No global \(fileSystemTypeGet\) found$
-^upgrade.py:[0-9]*: No global \(FileSystemSetEntry\) found$
-^upgrade.py:[0-9]*: No global \(readFstab\) found$
-^iutil.py:[0-9]*: Object \(ppcType\) has no attribute \(items\)$
-^livecd.py:[0-9]*: Object \(fsdict\) has no attribute \(keys\)$
-^packages.py:[0-9]*: Object \(publicBetas\) has no attribute \(items\)$
-^partRequests.py:[0-9]*: Format string argument count \(0\) doesn't match arguments \(1\)$
-Note this last line must never end with a newline
\ No newline at end of file
diff --git a/pycheckrc-for-anaconda b/pycheckrc-for-anaconda
deleted file mode 100644
index dc2c2e0..0000000
--- a/pycheckrc-for-anaconda
+++ /dev/null
@@ -1,223 +0,0 @@
-#
-# .pycheckrc file created by PyChecker v0.8.17 @ Wed Sep 19 12:18:52 2007
-#
-# It should be placed in your home directory (value of $HOME).
-# If $HOME is not set, it will look in the current directory.
-#
-
-# only warn about files passed on the command line
-only = 1
-
-# the maximum number of warnings to be displayed
-limit = 0
-
-# list of evil C extensions that crash the interpreter
-evil = []
-
-# unused imports
-importUsed = 1
-
-# unused imports from __init__.py
-packageImportUsed = 1
-
-# module imports itself
-reimportSelf = 1
-
-# reimporting a module
-moduleImportErrors = 1
-
-# module does import and from ... import
-mixImport = 1
-
-# unused local variables, except tuples
-localVariablesUsed = 0
-
-# all unused local variables, including tuples
-unusedLocalTuple = 0
-
-# all unused class data members
-membersUsed = 0
-
-# all unused module variables
-allVariablesUsed = 0
-
-# unused private module variables
-privateVariableUsed = 1
-
-# report each occurrence of global warnings
-reportAllGlobals = 0
-
-# functions called with named arguments (like keywords)
-namedArgs = 0
-
-# Attributes (members) must be defined in __init__()
-onlyCheckInitForMembers = 0
-
-# Subclass.__init__() not defined
-initDefinedInSubclass = 0
-
-# Baseclass.__init__() not called
-baseClassInitted = 1
-
-# Subclass needs to override methods that only throw exceptions
-abstractClasses = 1
-
-# Return None from __init__()
-returnNoneFromInit = 1
-
-# unreachable code
-unreachableCode = 0
-
-# a constant is used in a conditional statement
-constantConditions = 1
-
-# 1 is used in a conditional statement (if 1: or while 1:)
-constant1 = 0
-
-# check if iterating over a string
-stringIteration = 1
-
-# check improper use of string.find()
-stringFind = 1
-
-# Calling data members as functions
-callingAttribute = 0
-
-# class attribute does not exist
-classAttrExists = 1
-
-# First argument to methods
-methodArgName = 'self'
-
-# First argument to classmethods
-classmethodArgNames = ['cls', 'klass']
-
-# unused method/function arguments
-argumentsUsed = 0
-
-# unused method/function variable arguments
-varArgumentsUsed = 1
-
-# ignore if self is unused in methods
-ignoreSelfUnused = 0
-
-# check if overridden methods have the same signature
-checkOverridenMethods = 0
-
-# check if __special__ methods exist and have the correct signature
-checkSpecialMethods = 1
-
-# check if function/class/method names are reused
-redefiningFunction = 1
-
-# check if using unary positive (+) which is usually meaningless
-unaryPositive = 1
-
-# check if modify (call method) on a parameter that has a default value
-modifyDefaultValue = 1
-
-# check if variables are set to different types
-inconsistentTypes = 0
-
-# check if unpacking a non-sequence
-unpackNonSequence = 1
-
-# check if unpacking sequence with the wrong length
-unpackLength = 1
-
-# check if raising or catching bad exceptions
-badExceptions = 1
-
-# check if statement appears to have no effect
-noEffect = 1
-
-# check if using (expr % 1), it has no effect on integers and strings
-modulo1 = 1
-
-# check if using (expr is const-literal), doesn't always work on integers and strings
-isLiteral = 1
-
-# check consistent return values
-checkReturnValues = 1
-
-# check if using implict and explicit return values
-checkImplicitReturns = 1
-
-# check that attributes of objects exist
-checkObjectAttrs = 1
-
-# various warnings about incorrect usage of __slots__
-slots = 1
-
-# using properties with classic classes
-classicProperties = 1
-
-# check if __slots__ is empty
-emptySlots = 1
-
-# check if using integer division
-intDivide = 1
-
-# check if local variable shadows a global
-shadows = 0
-
-# check if a variable shadows a builtin
-shadowBuiltins = 0
-
-# check if input() is used
-usesInput = 1
-
-# check if the exec statement is used
-usesExec = 0
-
-# ignore warnings from files under standard library
-ignoreStandardLibrary = 0
-
-# ignore warnings from the list of modules
-blacklist = ['Tkinter', 'wxPython', 'gtk', 'GTK', 'GDK']
-
-# ignore global variables not used if name is one of these values
-variablesToIgnore = ['__version__', '__warningregistry__', '__all__', '__credits__', '__test__', '__author__', '__email__', '__revision__', '__id__', '__copyright__', '__license__', '__date__']
-
-# ignore unused locals/arguments if name is one of these values
-unusedNames = ['_', 'empty', 'unused', 'dummy']
-
-# ignore use of deprecated modules/functions
-deprecated = 1
-
-# maximum lines in a function
-maxLines = 0
-
-# maximum branches in a function
-maxBranches = 0
-
-# maximum returns in a function
-maxReturns = 0
-
-# maximum # of arguments to a function
-maxArgs = 0
-
-# maximum # of locals in a function
-maxLocals = 0
-
-# maximum # of identifier references (Law of Demeter)
-maxReferences = 5
-
-# no module doc strings
-noDocModule = 0
-
-# no class doc strings
-noDocClass = 0
-
-# no function/method doc strings
-noDocFunc = 0
-
-# print internal checker parse structures
-printParse = 0
-
-# turn on debugging for checker
-debug = 0
-
-# turn off all output except warnings
-quiet = 0
-
diff --git a/runpychecker.sh b/runpychecker.sh
deleted file mode 100755
index 186a551..0000000
--- a/runpychecker.sh
+++ /dev/null
@@ -1,59 +0,0 @@
-#!/bin/bash
-
-# This script will check anaconda for any pychecker warning using a set of
-# options minimizing false positives, in combination with filtering of any
-# warning regularexpressions listed in pychecker-false-positives.
-# 
-# If any warnings our found they will be stored in pychecker-log and printed
-# to stdout and this script will exit with a status of 1, if no (non filtered)
-# warnings are found it exits with a status of 0
-
-FALSE_POSITIVES=pychecker-false-positives
-NON_STRICT_OPTIONS="--no-deprecated --no-returnvalues --no-abstract"
-
-usage () {
-  echo "usage: `basename $0` [--strict] [--help]"
-  exit $1
-}
-
-while [ $# -gt 0 ]; do
-  case $1 in
-    --strict)
-      NON_STRICT_OPTIONS=""
-      ;;
-    --help)
-      usage 0
-      ;;
-    *)
-      echo "Error unknown option: $1"
-      usage 1
-  esac
-  shift
-done
-
-if [ "`tail -c 1 pychecker-false-positives`" == "`echo`" ]; then
-  echo "Error $FALSE_POSITIVES ends with an enter."
-  echo "Error the last line of $FALSE_POSITIVES should never have an enter!"
-  exit 1
-fi
-
-export PYTHONPATH=".:.libs:isys:isys/.libs:textw:iw:installclasses:/usr/share/system-config-date"
-
-pychecker --only --limit 1000 \
-  --maxlines 500 --maxargs 20 --maxbranches 80 --maxlocals 60 --maxreturns 20 \
-  --no-callinit --no-local --no-shadow --no-shadowbuiltin \
-  --no-import --no-miximport --no-pkgimport --no-reimport \
-  --no-argsused --no-varargsused --no-override \
-  $NON_STRICT_OPTIONS \
-  anaconda anaconda *.py textw/*.py iw/*.py installclasses/*.py isys/*.py booty/*.py booty/*/*.py | \
-  egrep -v "`cat $FALSE_POSITIVES | tr '\n' '|'`" > pychecker-log
-
-if [ -s pychecker-log ]; then
-  echo "Pychecker reports the following issues:"
-  cat pychecker-log
-  exit 1
-fi
-
-rm pychecker-log
-
-exit 0
-- 
1.6.5.1

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux