On Tue, 1 Nov 2022 at 20:05, Martin Ågren <martin.agren@xxxxxxxxx> wrote: > > Hi Sarah, > > On Tue, 1 Nov 2022 at 19:38, Sarah Julia Kriesch > <sarah.kriesch@xxxxxxxxxxxx> wrote: > > > > I am one of the zsystems Maintainers (s390x architecture) at openSUSE > > and git can not be built any more since the update from 2.37.3 to > > 2.38.0, and with 2.38.1 afterwards. We have tried to debug it. > > > > There is no real test error output, but we can identify, that it is > > happening in the test part with check-chainlint. > > > [ 387s] make[1]: *** [Makefile:83: check-chainlint] Error 1 > > [ 387s] make[1]: Leaving directory > > '/home/abuild/rpmbuild/BUILD/git-2.38.1/t' > > [ 387s] make[1]: *** Waiting for unfinished jobs.... > > [ 388s] make: *** [Makefile:3075: test] Error 2 > > [ 388s] error: Bad exit status from /var/tmp/rpm-tmp.ztqAAp (%check) > > > How can we fix this issue? > > You can disable this feature by setting the GIT_TEST_CHAIN_LINT > environment variable to "0". That should hopefully allow you to run the > tests successfully. > > That shouldn't risk you anything, like reduce test coverage or similar. > That the tests are chainlint-ok (roughly speaking, that they are written > the way the Git project likes its tests) has already been established by > many others at this point. On further reading, it appears this error happens in the "check-chainlint" target without even checking this environment variable (that happens elsewhere). I doubt my original suggestion will be a complete workaround. One approach might be to patch the "check-chainlint" target into a no-op, possibly combined with turning off the chainlinting as I mentioned above. My makefile-foo is unfortunately not sufficiently advanced to immediately spot what might be going on here. At any rate, all of the above is just about working around the issue. As for a proper fix, let's hope someone like Eric can chime in. Martin
