dulwich-users team mailing list archive

[David Blewett <david@xxxxxxxxxxxxxxxx>]

On Mon, Feb 28, 2011 at 6:38 PM, Dave Borowitz <dborowitz@xxxxxxxxxx> 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