Re: [PATCH 11/18] gitweb: add isDumbClient() check

On 12/10/2010 04:15 PM, Jakub Narebski wrote:

Junio C Hamano wrote:

quoted

"J.H." [off-list ref] writes:

quoted

My initial look indicated that perl-http-browserdetect wasn't available
for RHEL / CentOS 5 - it is however available in EPEL.

However there are a couple of things to note about User Agents at all:
	- They lie... a lot
	- Robots lie even more

Blacklisting is still the better option, by a lot.  I'll re-work this
some in v9, as I'm fine with the added dependency.

Thanks, both.  I sense that we finally are going to get a single version
of gitweb that can be used at larger sites ;-)

I wouldn't be so optimistic.  While we borrow features and ideas from
each other, the difference still remains that J.H. patches are bit hacky
but are tested, while my rewrite is IMHO cleaner but untested (well, 
untested on real life load).

At this point I'm not sure there is a way to rectify the two patch
series, and while we may borrow ideas from each other it's becoming
clear that we are both, generally speaking, heading in different
directions for what we want and need out of gitweb.  Jakub's patches for
the admin page are indicative of that.

Anyway the main issue that was discovered by PATCHv6 by me, and v8 by J.H.
is that die_error sucks... well, at least if background caching is enabled.

I'd agree with that, and as such I'm working on a complete re-work of
error handling in gitweb for v9.  Things are looking pretty good so far,
but to claim that it's a non-invasive patch would be akin to selling
someone the Brooklyn bridge.

That said, the way Gitweb handles it's errors and things like exit are
appalling and this has been something that's needed doing for a while
anyway.  Guess now's the time to do it.  Might be a few days for me to
get far enough for any of it to be worthwhile sharing, late next week
maybe.  That said I hit vacation starting on the 20th so it might be
next year before that is finalized.

- John 'Warthog9' Hawley