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

Test fails on all big-endian architectures #9

Closed
skitt opened this issue Dec 31, 2013 · 3 comments
Closed

Test fails on all big-endian architectures #9

skitt opened this issue Dec 31, 2013 · 3 comments

Comments

@skitt
Copy link
Contributor

skitt commented Dec 31, 2013

Hi Claudio,

In 4.2.1 the tests fail on all big-endian architectures Debian builds for (mips, powerpc, s390x, sparc); see https://buildd.debian.org/status/package.php?p=libxmp for the build logs (not very detailed on the test side). I'll look into the failures some more as time permits!

Regards,

Stephen

@cmatsuoka
Copy link
Collaborator

Hi Stephen,

Thanks, this is probably caused by a change in the way the test is
performed (it now checks the MD5 digest of the generated data instead of
comparing it to a large raw PCM dump), to save about 200KB in the
distribution package. I'll set up a big-endian system emulator here to fix
that and provide a patch.

Best regards,
Claudio

On Tue, Dec 31, 2013 at 12:35 PM, Stephen Kitt notifications@github.comwrote:

Hi Claudio,

In 4.2.1 the tests fail on all big-endian architectures Debian builds for
(mips, powerpc, s390x, sparc); see
https://buildd.debian.org/status/package.php?p=libxmp for the build logs
(not very detailed on the test side). I'll look into the failures some more
as time permits!

Regards,

Stephen


Reply to this email directly or view it on GitHubhttps://github.com//issues/9
.

@cmatsuoka
Copy link
Collaborator

Hi,

It was indeed a bug in the test validation. The attached patch should fix
it.

Best regards,
Claudio

On Tue, Dec 31, 2013 at 12:35 PM, Stephen Kitt notifications@github.comwrote:

Hi Claudio,

In 4.2.1 the tests fail on all big-endian architectures Debian builds for
(mips, powerpc, s390x, sparc); see
https://buildd.debian.org/status/package.php?p=libxmp for the build logs
(not very detailed on the test side). I'll look into the failures some more
as time permits!

Regards,

Stephen


Reply to this email directly or view it on GitHubhttps://github.com//issues/9
.

@skitt
Copy link
Contributor Author

skitt commented Jan 1, 2014

Excellent, that does indeed fix the validation. Thanks!

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

No branches or pull requests

2 participants