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

Tests failing on s390x on Alpine Linux #1078

Closed
chereskata opened this issue Sep 13, 2023 · 5 comments · Fixed by #1079
Closed

Tests failing on s390x on Alpine Linux #1078

chereskata opened this issue Sep 13, 2023 · 5 comments · Fixed by #1079

Comments

@chereskata
Copy link

Hi,

after trying to package your cool linter for Alpine Linux, i noticed, that the following four tests failed on s390x architecture:

 PASSED  ] 2340 tests.
[  FAILED  ] 4 tests, listed below:
[  FAILED  ] Test_Trace_Reader_Writer.event_vscode_document_opened
[  FAILED  ] Test_Trace_Reader_Writer.event_vscode_document_closed
[  FAILED  ] Test_Trace_Reader_Writer.event_vscode_document_changed
[  FAILED  ] Test_Trace_Reader_Writer.event_vscode_document_sync

The whole build log is attached.

Thanks a lot!
log.txt

@strager
Copy link
Collaborator

strager commented Sep 13, 2023

The title says "segfaulting", but I don't see evidence of a segfault in the logs. Did you mean that the test just failed?

Is s390x big endian? My hypothesis is that we just don't support big endian for trace file parsing. (Trace files are little endian.)

// TODO(strager): This assumes the native endian is little endian.

I created #1079 to address this issue, but I cannot test it.

@chereskata
Copy link
Author

chereskata commented Sep 14, 2023 via email

@chereskata
Copy link
Author

chereskata commented Sep 14, 2023 via email

@chereskata
Copy link
Author

Unfortunately it does not work, see here

@chereskata chereskata changed the title Tests segfaulting on s390x on Alpine Linux Tests failing on s390x on Alpine Linux Sep 14, 2023
@strager
Copy link
Collaborator

strager commented Sep 15, 2023

@chereskata Can you try again (Git commit 23b3099)?

If it doesn't work, I guess I gotta figure out how to emulate a big endian machine. (I have a MIPS router, but I don't think it'll fit quick-lint-js. xD)

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

Successfully merging a pull request may close this issue.

2 participants