why does python not believe in variable scoping, but then complain about it?

[mirror/userdir-ldap.git] / ud-generate

diff --git a/ud-generate b/ud-generate
index f00cf6e..d3da309 100755 (executable)
--- a/ud-generate
+++ b/ud-generate
@@ -1242,17 +1242,18 @@ for x in attrs:
    if x[1].has_key("subGroup") != 0:
       SubGroupMap.setdefault(x[1]["gid"][0], []).extend(x[1]["subGroup"])
 
+lock = None
 try:
-   lockfile = os.path.join(GenerateDir, 'ud-generate.lock')
-   lock = get_lock( lockfile )
+   lockf = os.path.join(GenerateDir, 'ud-generate.lock')
+   lock = get_lock( lockf )
    if lock is None:
-      sys.stderr.write("Could not acquire lock %s.\n"%(lockfile))
+      sys.stderr.write("Could not acquire lock %s.\n"%(lockf))
       sys.exit(1)
 
    generate_all(GenerateDir, l)
 
 finally:
-   if not lock is None:
+   if lock is not None:
       lock.release()
 
 # vim:set et: