chafa icon indicating copy to clipboard operation
chafa copied to clipboard

v1.14.0: test failure on sparc64

Open cdluminate opened this issue 1 year ago • 7 comments

See https://buildd.debian.org/status/package.php?p=chafa

sparc64 fails to pass 6/7 tests.

============================================================================
Testsuite summary for chafa 1.14.0
============================================================================
# TOTAL: 7
# PASS:  1
# SKIP:  0
# XFAIL: 0
# FAIL:  6
# XPASS: 0
# ERROR: 0
============================================================================
See tests/test-suite.log

I do not have the tests/test-suite.log file. I need to dump it this file to stdout upon the next time I upload a revision to this package. Will get back to you when I have more details on this.

You may also notice the build failure on long64 -- that's pretty much toolchain issues, and you can skip it.

cdluminate avatar Jan 19 '24 14:01 cdluminate

Thanks. Maybe I can get the build to dump tests/test-suite.log on failure to make things easier in the future. I'll look into that.

Wild speculation: The "bad files" test succeeded, since it only tests known broken input files that should be rejected. All other tests failed, either because the PNG loader is broken, or the new Smolscale code I imported has an arch-specific issue (most likely).

hpjansson avatar Jan 20 '24 02:01 hpjansson

Thanks. Maybe I can get the build to dump tests/test-suite.log on failure to make things easier in the future. I'll look into that.

Maybe something like configure --verbose-test?

Wild speculation: The "bad files" test succeeded, since it only tests known broken input files that should be rejected. All other tests failed, either because the PNG loader is broken, or the new Smolscale code I imported has an arch-specific issue (most likely).

Yes. You can click the old link on our buildd page. The sparc64 build failure is in fact a regression. The new code added between 1.12.4 and 1.14.0 is highly likely the root of the issue.

cdluminate avatar Jan 20 '24 03:01 cdluminate

Just added commit c4695d4fd8614329544a7cf6d6a86c8cab85a039 - it will dump logs of failed tests to console.

hpjansson avatar Jan 20 '24 16:01 hpjansson

I suspect it's an alignment issue, and the logs will probably just say "Bus error". So no need to hurry, I can probably trigger the issue here. I don't have sparc64, but I have an extremely old arm board and a riscv one here somewhere.

hpjansson avatar Jan 21 '24 17:01 hpjansson

The tests succeed on armv6l and riscv64. Looking closer at the Debian sparc64 log, it seems like the successful test is the ChafaTermInfo standalone API test, but all the tests that involve running the chafa CLI tool fail. So maybe the bug happens earlier in the CLI tool run, and not in Smolscale after all. But it's hard to tell:

PASS: term-info-test
FAIL: chafa-tool-bad-test.sh
FAIL: chafa-tool-cmode-test.sh
FAIL: chafa-tool-format-test.sh
FAIL: chafa-tool-loader-test.sh
FAIL: chafa-tool-options-test.sh
FAIL: chafa-tool-pipe-test.sh

hpjansson avatar Jan 24 '24 15:01 hpjansson

It seems DDs have access to sparc64 porterbox https://db.debian.org/machines.cgi . I'll rebuild with that patch and get back. May lag a little bit as I'm busy with paper rebuttal recently.

cdluminate avatar Jan 24 '24 19:01 cdluminate

I wasn't able to get the Debian sparc64 port working in qemu, but managed with NetBSD. It passes the tests and runs fine there from what I can tell. Maybe something going on with the Debian builds?

(Had to build it with clang, since lodepng.c made gcc crash).

hpjansson avatar May 30 '24 15:05 hpjansson