"Unused import" warnings on Windows
In our reproducible-build script, as it's cross compiling for windows, we see:
Compiling termcolor v1.2.0
Compiling tor-rtmock v0.7.2 (/arti/crates/tor-rtmock)
Compiling clap_lex v0.2.4
Compiling tracing-subscriber v0.3.16
Compiling atty v0.2.14
Compiling windows v0.43.0
Compiling strsim v0.10.0
Compiling data-encoding v2.3.3
Compiling tor-cell v0.9.1 (/arti/crates/tor-cell)
Compiling tor-netdir v0.8.1 (/arti/crates/tor-netdir)
Compiling textwrap v0.16.0
Compiling hostname-validator v1.1.1
Compiling ipnet v2.7.2
Compiling clap v3.2.23
Compiling notify v5.1.0
Compiling rlimit v0.9.1
Compiling trust-dns-proto v0.22.0
Compiling tor-proto v0.9.1 (/arti/crates/tor-proto)
Compiling tracing-appender v0.2.2
Compiling tor-chanmgr v0.8.3 (/arti/crates/tor-chanmgr)
Compiling tor-guardmgr v0.8.3 (/arti/crates/tor-guardmgr)
Compiling tor-ptmgr v0.2.3 (/arti/crates/tor-ptmgr)
Compiling tor-circmgr v0.7.3 (/arti/crates/tor-circmgr)
Compiling xz2 v0.1.7
Compiling zstd-safe v5.0.2+zstd.1.5.2
Compiling zstd v0.11.2+zstd.1.5.2
Compiling async-compression v0.3.15
Compiling tor-dirclient v0.6.3 (/arti/crates/tor-dirclient)
Compiling secmem-proc v0.3.0
Compiling rusqlite v0.29.0
Compiling tor-dirmgr v0.9.3 (/arti/crates/tor-dirmgr)
Compiling arti-client v0.8.3 (/arti/crates/arti-client)
Compiling arti v1.1.3 (/arti/crates/arti)
warning: unused import: `futures::task::SpawnExt`
--> crates/arti/src/reload_cfg.rs:16:5
|
16 | use futures::task::SpawnExt;
| ^^^^^^^^^^^^^^^^^^^^^^^
|
= note: `#[warn(unused_imports)]` on by default
warning: unused import: `crate::process::sighup_stream`
--> crates/arti/src/reload_cfg.rs:18:5
|
18 | use crate::process::sighup_stream;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
warning: function `sighup_stream` is never used
--> crates/arti/src/process.rs:80:15
|
80 | pub(crate) fn sighup_stream() -> crate::Result<impl futures::Stream<Item = ()>> {
| ^^^^^^^^^^^^^
|
= note: `#[warn(dead_code)]` on by default
warning: variant `SigHup` is never constructed
--> crates/arti/src/reload_cfg.rs:28:5
|
26 | enum Event {
| ----- variant in this enum
27 | /// SIGHUP has been received.
28 | SigHup,
| ^^^^^^
|
= note: `Event` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
warning: `arti` (lib) generated 4 warning
It would be nice if this would build without warnings everywhere.