{"thread":{"id":"30220","subject":"non-fast-forward advice breaks tests on OS X","startedAt":"2012-04-12T05:49:14Z","lastAt":"2012-04-12T21:51:47Z","messageCount":6,"participants":["Brian Gernhardt","Christopher Tiwald","Junio C Hamano"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"189077","messageId":"9F768A58-DEB0-43E1-8AE4-B2A5C4E6CDE9@silverinsanity.com","threadId":"30220","inReplyTo":null,"subject":"non-fast-forward advice breaks tests on OS X","fromName":"Brian Gernhardt","fromEmail":"brian@silverinsanity.com","sentAt":"2012-04-12T05:49:14Z","receivedAt":"2012-04-12T05:49:14Z","isPatch":false,"sender":{"key":"brian@silverinsanity.com","avatar":null},"body":"After a break in regularly compiling git on my Mac 10.7.3 laptop, I had a few tests fail:  t5540.15, t5541.{12,15}.  All of these tests use test_i18ngrep to test the output of git-push for the string \"To prevent you from losing history, non-fast-forward updates were rejected\".\n\nBisecting traced it back to f25950f: \"push: Provide situational hints for non-fast-forward errors\".\n\nI'm sending this off to the list in case someone else has some idea of the root cause.  I'll get back to this problem myself sometime tomorrow.\n\n~~ Brian Gernhardt\n"},{"id":"189100","messageId":"20120412133701.GA367@gmail.com","threadId":"30220","inReplyTo":"9F768A58-DEB0-43E1-8AE4-B2A5C4E6CDE9@silverinsanity.com","subject":"Re: non-fast-forward advice breaks tests on OS X","fromName":"Christopher Tiwald","fromEmail":"christiwald@gmail.com","sentAt":"2012-04-12T13:37:01Z","receivedAt":"2012-04-12T13:37:01Z","isPatch":false,"sender":{"key":"christiwald@gmail.com","avatar":"https://avatars.githubusercontent.com/u/667276?v=4"},"body":"On Thu, Apr 12, 2012 at 01:49:14AM -0400, Brian Gernhardt wrote:\n> After a break in regularly compiling git on my Mac 10.7.3 laptop, I had a few tests fail:  t5540.15, t5541.{12,15}.  All of these tests use test_i18ngrep to test the output of git-push for the string \"To prevent you from losing history, non-fast-forward updates were rejected\".\n> \n> Bisecting traced it back to f25950f: \"push: Provide situational hints for non-fast-forward errors\".\n\nI also run 10.7.3 on a MacBook Pro. I will investigate this now and see\nif I can get tests to break. I know the original patch passed the test\nsuite on this machine.\n\n--\nChristopher Tiwald\n"},{"id":"189102","messageId":"20120412140049.GB367@gmail.com","threadId":"30220","inReplyTo":"20120412133701.GA367@gmail.com","subject":"Re: non-fast-forward advice breaks tests on OS X","fromName":"Christopher Tiwald","fromEmail":"christiwald@gmail.com","sentAt":"2012-04-12T14:00:49Z","receivedAt":"2012-04-12T14:00:49Z","isPatch":false,"sender":{"key":"christiwald@gmail.com","avatar":"https://avatars.githubusercontent.com/u/667276?v=4"},"body":"On Thu, Apr 12, 2012 at 09:37:01AM -0400, Christopher Tiwald wrote:\n> On Thu, Apr 12, 2012 at 01:49:14AM -0400, Brian Gernhardt wrote:\n> > After a break in regularly compiling git on my Mac 10.7.3 laptop, I had a few tests fail:  t5540.15, t5541.{12,15}.  All of these tests use test_i18ngrep to test the output of git-push for the string \"To prevent you from losing history, non-fast-forward updates were rejected\".\n> > \n> > Bisecting traced it back to f25950f: \"push: Provide situational hints for non-fast-forward errors\".\n> \n> I also run 10.7.3 on a MacBook Pro. I will investigate this now and see\n> if I can get tests to break. I know the original patch passed the test\n> suite on this machine.\n\nAh ha, I wasn't running the proper tests. I'll send a patch to the\naffected ones later today.\n\n*** t5540-http-push.sh ***\n# passed all 0 test(s)\n1..0 # SKIP Network testing disabled (define GIT_TEST_HTTPD to enable)\n*** t5541-http-push.sh ***\n# passed all 0 test(s)\n1..0 # SKIP Network testing disabled (define GIT_TEST_HTTPD to enable)\n*** t5550-http-fetch.sh ***\n# passed all 0 test(s)\n1..0 # SKIP Network testing disabled (define GIT_TEST_HTTPD to enable)\n*** t5551-http-fetch.sh ***\n# passed all 0 test(s)\n1..0 # SKIP Network testing disabled (define GIT_TEST_HTTPD to enable)\n"},{"id":"189109","messageId":"7v8vi1gdzk.fsf@alter.siamese.dyndns.org","threadId":"30220","inReplyTo":"20120412140049.GB367@gmail.com","subject":"Re: non-fast-forward advice breaks tests on OS X","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-04-12T15:35:11Z","receivedAt":"2012-04-12T15:35:11Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Christopher Tiwald <christiwald@gmail.com> writes:\n\n>> I also run 10.7.3 on a MacBook Pro. I will investigate this now and see\n>> if I can get tests to break. I know the original patch passed the test\n>> suite on this machine.\n>\n> Ah ha, I wasn't running the proper tests. I'll send a patch to the\n> affected ones later today.\n\nPlease do so.  I assume that the test vectors were expecting specific\nerror/advice messages that need to be updated?\n"},{"id":"189124","messageId":"20120412175628.GH367@gmail.com","threadId":"30220","inReplyTo":"7v8vi1gdzk.fsf@alter.siamese.dyndns.org","subject":"[PATCH] Fix httpd tests that broke when non-ff push advice changed","fromName":"Christopher Tiwald","fromEmail":"christiwald@gmail.com","sentAt":"2012-04-12T17:56:28Z","receivedAt":"2012-04-12T17:56:28Z","isPatch":true,"sender":{"key":"christiwald@gmail.com","avatar":"https://avatars.githubusercontent.com/u/667276?v=4"},"body":"On Thu, Apr 12, 2012 at 08:35:11AM -0700, Junio C Hamano wrote:\n> Please do so.  I assume that the test vectors were expecting specific\n> error/advice messages that need to be updated?\n\nLooks to be the case. The below fixes the tests on my machine. It might\nbe worth adding tests to check the specific advice messages, but that's a\ndifferent topic and regardless wouldn't belong in the http-push series.\n\n---- >8 ----\nSigned-off-by: Christopher Tiwald <christiwald@gmail.com>\n---\n t/lib-httpd.sh       |    2 +-\n t/t5541-http-push.sh |    2 +-\n 2 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/t/lib-httpd.sh b/t/lib-httpd.sh\nindex f7dc078..094d490 100644\n--- a/t/lib-httpd.sh\n+++ b/t/lib-httpd.sh\n@@ -160,6 +160,6 @@ test_http_push_nonff() {\n \t'\n \n \ttest_expect_success 'non-fast-forward push shows help message' '\n-\t\ttest_i18ngrep \"To prevent you from losing history, non-fast-forward updates were rejected\" output\n+\t\ttest_i18ngrep \"Updates were rejected because\" output\n \t'\n }\ndiff --git a/t/t5541-http-push.sh b/t/t5541-http-push.sh\nindex cc6f081..57c3e48 100755\n--- a/t/t5541-http-push.sh\n+++ b/t/t5541-http-push.sh\n@@ -167,7 +167,7 @@ test_expect_success 'push fails for non-fast-forward refs unmatched by remote he\n '\n \n test_expect_success 'push fails for non-fast-forward refs unmatched by remote helper: our output' '\n-\ttest_i18ngrep \"To prevent you from losing history, non-fast-forward updates were rejected\" \\\n+\ttest_i18ngrep \"Updates were rejected because\" \\\n \t\toutput\n '\n \n-- \n1.7.10.167.gf245c\n\n--\nChristopher Tiwald\n"},{"id":"189149","messageId":"7vhawo4o0c.fsf@alter.siamese.dyndns.org","threadId":"30220","inReplyTo":"20120412175628.GH367@gmail.com","subject":"Re: [PATCH] Fix httpd tests that broke when non-ff push advice changed","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-04-12T21:51:47Z","receivedAt":"2012-04-12T21:51:47Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Christopher Tiwald <christiwald@gmail.com> writes:\n\n> On Thu, Apr 12, 2012 at 08:35:11AM -0700, Junio C Hamano wrote:\n>> Please do so.  I assume that the test vectors were expecting specific\n>> error/advice messages that need to be updated?\n>\n> Looks to be the case. The below fixes the tests on my machine. It might\n> be worth adding tests to check the specific advice messages, but that's a\n> different topic and regardless wouldn't belong in the http-push series.\n\nThanks.\n"}]}