From: Junio C Hamano <gitster@pobox.com>
Date: Wed, 01 Aug 2007 23:12:41 GMT
Subject: Re: [RFC/PATCH 2/2] gitweb: Add an option to show size of blobs in the tree view
Message-ID: <7v3az24zo6.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <200708011505.02078.jnareb@gmail.com>

Jakub Narebski <jnareb@gmail.com> writes:

> But I agree that this should be protected by the %feature mechanism.
> Two questions:
>
>  1. Should we make '-l' default when turned on? Or make 'showsizes'
>     %feature tristate: off, on, by default on?
>
>  2. If it is turned off, should we silently (or not so silently)
>     ignore this option, or return "Permission denied" or perhaps
>     "Invalid extra options parameter"?
>
> And how we should name this feature (key in %feature hash)?

I would say we would not do this by default, only with an
explicit override with gitweb-config.perl.  I am not sure what
the good name would be.  "expensive-ls-tree" perhaps?

> P.S. I have received no comments on
>  [RFC/PATCH] gitweb: Enable transparent compression for HTTP output
> (trade CPU load for lower bandwidth usage).

That's probably nobody was interested in it.