{"thread":{"id":"45929","subject":"https://travis-ci.org/git/git/builds/","startedAt":"2017-05-11T07:46:49Z","lastAt":"2017-05-11T09:51:21Z","messageCount":3,"participants":["Junio C Hamano","Ævar Arnfjörð Bjarmason"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"319332","messageId":"xmqqfugbbzto.fsf@gitster.mtv.corp.google.com","threadId":"45929","inReplyTo":null,"subject":"https://travis-ci.org/git/git/builds/","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-05-11T07:46:43Z","receivedAt":"2017-05-11T07:46:49Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"I was hoping that by ejecting a few topics out of 'pu', we could\nfinally see all the build jobs pass their tests under Travis CI.\nUnfortunately, no such luck.  It seems that we have some new issues\nwith Gettext-Poison build/test after updates to t0027.  Also the\nupdated compatibility regexp thing taken from gawk is not liked by\nthe Windows compiler.\n\nNot all of these may not be new issues, but nevertheless, it would\nbe nice if we can see all build jobs succeed every once in a while,\nnot just for the three stable integration branches but also for 'pu'\n;-)\n\nThanks.\n\n\n\n\n"},{"id":"319338","messageId":"CACBZZX5tf9seb0mCR+UOUCk3jA-L47C9qowSnBzzNq9eZ4+H+g@mail.gmail.com","threadId":"45929","inReplyTo":"xmqqfugbbzto.fsf@gitster.mtv.corp.google.com","subject":"Re: https://travis-ci.org/git/git/builds/","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2017-05-11T08:11:33Z","receivedAt":"2017-05-11T08:12:01Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"On Thu, May 11, 2017 at 9:46 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> I was hoping that by ejecting a few topics out of 'pu', we could\n> finally see all the build jobs pass their tests under Travis CI.\n> Unfortunately, no such luck.  It seems that we have some new issues\n> with Gettext-Poison build/test after updates to t0027.  Also the\n> updated compatibility regexp thing taken from gawk is not liked by\n> the Windows compiler.\n>\n> Not all of these may not be new issues, but nevertheless, it would\n> be nice if we can see all build jobs succeed every once in a while,\n> not just for the three stable integration branches but also for 'pu'\n> ;-)\n\nThe gettext part is my bad, I never tested with EXPENSIVE which\nTorsten removed, so now the test runs & fails under poison.\n\nThis trivial fixup makes it pass:\n\ndiff --git a/t/t0027-auto-crlf.sh b/t/t0027-auto-crlf.sh\nindex effc3db0d5..deb3ae7813 100755\n--- a/t/t0027-auto-crlf.sh\n+++ b/t/t0027-auto-crlf.sh\n@@ -69,7 +69,7 @@ check_warning () {\n        *) echo >&2 \"Illegal 1\": \"$1\" ; return false ;;\n        esac\n        grep \"will be replaced by\" \"$2\" | sed -e \"s/\\(.*\\) in [^\n]*$/\\1/\" | uniq  >\"$2\".actual\n-       test_cmp \"$2\".expect \"$2\".actual\n+       test_i18ncmp \"$2\".expect \"$2\".actual\n }\n\n commit_check_warn () {\n\nCan you please squash that into my patch?\n"},{"id":"319380","messageId":"xmqqd1bfafhp.fsf@gitster.mtv.corp.google.com","threadId":"45929","inReplyTo":"CACBZZX5tf9seb0mCR+UOUCk3jA-L47C9qowSnBzzNq9eZ4+H+g@mail.gmail.com","subject":"Re: https://travis-ci.org/git/git/builds/","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-05-11T09:51:14Z","receivedAt":"2017-05-11T09:51:21Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:\n\n> On Thu, May 11, 2017 at 9:46 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>> I was hoping that by ejecting a few topics out of 'pu', we could\n>> finally see all the build jobs pass their tests under Travis CI.\n>> Unfortunately, no such luck.  It seems that we have some new issues\n>> with Gettext-Poison build/test after updates to t0027.  Also the\n>> updated compatibility regexp thing taken from gawk is not liked by\n>> the Windows compiler.\n>>\n>> Not all of these may not be new issues, but nevertheless, it would\n>> be nice if we can see all build jobs succeed every once in a while,\n>> not just for the three stable integration branches but also for 'pu'\n>> ;-)\n>\n> The gettext part is my bad, I never tested with EXPENSIVE which\n> Torsten removed, so now the test runs & fails under poison.\n>\n> This trivial fixup makes it pass:\n>\n> diff --git a/t/t0027-auto-crlf.sh b/t/t0027-auto-crlf.sh\n> index effc3db0d5..deb3ae7813 100755\n> --- a/t/t0027-auto-crlf.sh\n> +++ b/t/t0027-auto-crlf.sh\n> @@ -69,7 +69,7 @@ check_warning () {\n>         *) echo >&2 \"Illegal 1\": \"$1\" ; return false ;;\n>         esac\n>         grep \"will be replaced by\" \"$2\" | sed -e \"s/\\(.*\\) in [^\n> ]*$/\\1/\" | uniq  >\"$2\".actual\n> -       test_cmp \"$2\".expect \"$2\".actual\n> +       test_i18ncmp \"$2\".expect \"$2\".actual\n>  }\n>\n>  commit_check_warn () {\n>\n> Can you please squash that into my patch?\n\nSure.  Thanks for a quick response.\n"}]}