On 25.03.2012 04:32, Jeff King wrote:
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
Yes! That patch did it, t5551-http-fetch.sh now passes fully. Thank you.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html