[PATCH] ui-diff: Add configuration option to limit number of files

Tim Nordell tim.nordell at logicpd.com
Mon Feb 29 21:48:15 CET 2016


On 02/28/16 06:38, John Keeping wrote:
> I wonder if people will miss the fact that the diff stat has been
> truncated.  Would it be better to just drop the entire thing if there
> are too many changed files and replace it with something like:
>
> 	This diff is really big, are you sure you want to see it?
>
> and provide a link that bypasses this limit?
>

Good idea.  What if the diffstat is truncated, and rather than showing 
any diffs, it shows the message above?  It's still sometimes useful to 
see a snippet of what changed even if it's huge.  I suppose maybe the 
"forced" view could be a link to downloading the patch locally?

There are circumstances in which I really don't want an HTML view of the 
full changeset to ever be rendered to someone remotely.  An extreme 
example of this is when the kernel tree was introduced (commit tagged 
v2.6.12-rc2 in any CGIT hosted kernel repo).  I won't link it here, but 
it takes well over 100 seconds to generate on kernel.org's server, and 
content-wise is well over 300MiB since it contains essentially the 
entire source code of kernel v2.6.12-rc2 in HTML form since it was the 
introduction commit to the kernel.

- Tim


More information about the CGit mailing list