← Back to team overview

dulwich-users team mailing list archive

Re: [PATCH] Unified gzip/paster patch

 

Hi David,

Welcome back.

On 09/22/2011 02:56 AM, David Blewett wrote:
On Sun, Apr 24, 2011 at 3:41 PM, Jelmer Vernooij<jelmer@xxxxxxxxx>  wrote:
Unfortunately, as discussed on IRC, this still fails.

It'd be nice if we can land this for 0.7.2. I'm also going to have a
look at HTTP smart server support for 0.7.2.
This patch seems to have stalled; are you still planning to work on it?
If not, I might have a look to see if I can fix this test.. it'd be a
shame to have to discard it now.
Sorry for the long delay. Got tied up with work projects. Attached is
a rebased patch to current HEAD. The problem seems to have been that
the egg-info directory wasn't getting created, so the entry points
weren't being found. An updated patch adds a step to the Makefile to
call the egg_info action during the build step.

I also pushed these changes to my github fork.
With this patch the testsuite seems to output quite a bit more data than previously. I'd really rather reduce the amount of data it writes to standard out instead of increasing it.

My familiarity with the web server is limited, so I'm hoping other people can also have a look. In particular, if Dave has a chance to review, that'd be great.

Cheers,

Jelmer


Follow ups

References