On Thu, Nov 16, 2006 at 07:45:08AM -0500, Jeff Trawick wrote: > On 10/16/06, Eric Covener <[EMAIL PROTECTED]> wrote: > >Patch below logs the path to the config file just before ap_mpm_run() > > > >Can help clear up some mysteries when posthumously analyzing an ErrorLog > > as a further aid, is it practical to add a list of defines used to > parse the conf file? > > [notice] Using config file conf/foo.conf with -DHIGHPERF, -DPORTAL_REWRITES > [notice] Using config file conf/foo.conf
I don't think it's really appropriate to log anything like this at NOTICE level. You're working down the slippery slope with argv[1..]; also on that slope are argv[0] which can be pretty critical, and then why not all of _environ too? For N% of users this stuff is *not* going to vary per invocation, and I'd bet N is pretty high. So logging it by default seems wrong. Regards, joe
