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

Support running on big-endian architectures #59

Open
Tachi107 opened this issue Nov 9, 2022 · 3 comments
Open

Support running on big-endian architectures #59

Tachi107 opened this issue Nov 9, 2022 · 3 comments
Labels
C-bug Category: This is a bug P-low Priority: Low

Comments

@Tachi107
Copy link
Contributor

Tachi107 commented Nov 9, 2022

Hi, it seems that the VectorTest.Destructor test is failing on Debian's powerpc and powerpc64 architectures :/

I have not looked into the issue, but I guess that this might be related to ppc's big endianness.

You can find the logs for both here:

More generally, the build logs for all architectures are here: https://buildd.debian.org/status/logs.php?pkg=zycore-c

@athre0z
Copy link
Member

athre0z commented Nov 9, 2022

Yeah, big-endian is currently completely unsupported and this would require fixes all over the code-base. We'll have to introduce some endianess conversion helpers at some point and then go through the whole code-base. We usually left TODO comments where endianess is relevant, but there are likely also cases where such comments are missing. Given how rare BE has become in practice, I'm afraid this is not a high priority issue for us.

@athre0z athre0z added C-bug Category: This is a bug P-low Priority: Low labels Nov 9, 2022
@Tachi107
Copy link
Contributor Author

Tachi107 commented Nov 9, 2022

Oh ok, thanks for the fast reply! powerpc isn't an "official" Debian architecture, so this isn't really a big deal. Feel free to close this, or to keep it open as a kind of todo item :)

Edit: actually, one of Debian's official architectures is big endian, s390x. In any case, you can monitor Zycore's Debian status on https://buildd.debian.org/status/package.php?p=zycore-c (yeah, we have a lot of ports :)

@athre0z athre0z changed the title Vector test failing on powerpc Support running on big-endian architectures Nov 9, 2022
@athre0z
Copy link
Member

athre0z commented Nov 9, 2022

Cool. Let's leave this open as a low priority TODO item and as a reference for other people who are wondering about the BE support status. I adjusted the title to improve discoverability.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug P-low Priority: Low
Projects
None yet
Development

No branches or pull requests

2 participants