crosvm/tools/clippy
Noah Gold 35290a48c0 cros_async: enable tests on Windows.
BUG=b:213147081
TEST=bots

Change-Id: I88ac2eb0c4c9b4f3f77e2b3c0cc994979a14d14c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3585009
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Dennis Kempin <denniskempin@google.com>
Commit-Queue: Noah Gold <nkgold@google.com>
2022-04-20 18:56:54 +00:00

65 lines
1.8 KiB
Python
Executable file

#!/usr/bin/env python3
# Copyright 2019 The Chromium OS Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
# To check for violations:
# $ ./tools/clippy
#
# To fix violations where possible:
# $ ./tools/clippy --fix
import os
from impl.common import CROSVM_ROOT, cwd, run_main, cmd, chdir
from impl.test_runner import get_workspace_excludes
clippy = cmd("cargo clippy")
excluded_crates: list[str] = []
features: str = ""
if os.name == "posix":
features = "--features=all-linux"
elif os.name == "nt":
excluded_crates = list(get_workspace_excludes("win64"))
else:
raise Exception(f"Unsupported build target: {os.name}")
def is_crate_excluded(crate: str) -> bool:
return crate in excluded_crates
def main(fix: bool = False):
chdir(CROSVM_ROOT)
# Note: Clippy checks are configured in .cargo/config.toml
common_args = [
"--fix" if fix else None,
"--all-targets",
"--",
"-Dwarnings",
]
print("Clippy crosvm workspace")
clippy(
"--workspace",
features,
*[f"--exclude={crate}" for crate in excluded_crates],
*common_args,
).fg()
for crate in CROSVM_ROOT.glob("common/*/Cargo.toml"):
# TODO(b/213147081): remove the cros_async exclude this once
# common/cros_async is gone.
if is_crate_excluded(crate.parent.name) or (
os.name == "nt" and crate.parent.name == "cros_async"
):
print("Skipping crate", crate.parent.relative_to(CROSVM_ROOT))
else:
print("Clippy", crate.parent.relative_to(CROSVM_ROOT))
with cwd(crate.parent):
clippy("--all-features", *common_args).fg()
if __name__ == "__main__":
run_main(main)