{"thread":{"id":"36848","subject":"[PATCH] t5551: fix the 50,000 tag test","startedAt":"2014-06-05T21:36:21Z","lastAt":"2014-06-10T05:53:31Z","messageCount":4,"participants":["Torsten Bögershausen","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"243420","messageId":"5390E2D5.9070000@web.de","threadId":"36848","inReplyTo":null,"subject":"[PATCH] t5551: fix the 50,000 tag test","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2014-06-05T21:36:21Z","receivedAt":"2014-06-05T21:36:21Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"The first version of test 23 did simply check that no output was send\nto stderr.\n\nCommit 5e2c7cd2 verified that the expected tags were actually cloned.\n\nSince the day \"git clone\" printed \"Cloning into 'too-many-refs'\" to stderr,\nthe test failed because stderr was not empty.\n\nRemove the check for stderr and make t5551-23 pass again\n\nSigned-off-by: Torsten Bögershausen <tboegi@web.de>\n---\n t/t5551-http-fetch-smart.sh | 3 +--\n 1 file changed, 1 insertion(+), 2 deletions(-)\n\ndiff --git a/t/t5551-http-fetch-smart.sh b/t/t5551-http-fetch-smart.sh\nindex e07eaf3..2c49133 100755\n--- a/t/t5551-http-fetch-smart.sh\n+++ b/t/t5551-http-fetch-smart.sh\n@@ -240,8 +240,7 @@ test_expect_success EXPENSIVE 'create 50,000 tags in the repo' '\n '\n \n test_expect_success EXPENSIVE 'clone the 50,000 tag repo to check OS command line overflow' '\n-\tgit clone $HTTPD_URL/smart/repo.git too-many-refs 2>err &&\n-\ttest_line_count = 0 err &&\n+\tgit clone $HTTPD_URL/smart/repo.git too-many-refs &&\n \t(\n \t\tcd too-many-refs &&\n \t\ttest $(git for-each-ref refs/tags | wc -l) = 50000\n-- \n2.0.0.553.ged01b91\n"},{"id":"243646","messageId":"xmqqr42xevwi.fsf@gitster.dls.corp.google.com","threadId":"36848","inReplyTo":"5390E2D5.9070000@web.de","subject":"Re: [PATCH] t5551: fix the 50,000 tag test","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-06-09T19:16:13Z","receivedAt":"2014-06-09T19:16:13Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Torsten Bögershausen <tboegi@web.de> writes:\n\n> The first version of test 23 did simply check that no output was send\n> to stderr.\n>\n> Commit 5e2c7cd2 verified that the expected tags were actually cloned.\n>\n> Since the day \"git clone\" printed \"Cloning into 'too-many-refs'\" to stderr,\n\nThanks.  It is 68b939b2 (clone: send diagnostic messages to stderr,\n2013-09-18); before it we showed the message to the standard output\nstream instead.\n\nWill queue.\n\n> the test failed because stderr was not empty.\n>\n> Remove the check for stderr and make t5551-23 pass again\n>\n> Signed-off-by: Torsten Bögershausen <tboegi@web.de>\n> ---\n>  t/t5551-http-fetch-smart.sh | 3 +--\n>  1 file changed, 1 insertion(+), 2 deletions(-)\n>\n> diff --git a/t/t5551-http-fetch-smart.sh b/t/t5551-http-fetch-smart.sh\n> index e07eaf3..2c49133 100755\n> --- a/t/t5551-http-fetch-smart.sh\n> +++ b/t/t5551-http-fetch-smart.sh\n> @@ -240,8 +240,7 @@ test_expect_success EXPENSIVE 'create 50,000 tags in the repo' '\n>  '\n>  \n>  test_expect_success EXPENSIVE 'clone the 50,000 tag repo to check OS command line overflow' '\n> -\tgit clone $HTTPD_URL/smart/repo.git too-many-refs 2>err &&\n> -\ttest_line_count = 0 err &&\n> +\tgit clone $HTTPD_URL/smart/repo.git too-many-refs &&\n>  \t(\n>  \t\tcd too-many-refs &&\n>  \t\ttest $(git for-each-ref refs/tags | wc -l) = 50000\n"},{"id":"243697","messageId":"53967F48.1080103@web.de","threadId":"36848","inReplyTo":"xmqqr42xevwi.fsf@gitster.dls.corp.google.com","subject":"Re: [PATCH] t5551: fix the 50,000 tag test","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2014-06-10T03:45:12Z","receivedAt":"2014-06-10T03:45:12Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On 2014-06-09 21.16, Junio C Hamano wrote:\n\n>> Since the day \"git clone\" printed \"Cloning into 'too-many-refs'\" to stderr,\n> \n> Thanks.  It is 68b939b2 (clone: send diagnostic messages to stderr,\n> 2013-09-18); before it we showed the message to the standard output\n> stream instead.\n> \n> Will queue.\nThanks for digging, as we now know better:\ndo you want to squeeze in someting like this:\n\ns/Since the day/Since commit 5e2c7cd2/ \n"},{"id":"243705","messageId":"xmqqoay1b99g.fsf@gitster.dls.corp.google.com","threadId":"36848","inReplyTo":"53967F48.1080103@web.de","subject":"Re: [PATCH] t5551: fix the 50,000 tag test","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-06-10T05:53:31Z","receivedAt":"2014-06-10T05:53:31Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Torsten Bögershausen <tboegi@web.de> writes:\n\n> Thanks for digging, as we now know better:\n> do you want to squeeze in someting like this:\n>\n> s/Since the day/Since commit 5e2c7cd2/ \n\nI usually do not dig only to give suggestions to others, which will\nrisk the time I spent to go to total waste.  When I find the log\nmessage lacking, and if the patch looked otherwise OK enough, I do\nthe digging myself in order to tweak to reduce one round-trip\n(otherwise I'd just discard and tell the submitter to do the\ndigging).  I may send out what I learned as a response, but that is\na mere \"side effect\"; the primary effect of queuing an improved\npatch has often already happened when you see the result of my\ndigging.\n\nI do not recall what I did for this particular patch, but you should\nbe able to fetch and run \"git log origin/master..origin/pu\" to find\nout what I did ;-).\n\nThanks.\n"}]}