-
Notifications
You must be signed in to change notification settings - Fork 681
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
ci: remove thread parallelism #539
ci: remove thread parallelism #539
Conversation
Added: - arm-unknown-linux-gnueabi - arm-unknown-linux-musleabi Removed: - powerpc64-unknown-linux-gnu (not suppported by nix) - mips64el-unknown-linux-gnu (not suppported by nix) - mipsel-unknown-linux-gnu (not suppported by nix)
Removed: - mips64-unknown-linux-gnu - mips64el-unknown-linux-gnu - arm-unknown-linux-musleabi
Switch to trust-based CI
Switched bsd speed_t def. to libc::speed_t
99dd770
to
ee0ecea
Compare
Fixes nix-rust#529, also refer to nix-rust#529 which first added this for the old CI infrastructure. Signed-off-by: Paul Osborne <[email protected]>
ee0ecea
to
d0a151b
Compare
You meantion the same issue twice. Do you want me to homu r+ this? Or do you just want to merge this stuff yourself? |
r+ is fine. It doesn't look like this is enough to resolve all issues on its own, so I'm still investigating. I need to update this as well as it should be |
Your answer confuses me. But @homu r+ |
📌 Commit d0a151b has been approved by |
@homu r- |
💥 Test timed out |
@posborne can you double check the issue references in your PR / commit message? |
☔ The latest upstream changes (presumably c3d5eda) made this pull request unmergeable. Please resolve the merge conflicts. |
c192289
to
a076c7f
Compare
Have we figured out if this is still necessary? |
I think that PR #579 fixes the root problem in a better way. At least, it does for now. |
But it didn't change anything in CI, so I guess we're still trying to figure out the root of those errors, yes? |
Correct. I don't know why those tests are still failing in CI. |
Then let's close this PR in favor of a separate issue to figure that out. |
Fixes #529, also refer to
#529 which first added this
for the old CI infrastructure.