<div dir="ltr"><br><div class="gmail_quote">2008/9/14 Andrew Bennetts <span dir="ltr"><<a href="mailto:andrew@canonical.com">andrew@canonical.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">Marius Kruger wrote:<br>
</div><br><div class="Ih2E3d">
> do you think we should get `wsgi.make_app()` to do this automatically?<br>
><br>
> then we can add additional parameters:<br>
> smart_server_app = wsgi.make_app(<br>
> root='/var/lib/gforge/bzrroot',<br>
> prefix='/bzr/',<br>
> path_var='REQUEST_URI',<br>
> readonly=False,<br>
> load_plugins=True,<br>
> logfile='/var/log/bazaar.fcgi.log',<br>
> )<br>
<br>
</div>I think that's a good idea.<br>
<div class="Ih2E3d"><br>
> note I also want to get the logging working (properly set, or properly<br>
> disabled), because I get a lot of `No handlers could be found for logger<br>
> "bzr"` in my apache log.<br>
><br>
> I'll probably hack a patch together for this soon, if it sounds ok to you.<br>
<br>
</div>It does, thank you!<font color="#888888"><br>
-Andrew.<br>
<br>
</font></blockquote></div><br>Here is a quick patch which adds `load_plugins=True, enable_logging=True` parameters<br>to wsgi.make_app ()<br>Which allows bzr+http:// users to easily get plugins loading and logs written!<br>
<br>* I didn't give the option to name the log file, because I didn't have the time and energy now.<br> So if people think its a must-have, I can do that a little later.<br>* I could not find a good place to add tests, and saw that tests for this might be quite tricky.<br>
* I would add news if people are happy with what I did so far.<br><br clear="all"><br>-- <br>regards<br>Marius<br><br><><<br><a href="http://amanica.blogspot.com/">http://amanica.blogspot.com/</a><br><br><br>
</div>