Changeset 1197 for trunk/packages/sipb-xen-base/files/usr/share/python-support/sipb-xen-base/invirt/config.py

Timestamp:

Oct 24, 2008, 3:35:21 AM (16 years ago)

Author:

broder

Message:

Now that we're using Python 2.5, we can actually write with statements

File:

• trunk/packages/sipb-xen-base/files/usr/share/python-support/sipb-xen-base/invirt/config.py (modified) (3 diffs)

trunk/packages/sipb-xen-base/files/usr/share/python-support/sipb-xen-base/invirt/config.py

@@ +1 @@
+from __future__ import with_statement
+
 import json
 from invirt.common import *
 from os import rename
 from os.path import getmtime
+from contextlib import closing
 
 default_src_path   = '/etc/invirt/master.yaml'
 default_cache_path = '/var/lib/invirt/cache.json'
-lock_file          = '/var/lib/invirt/cache.lock'
+lock_path          = '/var/lib/invirt/cache.lock'
 
 def load(src_path = default_src_path,
@@ -54 +57 @@
         # lock with other concurrent reads).  This isolation is accomplished
         # using an atomic filesystem rename in the refreshing stage.
-        try: ns.cfg = with_closing(file(cache_path)) (
-                lambda f: json.read(f.read()))
+        try: 
+            with closing(file(cache_path)) as f:
+                ns.cfg = json.read(f.read())
         except: do_refresh = True
 
@@ -69 +73 @@
         except: loader = yaml.SafeLoader
         try:
-            @with_lock_file(lock_file)
-            def refresh_cache():
-                ns.cfg = with_closing(file(src_path)) (
-                        lambda f: yaml.load(f, loader))
-                try: with_closing(file(cache_path + '.tmp', 'w')) (
-                        lambda f: f.write(json.write(ns.cfg)))
+            with lock_file(lock_path):
+                with closing(file(src_path)) as f:
+                    ns.cfg = yaml.load(f, loader)
+                try: 
+                    with closing(file(cache_path + '.tmp', 'w')) as f:
+                        f.write(json.write(ns.cfg))
                 except: pass # silent failure
                 else: rename(cache_path + '.tmp', cache_path)
         except IOError:
-            ns.cfg = with_closing(file(src_path)) (
-                    lambda f: yaml.load(f, loader))
+            with closing(file(src_path)) as f:
+                ns.cfg = yaml.load(f, loader)
     return ns.cfg