Code review comment for lp://staging/~tseaver/loggerhead/sphinxify
Revision history for this message
| Matt Nordhoff (mnordhoff) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
9199653
demo site
(Get the code!)
Some small comments on part of it (I'm watching TV now...):
> 317 +oggerhead is heavily based on `bazaar-webserve
Missing an L in "Loggerhead". ;-)
> 331 +- Paste for the server. (You need version 1.2 or newer of Paste.)
> 332 +
> 333 +- Paste Deploy (optional, needed when proxying through Apache)
> 334 +
> 335 +- flup (optional, needed to use FastCGI, SCGI or AJP)
The style is rather inconsistent here -- the spacing and "(foo)" vs. "(Foo.)".
Also, what should be done with the README now?