Made corrections based on Jyri's feedback.
Latest webrev is available at http://cr.opensolaris.org/~seema/6631480/

Thanks,
Seema.

Seema Alevoor wrote:
>> 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?
> 
> No need.
> This was taken from the existing "apache" man page !
> 
>> "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'.
>>
> 
> Fine. Will change it to use "isainfo -n" on a 64-bit platform.
> 
>> 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.
> 
> Fine.
> 
>> SEE ALSO section has a link to "http://htto://www.apache.org";, correct
>> that.
> 
> Will fix this.
> 


Reply via email to