On Mon, Feb 28, 2011 at 6:38 PM, Dave Borowitz <[email protected]> wrote: > I've got no objections to your approach, but my feeling is that web.py is > not the right place for this. We provide a reference implementation in that > module because wsgiref is the reference (WSGI) web server implementation, > but I don't think we want to go adding factory functions to this module for > every web server out there.
I completely understand; I put it there so as to be as small a change as possible. Maybe a separate module named serving.py or even as specific as paster.py ? > If we want to include paster support in Dulwich proper (which is Jelmer's > decision to make, not mine), I would propose putting it in a separate > module/package. Understood. It isn't a show-stopper that this entry-point doesn't live in Dulwich, but it does mean that everyone that wanted to combine the two would have to roll their own factory function similar to what is in the patch. It seems friendlier to have it included with Dulwich. Thanks, David Blewett _______________________________________________ Mailing list: https://launchpad.net/~dulwich-users Post to : [email protected] Unsubscribe : https://launchpad.net/~dulwich-users More help : https://help.launchpad.net/ListHelp
