a little nicer wrt pep8

[sfa.git] / config / sfa-config

diff --git a/config/sfa-config b/config/sfa-config
index ab59fe8..346356b 100755 (executable)
--- a/config/sfa-config
+++ b/config/sfa-config
@@ -1,20 +1,20 @@
-#!/usr/bin/env python2
+#!/usr/bin/env python3
 #
 # Script for basic access to the SFA configuration file store.
 #
 
 import sys
 import os
-import fcntl
+#import fcntl
 import getopt
-import signal
+#import signal
 from sfa.util.config import Config
 
 
 def usage():
-    print """
+    print("""
 Script to access the SFA configuration file store.
-    
+
 Usage: %s [OPTION]... [FILES]
         Conversion:
 
@@ -44,13 +44,13 @@ Usage: %s [OPTION]... [FILES]
 
         -h, --help      This message
         -s, --save      Save changes to first configuration file
-""".lstrip() % sys.argv[0]
+""".lstrip() % sys.argv[0])
     sys.exit(1)
 
 
 def deprecated(message):
-    print "%s: deprecated usage" % sys.argv[0]
-    print message
+    print("%s: deprecated usage" % sys.argv[0])
+    print(message)
     sys.exit(1)
 
 
@@ -80,7 +80,7 @@ def main():
 
     try:
         (opts, argv) = getopt.gnu_getopt(sys.argv[1:], shortopts, longopts)
-    except Exception, err:
+    except Exception as err:
         sys.stderr.write("Error: " + str(err) + os.linesep)
         sys.exit(1)
 
@@ -119,7 +119,7 @@ def main():
         elif opt == '-s' or opt == "--save":
             if not optval:
                 usage()
-            print 'parsed save option', optval
+            print('parsed save option', optval)
             save = optval
         elif opt == '-h' or opt == "--help":
             usage()
@@ -134,17 +134,17 @@ def main():
             config.load(file)
         except IOError:
             pass
-        except Exception, err:
+        except Exception as err:
             sys.stderr.write("Error: %s: %s" % (file, str(err)) + os.linesep)
             sys.exit(1)
 
     # --category, --variable, --value
-    if category.has_key('id') and variable.has_key('id'):
-        if variable.has_key('value'):
+    if 'id' in category and 'id' in variable:
+        if 'value' in variable:
             config.set(category['id'], variable['id'], variable['value'])
         else:
             value = config.get(category['id'], variable['id'])
-            print value
+            print(value)
 
     # --shell, --php, --xml
     if output is not None:
@@ -156,9 +156,9 @@ def main():
         # so that plc.d/{api,postgres} can create configs/site.xml
         dirname = os.path.dirname(save)
         if (not os.path.exists(dirname)):
-            os.makedirs(dirname, 0755)
+            os.makedirs(dirname, 0o755)
             if (not os.path.exists(dirname)):
-                print "Cannot create dir %s - exiting" % dirname
+                print("Cannot create dir %s - exiting" % dirname)
                 sys.exit(1)
         config.save(save)