We set GIT_CONFIG_NOSYSTEM in our test scripts so that we do not accidentally read /etc/gitconfig and have it influence the outcome of the tests. But when running smart-http tests, Apache will clean the environment, including this variable, and the "server" side of our http operations will read it.
You can see this breakage by doing something like:
make ./git config --system http.getanyfile false make test
which will cause t5561 to fail when it tests the fallback-to-dumb operation.
We can fix this by instructing Apache to pass through the variable. Unlike with other variables (e.g., 89c57ab3's GIT_TRACE), we don't need to set a dummy value to prevent warnings from Apache. test-lib.sh already makes sure that GIT_CONFIG_NOSYSTEM is set and exported.
Signed-off-by: Jeff King <peff@peff.net> --- Obviously trivial enough for "maint", but not at all urgent; it has been this way for ages. I only noticed because I happened to recently munge my system config to replicate a particular server config which disables http.getanyfile.
t/lib-httpd/apache.conf | 1 + 1 file changed, 1 insertion(+)
Show changes to t/lib-httpd/apache.conf +1 −0
diff --git a/t/lib-httpd/apache.conf b/t/lib-httpd/apache.conf index f667e7c..9317ba0 100644 --- a/t/lib-httpd/apache.conf +++ b/t/lib-httpd/apache.conf @@ -74,6 +74,7 @@ PassEnv GIT_VALGRIND_OPTIONS PassEnv GNUPGHOME PassEnv ASAN_OPTIONS PassEnv GIT_TRACE +PassEnv GIT_CONFIG_NOSYSTEM Alias /dumb/ www/ Alias /auth/dumb/ www/auth/dumb/
-- 2.8.0.rc2.331.ga574393