threads / discuss / 45929

https://travis-ci.org/git/git/builds/

Subject: https://travis-ci.org/git/git/builds/

## tl;dr

3 messages between May 11, 2017 and May 11, 2017.

replies: 2people: 2as markdown or json

Junio C Hamano· May 11, 2017, 07:46 UTC · lore

I was hoping that by ejecting a few topics out of 'pu', we could finally see all the build jobs pass their tests under Travis CI. Unfortunately, no such luck. It seems that we have some new issues with Gettext-Poison build/test after updates to t0027. Also the updated compatibility regexp thing taken from gawk is not liked by the Windows compiler.

Not all of these may not be new issues, but nevertheless, it would be nice if we can see all build jobs succeed every once in a while, not just for the three stable integration branches but also for 'pu' ;-)

Thanks.
Ævar Arnfjörð Bjarmason· May 11, 2017, 08:11 UTC · re: Junio C Hamano · lore

Re: https://travis-ci.org/git/git/builds/

On Thu, May 11, 2017 at 9:46 AM, Junio C Hamano <gitster@pobox.com> wrote:
Show 11 quoted lines
> I was hoping that by ejecting a few topics out of 'pu', we could
> finally see all the build jobs pass their tests under Travis CI.
> Unfortunately, no such luck.  It seems that we have some new issues
> with Gettext-Poison build/test after updates to t0027.  Also the
> updated compatibility regexp thing taken from gawk is not liked by
> the Windows compiler.
>
> Not all of these may not be new issues, but nevertheless, it would
> be nice if we can see all build jobs succeed every once in a while,
> not just for the three stable integration branches but also for 'pu'
> ;-)

The gettext part is my bad, I never tested with EXPENSIVE which Torsten removed, so now the test runs & fails under poison.

This trivial fixup makes it pass:
diff --git a/t/t0027-auto-crlf.sh b/t/t0027-auto-crlf.sh
index effc3db0d5..deb3ae7813 100755
--- a/t/t0027-auto-crlf.sh
+++ b/t/t0027-auto-crlf.sh
@@ -69,7 +69,7 @@ check_warning () {
        *) echo >&2 "Illegal 1": "$1" ; return false ;;
        esac
        grep "will be replaced by" "$2" | sed -e "s/\(.*\) in [^
]*$/\1/" | uniq  >"$2".actual
-       test_cmp "$2".expect "$2".actual
+       test_i18ncmp "$2".expect "$2".actual
 }

 commit_check_warn () {

Can you please squash that into my patch?
Junio C Hamano· May 11, 2017, 09:51 UTC · re: Ævar Arnfjörð Bjarmason · lore

Re: https://travis-ci.org/git/git/builds/

Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
Show 34 quoted lines
> On Thu, May 11, 2017 at 9:46 AM, Junio C Hamano <gitster@pobox.com> wrote:
>> I was hoping that by ejecting a few topics out of 'pu', we could
>> finally see all the build jobs pass their tests under Travis CI.
>> Unfortunately, no such luck.  It seems that we have some new issues
>> with Gettext-Poison build/test after updates to t0027.  Also the
>> updated compatibility regexp thing taken from gawk is not liked by
>> the Windows compiler.
>>
>> Not all of these may not be new issues, but nevertheless, it would
>> be nice if we can see all build jobs succeed every once in a while,
>> not just for the three stable integration branches but also for 'pu'
>> ;-)
>
> The gettext part is my bad, I never tested with EXPENSIVE which
> Torsten removed, so now the test runs & fails under poison.
>
> This trivial fixup makes it pass:
>
> diff --git a/t/t0027-auto-crlf.sh b/t/t0027-auto-crlf.sh
> index effc3db0d5..deb3ae7813 100755
> --- a/t/t0027-auto-crlf.sh
> +++ b/t/t0027-auto-crlf.sh
> @@ -69,7 +69,7 @@ check_warning () {
>         *) echo >&2 "Illegal 1": "$1" ; return false ;;
>         esac
>         grep "will be replaced by" "$2" | sed -e "s/\(.*\) in [^
> ]*$/\1/" | uniq  >"$2".actual
> -       test_cmp "$2".expect "$2".actual
> +       test_i18ncmp "$2".expect "$2".actual
>  }
>
>  commit_check_warn () {
>
> Can you please squash that into my patch?
Sure.  Thanks for a quick response.

← back to recent threads