From: Jeff King <peff@peff.net>
Date: Sun, 25 Mar 2012 02:32:15 GMT
Subject: Re: Clone fails on a repo with too many heads/tags
Message-ID: <20120325023215.GA13958@sigill.intra.peff.net>
In-Reply-To: <20120325010609.GB27651@sigill.intra.peff.net>

On Sat, Mar 24, 2012 at 09:06:09PM -0400, Jeff King wrote:

> That's weird. 405 is "Method Not Allowed". Clone shouldn't be doing
> anything more exotic than GET and POST. And the prior tests pass, so it
> means that it's working in general. The only thing different about this
> test is that apache is configured to use mod_rewrite to issue a
> redirect.
> 
> Does your machine have mod_rewrite installed and enabled? I would think
> apache would complain at startup if it wasn't.  I wonder if there's
> something non-portable in the minimal apache config we ship.
> 
> Does httpd/error.log in the trash directory say anything interesting?

Also, does it work any better with this patch?

diff --git a/t/lib-httpd/apache.conf b/t/lib-httpd/apache.conf
index 3c12b05..714760d 100644
--- a/t/lib-httpd/apache.conf
+++ b/t/lib-httpd/apache.conf
@@ -61,9 +61,8 @@ ScriptAlias /smart_noexport/ ${GIT_EXEC_PATH}/git-http-backend/
 	Options ExecCGI
 </Files>
 
-RewriteEngine on
-RewriteRule ^/smart-redir-perm/(.*)$ /smart/$1 [R=301]
-RewriteRule ^/smart-redir-temp/(.*)$ /smart/$1 [R=302]
+RedirectMatch 301 ^/smart-redir-perm/(.*)$ /smart/$1
+RedirectMatch 302 ^/smart-redir-temp/(.*)$ /smart/$1
 
 <IfDefine SSL>
 LoadModule ssl_module modules/mod_ssl.so