On 12/10/2010 04:15 PM, Jakub Narebski wrote: > Junio C Hamano wrote: >> "J.H." <warthog9@xxxxxxxxxxxxxx> writes: >> >>> 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html