Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v1.14.0: test failure on sparc64 #184

Open
cdluminate opened this issue Jan 19, 2024 · 7 comments
Open

v1.14.0: test failure on sparc64 #184

cdluminate opened this issue Jan 19, 2024 · 7 comments
Labels
bug Not working as advertised

Comments

@cdluminate
Copy link
Collaborator

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.

@hpjansson
Copy link
Owner

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).

@cdluminate
Copy link
Collaborator Author

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.

@hpjansson
Copy link
Owner

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

@hpjansson
Copy link
Owner

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 hpjansson added the bug Not working as advertised label Jan 23, 2024
@hpjansson
Copy link
Owner

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

@cdluminate
Copy link
Collaborator Author

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.

@hpjansson
Copy link
Owner

hpjansson commented May 30, 2024

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).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Not working as advertised
Projects
None yet
Development

No branches or pull requests

2 participants