On 03/04/09 15:11, Nick Kew wrote:
> Seema Alevoor wrote:
>> Hi,
>>
>> Kindly review the fix for CR 6729606.
>> Webrev is at http://cr.opensolaris.org/~seema/6729606/
>>
>> The fix involves removal of prefork specific settings from httpd.conf 
>> file.
>> mpm.conf in samples-conf.d already includes the default settings. 
> 
> You've also taken ServerName out.  That has nothing to do with prefork
> or any MPM-specific settings.

That was the redundant one.

> FWIW, I think you're right to do that.  Let admins who don't set it get
> a startup message alerting them to the fact.
> 

Actually, we are still setting the ServerName to 127.0.0.1 with a comment that
"Set ServerName if necessary (default is 127.0.0.1)"


Thanks,
Seema.

Reply via email to