Seema Alevoor wrote: > > Hi, > > Webrev is at http://cr.opensolaris.org/~seema/6631480/ > Please review.
usr/src/cmd/apache2/Solaris/apache2.1m.sunman "The Apache 2 is integrated with Solaris." That seems redundant.. if I'm reading the manpage is because it's installed ;-) So do we need to point it out? "Depending on your machine architecture, [isainfo] should be expanded to either sparcv9 or amd64." Wording it that way forces you to change the manpage every time there is a different architecture. Better to just say it represents the output of 'isainfo -n'. In the section listing directories, I suggesting adding /etc/apache2/[version]/original/ with explanation that its contents are delivered as-is from the apache distribution and these files are not meant to be read by the server, they are there as examples for the user. SEE ALSO section has a link to "http://htto://www.apache.org";, correct that. -- Jyri J. Virkki - jyri.virkki at sun.com - Sun Microsystems
