Jyri Virkki wrote:
> Seema Alevoor wrote:
>> Latest webrev is available at http://cr.opensolaris.org/~seema/6631480/
> 
> Why does "apache2.1m.pdf" show up on webrev list? It's not part of the
> putback right? Just a webrev hiccup?

Just a readable version of apache2.1m.sunman file.
I did add a comment saying "it will not be integrated".

> How come usr/src/cmd/apache2/install-apache2
> removes installing the favicons? That doesn't seem related to adding the
> manpage?
> 

No, it doesn't relate to man page installation.
favicon file installation is handled by another function "install_docroot_files"
(bugfix for 6631493 changed the function name from "install_favicon_files" to 
"install_docroot_files").

The one that I have removed was under "install_man_files" function and was 
redundant.


Thanks,
Seema.

Reply via email to