Hi, On Mon, 26 Feb 2007, Ed Schouten wrote: > I just wrote a really awful patch to add syntax highlighting to gitweb: > > [...] > > + $pid = open2(\*HLR, \*HLW, "highlight -f --syntax c"); Since this fork()s an external program, which is possibly expensive, or can fail because the program is not there, you should at least - document it, and - make it optional. Ciao, Dscho - 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