{"thread":{"id":"65607","subject":"[PATCH] t5551: \"GIT_TEST_LONG=Yes make test\" is broken","startedAt":"2026-05-08T05:31:06Z","lastAt":"2026-05-08T14:04:03Z","messageCount":2,"participants":["Junio C Hamano","Jeff King"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"542886","messageId":"xmqqqznmeaco.fsf@gitster.g","threadId":"65607","inReplyTo":null,"subject":"[PATCH] t5551: \"GIT_TEST_LONG=Yes make test\" is broken","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2026-05-08T05:31:03Z","receivedAt":"2026-05-08T05:31:06Z","isPatch":true,"body":"The \"test_expect_success 'tag following always works over v0 http'\"\ntest in t5551 fails when it tries to run \"git init tags\", but this\nhappens only when EXPENSIVE test is allowed to run.  \n\nThis is because the step tries to create a repository with \"git init\ntags\" but the EXPENSIVE test that runs way before it creates and\nleaves around a temporary file \"tags\".  Have the EXPENSIVE test\nclean it up after itself.\n\nSigned-off-by: Junio C Hamano <gitster@pobox.com>\n---\n\n * As we should give individual contributors a stable base, I have\n   identified tests that fail with EXPENSIVE enabled in \"master\".\n   With this fixed, we should add GIT_TEST_LONG=YesPlease to the CI\n   jobs run upon PRs are created by contributors.\n\n t/t5551-http-fetch-smart.sh | 1 +\n 1 file changed, 1 insertion(+)\n\ndiff --git c/t/t5551-http-fetch-smart.sh w/t/t5551-http-fetch-smart.sh\nindex a26b6c2844..e236e526f0 100755\n--- c/t/t5551-http-fetch-smart.sh\n+++ w/t/t5551-http-fetch-smart.sh\n@@ -481,6 +481,7 @@ test_expect_success 'test allowanysha1inwant with unreachable' '\n '\n \n test_expect_success EXPENSIVE 'http can handle enormous ref negotiation' '\n+\ttest_when_finished \"rm -f tags\" &&\n \t(\n \t\tcd \"$HTTPD_DOCUMENT_ROOT_PATH/repo.git\" &&\n \t\tcreate_tags 2001 50000\n"},{"id":"542920","messageId":"20260508140355.GA709299@coredump.intra.peff.net","threadId":"65607","inReplyTo":"xmqqqznmeaco.fsf@gitster.g","subject":"Re: [PATCH] t5551: \"GIT_TEST_LONG=Yes make test\" is broken","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2026-05-08T14:03:55Z","receivedAt":"2026-05-08T14:04:03Z","isPatch":true,"body":"On Fri, May 08, 2026 at 02:31:03PM +0900, Junio C Hamano wrote:\n\n> The \"test_expect_success 'tag following always works over v0 http'\"\n> test in t5551 fails when it tries to run \"git init tags\", but this\n> happens only when EXPENSIVE test is allowed to run.  \n> \n> This is because the step tries to create a repository with \"git init\n> tags\" but the EXPENSIVE test that runs way before it creates and\n> leaves around a temporary file \"tags\".  Have the EXPENSIVE test\n> clean it up after itself.\n\nI wondered how and when this broke, but it looks like it has been since\n2024 when the later v0 test was added. And it looks like I am the\nculprit for both the EXPENSIVE test adding the \"tags\" file and the v0\ntest assuming the path is available. :)\n\nAnyway, this seems like the obviously correct solution.\n\n-Peff\n"}]}