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
Add cross build targeting Redox to Travis CI #1095
Conversation
@ids1024 Can you clarify what part of this is breaking the CI, and why fixing it is not achievable? I can dig through this later today and take a look on the "why" myself, but the second question would be better answered by you. Thanks! |
It looks like the |
5fca63a
to
308b24d
Compare
I think adding this is a good idea, but maybe we should allow the Redox build to fail for now? |
That seems reasonable. |
@ids1024 can you rebase this? |
308b24d
to
75141f0
Compare
I rebased, but there seem to be some test failures. I think it's due to changes to the CI environment caused by adding |
I think those failures were already present due to changes in the standard library. Thanks for rebasing! |
Ah, I hadn't checking if they were happening on master. Thanks, this should be helpful to have, as Redox is currently working on moving to using uutils more. |
Oh nice, that's great to hear! |
Yep, the BINS variable here https://github.com/redox-os/cookbook/blob/master/recipes/uutils/recipe.sh identifies which programs we are currently using from uutils. We will continue to contribute to port others. |
Let us know if you run into any issues and we’ll try to fix them quickly. :) |
Good to hear XD |
It is helpful to make sure things are building on Redox, though It's understandable if you don't want Redox breaking CI.
I ran
cargo update
since Redox required newer versions of a dependency or two.