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

Fix the build + tests under non little-endian archs #1

Merged
merged 1 commit into from
Jul 24, 2020

Conversation

huitseeker
Copy link
Contributor

@huitseeker huitseeker commented Jul 22, 2020

Testing the build with cross test --target powerpc-unknown-linux-gnu results in the following:
https://gist.github.com/e0e56e236ea832ea7fc70da2e92049e1

See cross: https://github.com/rust-embedded/cross

Relatedly, the powerpc build is not tested in CI.

Testing the build with `cross test --target powerpc-unknown-linux-gnu` results in the following:
https://gist.github.com/e0e56e236ea832ea7fc70da2e92049e1

Relatedly, the powerpc build is not tested in CI.
@huitseeker
Copy link
Contributor Author

huitseeker commented Jul 22, 2020

@rozbb Happy to contribute a {normal, powerpc} build based on Github actions if you so wish. I'm not so familiar with travis, though >_<

@rozbb
Copy link
Owner

rozbb commented Jul 24, 2020

Thank you! I'm gonna push my Github workflow in a bit. I would absolutely welcome a PR that runs tests on more arches!

@huitseeker
Copy link
Contributor Author

huitseeker commented Jul 24, 2020

@rozbb See #2 for the CI.
Please consider releasing a version that includes this bugfix, as it would allow some dependencies to expand to BE archs!

@rozbb
Copy link
Owner

rozbb commented Jul 24, 2020

Already released the bugfix

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 this pull request may close these issues.

None yet

2 participants