Merge #37

37: Add CI check for redox OS r=lzutao a=lzutao

We actively support redox OS, so it would be nice to have a CI job for the target.

Co-authored-by: Lzu Tao <taolzu@gmail.com>

.github/workflows/main.yml

@@ -26,6 +26,17 @@ jobs:
       - run: cargo build --all-targets --workspace
       - run: cargo test --all-targets --workspace
 
+  redox_check:
+    runs-on: ubuntu-latest
+    env:
+      RUST_BACKTRACE: 1
+    steps:
+      - uses: actions/checkout@v2
+      - run: |
+          rustup default nightly
+          rustup target add x86_64-unknown-redox
+      - run: cargo check --all-targets --workspace --target x86_64-unknown-redox
+
   rustfmt:
     runs-on: ubuntu-latest
     steps:
@@ -34,24 +45,42 @@ jobs:
       - run: cargo fmt -- --check
 
   minimum-rust-version:
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
       - run: |
           rustup toolchain install 1.34.0
           rustup default 1.34.0
-      - run: cargo build --all-targets --all
+          rustup target add \
+            x86_64-pc-windows-gnu \
+            x86_64-apple-darwin \
+            x86_64-pc-windows-msvc
+      - run: |
+          for target in x86_64-pc-windows-gnu x86_64-apple-darwin x86_64-pc-windows-msvc; do
+            echo "== checking $target =="
+            cargo check --all-targets --all --target "$target"
+          done
+        shell: bash
 
   clippy:
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        os: [ubuntu-latest, macos-latest, windows-latest]
+    runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - run: rustup component add clippy
-        # FIXME: Remove `-- -A clippy::option_as_ref_deref` when MSRV is bumped past 1.40.0
-      - run: cargo clippy --all --all-targets -- -A clippy::option_as_ref_deref
+      - run: |
+          rustup component add clippy
+          rustup target add \
+            x86_64-pc-windows-gnu \
+            x86_64-apple-darwin \
+            x86_64-pc-windows-msvc
+      - run: |
+          for target in x86_64-pc-windows-gnu x86_64-apple-darwin x86_64-pc-windows-msvc; do
+            echo "== checking $target =="
+            cargo check --all-targets --all --target "$target"
+            git ls-files '*lib.rs' | xargs touch
+            # FIXME: Remove `-- -A clippy::option_as_ref_deref` when MSRV is bumped past 1.40.0
+            cargo clippy --all-targets --all --target "$target" \
+              -- \
+              -A clippy::option_as_ref_deref \
+              -A clippy::redundant_field_names
+          done
+        shell: bash

dirs-sys/src/lib.rs

@@ -68,7 +68,7 @@ mod target_redox {
 
     use std::path::PathBuf;
 
-    use super::redox_users::{All, AllUsers, Config};
+    use redox_users::{All, AllUsers, Config};
 
     pub fn home_dir() -> Option<PathBuf> {
         let current_uid = redox_users::get_uid().ok()?;