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

Multiple CI improvements #1497

Merged
merged 2 commits into from
Aug 22, 2021
Merged

Multiple CI improvements #1497

merged 2 commits into from
Aug 22, 2021

Conversation

asomers
Copy link
Member

@asomers asomers commented Aug 22, 2021

  • Install cross the easy way, via cargo
  • Don't test in release mode. Nix contains no release-dependent paths,
    and release mode testing has to my knowledge never revealed a bug in
    Nix.
  • Enable testing on several platforms now supported by cross:
    • Android, all arches
    • iOS, both arches
    • arm-unknown-linux-musleabi, powerpc64-unknown-linux-gnu
    • Add Linux powerpc back to CI, fixed by the latest cross
  • Check the tests even on platforms that can't run them.
  • DRY for the Illumos and Redox sections

Also, split the overbroad test_mknod_family into two tests
* Install cross the easy way, via cargo
* Don't test in release mode.  Nix contains no release-dependent paths,
  and release mode testing has to my knowledge never revealed a bug in
  Nix.
* Add Linux powerpc back to CI, fixed by the latest cross.
* Check the tests even on platforms that can't run them.
* DRY for the Illumos and Redox sections
* Cross-check iOS from a Linux VM instead of OSX
* Revert the workaround for rust-lang/rustup issue 2774
@asomers asomers merged commit dab7332 into nix-rust:master Aug 22, 2021
@asomers asomers deleted the cirrus2 branch August 22, 2021 16:47
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.

1 participant