git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: New diff test failures on s390x architecture (was: [ANNOUNCE] Git v2.22.0-rc1)

From
Duy Nguyen <pclouds@gmail.com>
Date
May 23, 2019, 20:30 UTC
Message-ID
<CACsJy8CY8CVVv1+s9sLtVEAhrC5Ki8NSY2qje8+se7VVudFo_g@mail.gmail.com>
In-Reply-To
<20190523191429.GO3654@pobox.com>
On Fri, May 24, 2019 at 2:14 AM Todd Zullinger <tmz@pobox.com> wrote:
Show 46 quoted lines
>
> I wrote:
> > While running the 2.22.0-rc1 tests on Fedora, I hit a few
> > new test failures since 2.21.0 -- but only on the s390x
> > architecture.
> >
> > I haven't had time to dig into these the past few days, so I
> > thought I would send what I do have in case the problem is
> > obvious to someone else.  I think all of the failing tests
> > are due to `git diff` commands.
> [...]
> > I don't have direct access to these s390x builders.  I may
> > be able to arrange shell access (or even reproduce this with
> > qemu's s390x emulation).
>
> I poked around a little with a qemu s390x instance and see
> the same failures.
>
> One simple failure in t4015 is with:
>
>     git diff -w >out &&
>     test_must_be_empty out
>
> Using git-2.21.0 this succeeds, while git-2.22.0-rc1 fails
> and produces:
>
>     diff --git a/x b/x
>     index d99af23..22d9f73 100644
>     --- a/x
>     +++ b/x
>     @@ -1,6 +1,6 @@
>     -whitespace at beginning
>     -whitespace change
>     -whitespace in the middle
>     -whitespace at end
>     +   whitespace at beginning
>     +whitespace          change
>     +white space in the middle
>     +whitespace at end
>      unchanged line
>     -CR at end
>     +CR at end
>
> I am guessing it's no coincidence that this only fails on
> s390x and it is the only big endian architecture in the
> fedora build system.

I see a problem with -w and wrong type casting. sizeof(int) and sizeof(long) on s390x are not the same, are they?

-- 
Duy
Previous: Todd ZullingerNext: Todd Zullinger
Message 11 of 12 in “[ANNOUNCE] Git v2.22.0-rc1”
  1. Junio C HamanoMay 19, 2019
  2. Johannes SchindelinMay 19, 2019
  3. Bryan TurnerMay 20, 2019
  4. Ævar Arnfjörð BjarmasonMay 20, 2019
  5. Duy NguyenMay 21, 2019
  6. Duy NguyenMay 21, 2019
  7. Ævar Arnfjörð BjarmasonMay 21, 2019
  8. Duy NguyenMay 21, 2019
  9. New diff test failures on s390x architecture (was: [ANNOUNCE] Git v2.22.0-rc1)Todd Zullinger, May 23, 2019
  10. Todd ZullingerMay 23, 2019
  11. Duy NguyenMay 23, 2019
  12. Todd ZullingerMay 23, 2019

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.