Merge lp:~statik/tarmac/web-status-cleanup into lp:tarmac

[Elliot Murphy]

Minor change to the web ui so that Config isn't loaded at each request.

[Elliot Murphy (statik)]

Now that I understand how the reloader works, don't reload the config on every web request.

[Paul Hummer (rockstar)]

review: Approve

=== modified file 'webui.py'
--- webui.py	2009-07-10 19:42:24 +0000
+++ webui.py	2009-07-11 02:33:03 +0000
@@ -31,14 +31,13 @@
 class index(object):
     """The main page of the status site."""
 
+    def __init__(self):
+        from tarmac.config import TarmacConfig
+        config = TarmacConfig(sys.argv[2])
+        self.statusfile = config.log_file
+
     def GET(self):
-        # XXX I cannot figure out how to make the config object from main
-        # available here.
-        # statusfile = config.log_file
-        from tarmac.config import TarmacConfig
-        statusfile = TarmacConfig(sys.argv[2]).log_file
-        print statusfile
-        tail = subprocess.Popen(('tail', '-n40', statusfile),
+        tail = subprocess.Popen(('tail', '-n40', self.statusfile),
             stdout = subprocess.PIPE,
             stderr = subprocess.PIPE)
         output, errput = tail.communicate()
@@ -49,8 +48,6 @@
     if len(sys.argv) != 3:
         print "Please specify a port number and a project name: ./webui.py 8080 default"
         return 1
-    projectname = sys.argv[2]
-    config = TarmacConfig(projectname)
     app = web.application(urls, globals())
     return app.run()