durusmail: quixote-users: Patch to mod_python_handler.py needs testing
Patch to mod_python_handler.py needs testing
2001-11-13
2001-11-13
2001-11-13
2001-11-13
Patch to mod_python_handler.py needs testing
Greg Ward
2001-11-13
Hi Erno (or anyone else using Quixote under mod_python) --

last week, Neil made this patch to Quixote's mod_python_handler.py:

--- mod_python_handler.py       2001/09/28 21:03:22     1.5
+++ mod_python_handler.py       2001/11/08 18:17:37     1.6
@@ -6,3 +6,3 @@

-__revision__ = "$Id: mod_python_handler.py,v 1.5 2001/09/28 21:03:22 gward Exp
$"
+__revision__ = "$Id: mod_python_handler.py,v 1.6 2001/11/08 18:17:37 nascheme
Exp $"

@@ -20,10 +20,6 @@
         """
-        real_env = os.environ
-        try:
-            os.environ = apache.build_cgi_env(req)
-            self.publish(apache.CGIStdin(req),
-                         apache.CGIStdout(req),
-                         sys.stderr)
-        finally:
-            os.environ = real_env
+        self.publish(apache.CGIStdin(req),
+                     apache.CGIStdout(req),
+                     sys.stderr,
+                     apache.build_cgi_env(req))
         return apache.OK

This looks good to me, and I'm sure Neil thought it was fine.  But we
haven't tested it yet, because none of us have setup mod_python yet.
Can you let us know if a mod_python-based Quixote still works after this
patch?  (This patch is going into Quixote 0.4.2, which I hope to release
today, so I'd like to know if it works!)

        Greg
--
Greg Ward - software developer                gward@mems-exchange.org
MEMS Exchange                            http://www.mems-exchange.org


reply