threads / discuss / 46267

Solaris 11.3 SPARC grep problem with t1450-fsck.sh

Subject: Solaris 11.3 SPARC grep problem with t1450-fsck.sh

## tl;dr

4 messages between Jun 27, 2017 and Jun 28, 2017.

replies: 3people: 2as markdown or json

Michael Kebe· Jun 27, 2017, 12:15 UTC · lore

Running the test suite on Solaris 11.3 SPARC fails on a test in t1450-fsck.sh.

not ok 60 - fsck --name-objects # # rm -rf name-objects && # git init name-objects && # ( # cd name-objects && # test_commit julius caesar.t && # test_commit augustus && # test_commit caesar && # remove_object $(git rev-parse julius:caesar.t) && # test_must_fail git fsck --name-objects >out && # tree=$(git rev-parse --verify julius:) && # grep "$tree ($refs/heads/master\|HEAD$@{[0-9]*}:" out # )

Solaris has /usr/bin/grep and /usr/bin/ggrep. grep is a solaris version and ggrep is the GNU grep.

Replacing grep with ggrep in t1450-fsck.sh script the fixes the problem.

I chatted with Ævar and he thinks that the problem is that configure finds the right grep but it is not properly passed to the tests.

Furthermore I discovered that some tests in t3900-i18n-commit.sh are failing.

e.g.: not ok 17 - ISO-2022-JP should be shown in UTF-8 now

But that's another topic, which I will open once this is fixed.

Greetings Michael

Junio C Hamano· Jun 27, 2017, 16:25 UTC · re: Michael Kebe · lore

Re: Solaris 11.3 SPARC grep problem with t1450-fsck.sh

Michael Kebe <michael.kebe@gmail.com> writes:

Show 19 quoted lines

> Running the test suite on Solaris 11.3 SPARC fails on a test in t1450-fsck.sh.
>
> not ok 60 - fsck --name-objects
> #
> #               rm -rf name-objects &&
> #               git init name-objects &&
> #               (
> #                       cd name-objects &&
> #                       test_commit julius caesar.t &&
> #                       test_commit augustus &&
> #                       test_commit caesar &&
> #                       remove_object $(git rev-parse julius:caesar.t) &&
> #                       test_must_fail git fsck --name-objects >out &&
> #                       tree=$(git rev-parse --verify julius:) &&
> #                       grep "$tree (\(refs/heads/master\|HEAD\)@{[0-9]*}:" out
> #               )
>
> Solaris has /usr/bin/grep and /usr/bin/ggrep. grep is a solaris
> version and ggrep is the GNU grep.

We do not need GNUism, but many tools that come in /usr/bin have been found unfit. I thought that Solaris users were expected to set SANE_TOOL_PATH to point at /usr/xpg[46]/bin to avoid this problem.

Ah, wait, that particular grep may have GNUism. If you changed it to

    egrep "$tree \((refs/heads/master|HEAD)@{[0-9]*}:" out

does it make it pass for you?

Michael Kebe· Jun 28, 2017, 06:00 UTC · re: Junio C Hamano · lore

Re: Solaris 11.3 SPARC grep problem with t1450-fsck.sh

2017-06-27 18:25 GMT+02:00 Junio C Hamano <gitster@pobox.com>:

Show 6 quoted lines

> Ah, wait, that particular grep may have GNUism.  If you changed it
> to
>
>     egrep "$tree \((refs/heads/master|HEAD)@{[0-9]*}:" out
>
> does it make it pass for you?

Yes, this is working.

Greetings Michael

Junio C Hamano· Jun 28, 2017, 17:15 UTC · re: Michael Kebe · lore

Re: Solaris 11.3 SPARC grep problem with t1450-fsck.sh

Michael Kebe <michael.kebe@gmail.com> writes:

Show 9 quoted lines

> 2017-06-27 18:25 GMT+02:00 Junio C Hamano <gitster@pobox.com>:
>> Ah, wait, that particular grep may have GNUism.  If you changed it
>> to
>>
>>     egrep "$tree \((refs/heads/master|HEAD)@{[0-9]*}:" out
>>
>> does it make it pass for you?
>
> Yes, this is working.

Thanks.