On 8/19/06, Junio C Hamano wrote: > Luben Tuikov writes: > > > --- "Aneesh Kumar K.V" wrote: > >> This adds snapshort support in gitweb. To enable one need to > >> set gitweb.snapshot = true in the config file. > > > > Could you use bzip2? It generates smaller files (better compression), > > which is a good thing when downloading over a network. > > Because bzip2 is heavier on the server than gzip is (and gzip is > heavier than "gzip -1" is), there obviously is a trade-off. We > would want it to be configurable just like blame and snapshot > itself. > > Maybe: > > config.snapshot = no | yes | gzip | bzip2 ... > > By the way, I think it is a mistake to use only $GIT_DIR/config > to control these features. > I have coded this at What should be the content-encoding in this case x-$snapshot ? This is the untested diff that i have. Is this what we are looking for ? -aneesh