You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Thom May <th...@planetarytramp.net> on 2002/03/07 12:30:05 UTC
[Patch] Add errordir to Debian Layout.
This fixes PR10067. However, this is a fix for the symptom rather than the
cause, I think. Fundamentally, going into a system's htdocs dir and blowing
away anything called CVS is a big mistake. It doesn't make allowances for
all number of things, including the fact that the domains the server is
hosting could come straight out of cvs and access to CVS directories is
merely denied by rule, etc.
I'm going to send another patch that will need testing with non-GNU versions
of find that I think may implement a better way of doing this.
Cheers,
-Thom
Index: config.layout
===================================================================
RCS file: /home/cvspublic/httpd-2.0/config.layout,v
retrieving revision 1.9
diff -u -u -3 -r1.9 config.layout
--- config.layout 26 Feb 2002 18:41:58 -0000 1.9
+++ config.layout 7 Mar 2002 11:20:56 -0000
@@ -306,4 +306,5 @@
proxycachedir: ${prefix}/var/cache/apache2/proxy
infodir: ${exec_prefix}/share/info
installbuilddir: ${prefix}/etc/apache2/build
+ errordir: ${data_dir}/error
</Layout>
Re: [Patch] Add errordir to Debian Layout.
Posted by Aaron Bannert <aa...@clove.org>.
On Thu, Mar 07, 2002 at 11:30:05AM +0000, Thom May wrote:
>
> This fixes PR10067. However, this is a fix for the symptom rather than the
> cause, I think. Fundamentally, going into a system's htdocs dir and blowing
> away anything called CVS is a big mistake. It doesn't make allowances for
> all number of things, including the fact that the domains the server is
> hosting could come straight out of cvs and access to CVS directories is
> merely denied by rule, etc.
> I'm going to send another patch that will need testing with non-GNU versions
> of find that I think may implement a better way of doing this.
> Cheers,
> -Thom
Sorry Thom, I didn't see this post until just now. (I saw the bug
report first and just went to fix that. :)
I did see your later post and did note you in the CHANGES. Thanks
for the patch.
-aaron