mirror of
https://chromium.googlesource.com/crosvm/crosvm
synced 2025-02-06 10:32:10 +00:00
235d76e9d2
This helps with integration of `tools/clippy` in `rust-analyzer` and potentially other IDEs which can parse the json output better than the text output. TEST=Ran it with `rust-analyzer` in VSCode. Change-Id: Iceb2748994cbbacf48dc473a9ad06eec567beb0a Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3630700 Reviewed-by: Dennis Kempin <denniskempin@google.com> Commit-Queue: Vaibhav Nagarnaik <vnagarnaik@google.com> Tested-by: kokoro <noreply+kokoro@google.com> Reviewed-by: Daniel Verkamp <dverkamp@chromium.org>
66 lines
1.8 KiB
Python
Executable file
66 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, json: bool = False):
|
|
chdir(CROSVM_ROOT)
|
|
|
|
# Note: Clippy checks are configured in .cargo/config.toml
|
|
common_args = [
|
|
"--fix" if fix else None,
|
|
"--message-format=json" if json 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)
|