httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <>
Subject patch comments Re: [PATCH] Add mod_gz to httpd-2.0
Date Wed, 05 Sep 2001 02:15:03 GMT
On Sat, Sep 01, 2001 at 02:56:45PM -0700, Justin Erenkrantz wrote:
> I told him I'd look at it a while ago, but never got a chance to do 
> so.  So, I spent this morning cleaning up the configuration and a bit 
> of the code to fit our style (nothing major).

You shouldn't have to do that. Ian should, and he must learn to deal with it
if he wants to continue contributing. We all learn different habits to work
in this group.

> I'd like to add this to the modules/filters directory (which seems
> like the most appropriate place).


> We could remove GZFilter as it really serves no purpose as well as the 
> text/html check in mod_gz.


I agree that it doesn't make sense to be in there. We should be using more
consistent mechanism to handle filter inclusion and management. One-off
commands here and there are simply serving to make the filter system more

> I'd like to commit something that is close
> to what Ian originally submitted and then tweak it slightly.

Push it back to him to tweak. It appears we have a bit of time :-)


Greg Stein,

View raw message