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 are broken with Python 3.12 on Debian build #96

Open
goffi-contrib opened this issue Jul 21, 2024 · 7 comments
Open

Tests are broken with Python 3.12 on Debian build #96

goffi-contrib opened this issue Jul 21, 2024 · 7 comments

Comments

@goffi-contrib
Copy link

Hello,

I'm writing this ticket for the Debian developer maintaining txdbus on this distribution, for some reason they can't publish here.

The Debian build it failing due to some unpacking error: https://ci.debian.net/packages/t/txdbus/testing/s390x/49118708/

From the debian dev:

I guess it is about endianess. s390x is big endian, TTBOMK. Maybe the tests assume (or fallback to) little endian?

Thanks!

@goffi-contrib
Copy link
Author

Note: here are the patches used in Debian to fix the issues: https://salsa.debian.org/python-team/packages/txdbus/-/tree/debian/master/debian/patches?ref_type=heads

@WhyNotHugo
Copy link
Collaborator

The error states:

 63s AssertionError: b'\x04(ii)\x00\x00\x00\x01\x00\x00\x00\x02\x00\x00\x00' != b'\x04(ii)\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x02' : Binary encoding differs from expected value

This definitely sound like the mismatch that one would see when assuming the opposite endianness.

@WhyNotHugo
Copy link
Collaborator

All the errors are quads of bytes in inverted order, so definitely an endiannes issue.

@goffi-contrib
Copy link
Author

Hello. Is there any plan to fix it soon? The package will be removed from Debian testing because of this unfortunately. Thanks.

@cocagne
Copy link
Owner

cocagne commented Aug 14, 2024 via email

@goffi-contrib
Copy link
Author

Thanks for your quick answer @cocagne .

According to distrowatch 3.12 is available on Debian testing (Trixie): https://distrowatch.com/table-mobile.php?distribution=debian

@WhyNotHugo
Copy link
Collaborator

Alpine 3.20 and ArchLinux both ship Python 3.12.

cocagne added a commit that referenced this issue Sep 13, 2024
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

3 participants