On Today at 4:12pm, SB=>Stas Bekman <[EMAIL PROTECTED]> wrote:

SB> 
SB> Sure, please find all the files that miss the end lines and post a
SB> patch fixing them, which will gladly apply and try to remember to
SB> keeping them in place in the future.

Really its no big deal. Just one file which is fixed in the attached
patch.

SB> 
SB> [...]
SB> 
SB> + # keep everything self-contained, to avoid problems with sandboxes
SB> + SetEnv TMPDIR @t_logs@

Seems logical. Attached patch follows this theme.

SB> 
SB> [...]
SB> 
SB> Does it solve the problem with failing tests? If so please post a
SB> patch that works for you.

Yes, the patch fixes the failing tests/sandbox violations. Find attached.

SB> 
SB> Another alternative is to introduce a new directory for Apache-Test:
SB> t/tmp and a corresponding -tmp option to override the default, which
SB> is probably a cleaner solution in the long run.

All good ideas for the long run. Not familiar with Apache-Test, so can't 
comment.

SB> 
SB> 2) > [Wed Oct 01 18:15:18 2003] [error] (13)Permission denied:
SB>    Couldn't unlink
SB>  > unix domain socket /var/run/cgisock
SB> 
SB> I'm trying to understand why do we have this problem when we don't use
SB> mod_cgid. Is it because mod_cgid gets loaded?

Good question. Here's what I believe is happening. /usr/sbin/apxs2 is the 
path to my APache eXtenSion tool. During the conf generation process apxs2 
is used to figure out the location of the central config file for apache2 
(/etc/apache2/conf/apache2.conf). This file does have a:
        LoadModule cgid_module                   modules/mod_cgid.so
line. References to cgid are then found abundantly in 
t/conf/apache_test_config.pm (another generated file). Finally, the 
following section appears in the generated file t/conf/httpd.conf:

<IfModule !mod_cgid.c>
    LoadModule cgid_module "/etc/apache2/modules/mod_cgid.so"
</IfModule>

Should (and more importantly how do) we attempt to keep cgid out of the 
conf files?

SB> 
SB> [...]
SB> 
SB> <IfModule mod_cgid.c>
SB>      ScriptSock logs/cgisock
SB> <IfModule>

Attached patch uses the above lines as well.
--
Haroon Rafique
<[EMAIL PROTECTED]>
Index: ModPerl-Registry/t/conf/extra.conf.in
===================================================================
RCS file: /home/cvspublic/modperl-2.0/ModPerl-Registry/t/conf/extra.conf.in,v
retrieving revision 1.10
diff -u -r1.10 extra.conf.in
--- ModPerl-Registry/t/conf/extra.conf.in       5 Aug 2003 18:51:10 -0000       1.10
+++ ModPerl-Registry/t/conf/extra.conf.in       2 Oct 2003 02:45:11 -0000
@@ -145,4 +145,9 @@
     <IfModule mod_deflate.c>
         SetOutputFilter DEFLATE
     </IfModule>
-</Location>
\ No newline at end of file
+</Location>
+# keep everything self-contained, to avoid problems with sandboxes
+<IfModule mod_cgid.c>
+    ScriptSock logs/cgisock
+</IfModule>
+SetEnv TMPDIR @t_logs@
Index: t/conf/extra.conf.in
===================================================================
RCS file: /home/cvspublic/modperl-2.0/t/conf/extra.conf.in,v
retrieving revision 1.6
diff -u -r1.6 extra.conf.in
--- t/conf/extra.conf.in        3 Feb 2003 22:56:19 -0000       1.6
+++ t/conf/extra.conf.in        2 Oct 2003 02:45:12 -0000
@@ -23,3 +23,8 @@
         PerlModule Doesnt::Exist
     </IfDefine>
 </IfModule>
+# keep everything self-contained, to avoid problems with sandboxes
+<IfModule mod_cgid.c>
+    ScriptSock logs/cgisock
+</IfModule>
+SetEnv TMPDIR @t_logs@

Reply via email to