Re: [PATCH] gitweb: Support for snapshot

On 8/19/06, Junio C Hamano [off-list ref] wrote:

Luben Tuikov [off-list ref] writes:

quoted

--- "Aneesh Kumar K.V" <aneesh.kumar@gmail.com> wrote:

quoted

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