Maybe we should have another directive for the suexec log? But I agree it shouldn't be hardcoded.
david ----- Original Message ----- From: "Thom May" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Tuesday, December 31, 2002 11:39 AM Subject: [PATCH] remove hardcoding of suexec log location > This is to fix PR 15713 > "suexec-logfile has hardcoded default instead of being > placed in --logfiledir" > > I'm not sure it's right, though, since I'm not sure of the status of suexec > on win32/other places where ap_config_layout.h isn't available. > -Thom > > > Index: support/suexec.h > =================================================================== > RCS file: /home/cvspublic/httpd-2.0/support/suexec.h,v > retrieving revision 1.7 > diff -u -u -r1.7 suexec.h > --- support/suexec.h 27 Sep 2002 23:53:04 -0000 1.7 > +++ support/suexec.h 30 Dec 2002 12:10:17 -0000 > @@ -62,6 +62,12 @@ > #define _SUEXEC_H > > /* > + * Include ap_config_layout so we can work out where the default htdocsdir > + * and logsdir are. > + */ > +#include "ap_config_layout.h" > + > +/* > * HTTPD_USER -- Define as the username under which Apache normally > * runs. This is the only user allowed to execute > * this program. > @@ -117,7 +123,7 @@ > * debugging purposes. > */ > #ifndef AP_LOG_EXEC > -#define AP_LOG_EXEC "/usr/local/apache2/logs/cgi.log" /* Need me? */ > +#define AP_LOG_EXEC DEFAULT_EXP_LOGFILEDIR "/suexec_log" /* Need me? */ > #endif > > /* > @@ -126,7 +132,7 @@ > * that can be used for suEXEC behavior. > */ > #ifndef AP_DOC_ROOT > -#define AP_DOC_ROOT "/usr/local/apache2/htdocs" > +#define AP_DOC_ROOT DEFAULT_EXP_HTDOCSDIR > #endif > > /* >