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
posborne
force-pushed
the
new-ci/remove-parallelism
branch
from
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 <osbpau@gmail.com>
posborne
force-pushed
the
new-ci/remove-parallelism
branch
from
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. |
Susurrus
force-pushed
the
new-ci/master
branch
3 times, most recently
from
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.