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
failed to build on 32bit powerpc #222
Comments
There is a few more architectures that did not build (red): https://buildd.debian.org/status/package.php?p=klystrack&suite=unstable |
The method I use there is terrible but I guess it's unavoidable since I'm not rewriting this in the near future. :) I assume the pointers something like 48 bits wide or so. Need to check if there's a safe way to force it to 64 bits or so. |
Maybe this has something to do with this?
Try turning that option off and see if it builds then. |
That indeed is what stops the build but it's never good to just ignore
warnings. It would probably work right now but what if the pointer type was
larger than the provided 64 bits? Better to fix the warning.
2018-01-03 21:50 GMT+02:00 darkhog <notifications@github.com>:
… Maybe this has something to do with this?
cc1: all warnings being treated as errors
Try turning that option off and see if it builds then.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#222 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABCK6ZWjS3rqrf67CuVbvP9yHafy6tmhks5tG9oAgaJpZM4RJ0lj>
.
|
I am not able to test all platforms but I just committed a fix here: kometbomb/klystron@ba8e768 This ensures the cast is done with a type whose width equals that of a pointer type. Can the build process above be tested with the branch referenced above? |
I can test it on 32bit ppc... will report back results here. |
works |
Thanks a lot for testing! I'll merge the klystron PR. |
The text was updated successfully, but these errors were encountered: