Skip to content

Commit

Permalink
Drop support for cloudabi targets
Browse files Browse the repository at this point in the history
  • Loading branch information
tesuji authored and Mark-Simulacrum committed Nov 22, 2020
1 parent 113c147 commit 4b698f2
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 34 deletions.
2 changes: 1 addition & 1 deletion clippy_lints/src/attrs.rs
Expand Up @@ -40,7 +40,7 @@ static UNIX_SYSTEMS: &[&str] = &[
];

// NOTE: windows is excluded from the list because it's also a valid target family.
static NON_UNIX_SYSTEMS: &[&str] = &["cloudabi", "hermit", "none", "wasi"];
static NON_UNIX_SYSTEMS: &[&str] = &["hermit", "none", "wasi"];

declare_clippy_lint! {
/// **What it does:** Checks for items annotated with `#[inline(always)]`,
Expand Down
5 changes: 1 addition & 4 deletions tests/ui/mismatched_target_os_non_unix.fixed
Expand Up @@ -3,9 +3,6 @@
#![warn(clippy::mismatched_target_os)]
#![allow(unused)]

#[cfg(target_os = "cloudabi")]
fn cloudabi() {}

#[cfg(target_os = "hermit")]
fn hermit() {}

Expand All @@ -16,7 +13,7 @@ fn wasi() {}
fn none() {}

// list with conditions
#[cfg(all(not(any(windows, target_os = "cloudabi")), target_os = "wasi"))]
#[cfg(all(not(windows), target_os = "wasi"))]
fn list() {}

// windows is a valid target family, should be ignored
Expand Down
5 changes: 1 addition & 4 deletions tests/ui/mismatched_target_os_non_unix.rs
Expand Up @@ -3,9 +3,6 @@
#![warn(clippy::mismatched_target_os)]
#![allow(unused)]

#[cfg(cloudabi)]
fn cloudabi() {}

#[cfg(hermit)]
fn hermit() {}

Expand All @@ -16,7 +13,7 @@ fn wasi() {}
fn none() {}

// list with conditions
#[cfg(all(not(any(windows, cloudabi)), wasi))]
#[cfg(all(not(windows), wasi))]
fn list() {}

// windows is a valid target family, should be ignored
Expand Down
35 changes: 10 additions & 25 deletions tests/ui/mismatched_target_os_non_unix.stderr
@@ -1,51 +1,36 @@
error: operating system used in target family position
--> $DIR/mismatched_target_os_non_unix.rs:6:1
|
LL | #[cfg(cloudabi)]
| ^^^^^^--------^^
| |
| help: try: `target_os = "cloudabi"`
|
= note: `-D clippy::mismatched-target-os` implied by `-D warnings`

error: operating system used in target family position
--> $DIR/mismatched_target_os_non_unix.rs:9:1
|
LL | #[cfg(hermit)]
| ^^^^^^------^^
| |
| help: try: `target_os = "hermit"`
|
= note: `-D clippy::mismatched-target-os` implied by `-D warnings`

error: operating system used in target family position
--> $DIR/mismatched_target_os_non_unix.rs:12:1
--> $DIR/mismatched_target_os_non_unix.rs:9:1
|
LL | #[cfg(wasi)]
| ^^^^^^----^^
| |
| help: try: `target_os = "wasi"`

error: operating system used in target family position
--> $DIR/mismatched_target_os_non_unix.rs:15:1
--> $DIR/mismatched_target_os_non_unix.rs:12:1
|
LL | #[cfg(none)]
| ^^^^^^----^^
| |
| help: try: `target_os = "none"`

error: operating system used in target family position
--> $DIR/mismatched_target_os_non_unix.rs:19:1
|
LL | #[cfg(all(not(any(windows, cloudabi)), wasi))]
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
help: try
|
LL | #[cfg(all(not(any(windows, target_os = "cloudabi")), wasi))]
| ^^^^^^^^^^^^^^^^^^^^^^
help: try
--> $DIR/mismatched_target_os_non_unix.rs:16:1
|
LL | #[cfg(all(not(any(windows, cloudabi)), target_os = "wasi"))]
| ^^^^^^^^^^^^^^^^^^
LL | #[cfg(all(not(windows), wasi))]
| ^^^^^^^^^^^^^^^^^^^^^^^^----^^^
| |
| help: try: `target_os = "wasi"`

error: aborting due to 5 previous errors
error: aborting due to 4 previous errors

0 comments on commit 4b698f2

Please sign in to comment.