Fixes additional false positives found in install/po/test_i18n.py and a missing import in ipa-nis-manage.

This should be the last patch dealing with pylint-discovered issues.

--
Jan Cholasta
>From 6937c6aa5966e4dfcd1586710b90c3118f76869d Mon Sep 17 00:00:00 2001
From: Jan Cholasta <jchol...@redhat.com>
Date: Thu, 14 Apr 2011 10:17:04 +0200
Subject: [PATCH] Fix some minor issues.

---
 install/po/test_i18n.py      |    4 ++--
 install/tools/ipa-nis-manage |    1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/install/po/test_i18n.py b/install/po/test_i18n.py
index f4de7c9..0797dd7 100755
--- a/install/po/test_i18n.py
+++ b/install/po/test_i18n.py
@@ -80,7 +80,7 @@ def main():
         print "Using message string \"%s\" found in file \"%s\"" % (msgid, test_file)
 
         # Get the translated version of the msgid string by invoking _()
-        translated = _(msgid)
+        translated = _(msgid)   #pylint: disable=E0602
 
         # Verify the first character is the test prefix
         if translated[0] != prefix:
@@ -98,7 +98,7 @@ def main():
             raise ValueError("Translated string \"%s\" minus the first & last character is not equal to msgid \"%s\"" % \
                                  (translated.encode('utf-8'), msgid))
 
-        print "Success: message string \"%s\" maps to translated string \"%s\"" % (msgid, _(msgid).encode('utf-8'))
+        print "Success: message string \"%s\" maps to translated string \"%s\"" % (msgid, _(msgid).encode('utf-8')) #pylint: disable=E0602
     except Exception, e:
         print >> sys.stderr, "ERROR: %s" % e
         return 1
diff --git a/install/tools/ipa-nis-manage b/install/tools/ipa-nis-manage
index 9cd6b5f..69db06a 100755
--- a/install/tools/ipa-nis-manage
+++ b/install/tools/ipa-nis-manage
@@ -20,6 +20,7 @@
 #
 
 import sys
+import os
 try:
     from optparse import OptionParser
     from ipapython import ipautil, config
-- 
1.7.4.2

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to