Looks good to me too John - assuming you've added the comment ;) Thanks,
Darren. On 16/07/2011 02:38, John Fischer wrote: > Dave, > > Thanks!! I'll add a comment. > > John > > On Jul 15, 2011, at 5:54 PM, Dave Miner wrote: > >> John, change is fine, though I would add a comment to note that this is >> intentionally commented out. >> >> Dave >> >> On 07/15/11 05:13 PM, John Fischer wrote: >>> All, >>> >>> Can I get a review for Defect 1271: >>> >>> https://cr.opensolaris.org/action/browse/caiman/johnfisc/12717/ >>> Bug 12717 -look into ai-httpd.conf not auto-including other files >>> >>> The webrev is located at: >>> >>> https://cr.opensolaris.org/action/browse/caiman/johnfisc/12717/ >>> >>> By Including all the configuration files within the /etc/apache/2.2/conf.d/ >>> directory we take the chance of an admin adding a new configuration >>> file into the directory for another instance of an apache webserver >>> with unknown effects to the AI webserver. Since there are no >>> configuration files within the directory at present the solution is to >>> remove (or actually comment out) the wildcard inclusion. >>> >>> Thanks, >>> >>> John >>> _______________________________________________ >>> caiman-discuss mailing list >>> [email protected] >>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >> > > _______________________________________________ > caiman-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss _______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
