mirrors/crosvm
1
0
Fork 0
mirror of https://chromium.googlesource.com/crosvm/crosvm synced 2025-02-06 02:25:23 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions
crosvm/.cargo/config.toml

55 lines
1.8 KiB
TOML
Raw Normal View History

tools/clippy: Move allow-list to .cargo/config.toml This applies the same clippy config to rust_analyzer as well. BUG=b:193893457 TEST=./tools/clippy and VSCode show the same linter results Change-Id: I5deec4a24d71fec4692ed708664130f46c1311b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3278773 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2021-11-12 17:35:42 +00:00
# Disable clippy lints project-wide.
# This allows ./tools/clippy and IDE integrations to use the same configuration.
# This should be replaced with a proper clippy config once available:
# https://github.com/rust-lang/cargo/issues/5034
[target.'cfg(all())']
rustflags = [
# TODO(b/181763000): Fail builds on warnings
# "-Dwarnings",
# TODO(crbug/908640): To be resolved.
Uprev rust toolchain to 1.56.1 Two new clippy fixes are affecting us, but will be easy to fix in a follow-up. Fixed the dev-container so we can create a new version with the new toolchain. BUG=b:210037151 TEST=Kokoro Change-Id: I9ac4d84aff72b1ee5219d6dab0a88667ca6c5951 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3328954 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2021-12-09 23:58:38 +00:00
"-Aclippy::needless_return",
"-Aclippy::needless_doctest_main",
"-Aclippy::blocks_in_if_conditions",
tools/clippy: Move allow-list to .cargo/config.toml This applies the same clippy config to rust_analyzer as well. BUG=b:193893457 TEST=./tools/clippy and VSCode show the same linter results Change-Id: I5deec4a24d71fec4692ed708664130f46c1311b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3278773 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2021-11-12 17:35:42 +00:00
"-Aclippy::missing_safety_doc", # 26 errors
# We don't care about these lints. Okay to remain suppressed globally.
clippy: fix devices crate Fixes clippy failures and enables clippy checks on the crate. BUG=b:192373803 TEST=./tools/presubmit Change-Id: I9022d40e997f94c03d3f936734116e3ea0505022 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3441348 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Reviewed-by: Noah Gold <nkgold@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2022-02-04 23:05:15 +00:00
"-Aclippy::bool_assert_comparison",
tools/clippy: Move allow-list to .cargo/config.toml This applies the same clippy config to rust_analyzer as well. BUG=b:193893457 TEST=./tools/clippy and VSCode show the same linter results Change-Id: I5deec4a24d71fec4692ed708664130f46c1311b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3278773 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2021-11-12 17:35:42 +00:00
"-Aclippy::cast_lossless",
"-Aclippy::cognitive_complexity",
clippy: fix devices crate Fixes clippy failures and enables clippy checks on the crate. BUG=b:192373803 TEST=./tools/presubmit Change-Id: I9022d40e997f94c03d3f936734116e3ea0505022 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3441348 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Reviewed-by: Noah Gold <nkgold@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2022-02-04 23:05:15 +00:00
"-Aclippy::collapsible_if",
tools/clippy: Move allow-list to .cargo/config.toml This applies the same clippy config to rust_analyzer as well. BUG=b:193893457 TEST=./tools/clippy and VSCode show the same linter results Change-Id: I5deec4a24d71fec4692ed708664130f46c1311b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3278773 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2021-11-12 17:35:42 +00:00
"-Aclippy::enum_variant_names",
"-Aclippy::identity_op",
"-Aclippy::len_without_is_empty",
"-Aclippy::len_zero",
"-Aclippy::match_bool",
"-Aclippy::match_wild_err_arm",
"-Aclippy::module_inception",
"-Aclippy::needless_bool",
"-Aclippy::new_without_default",
clippy: fix devices crate Fixes clippy failures and enables clippy checks on the crate. BUG=b:192373803 TEST=./tools/presubmit Change-Id: I9022d40e997f94c03d3f936734116e3ea0505022 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3441348 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Reviewed-by: Noah Gold <nkgold@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2022-02-04 23:05:15 +00:00
"-Aclippy::new-ret-no-self",
tools/clippy: Move allow-list to .cargo/config.toml This applies the same clippy config to rust_analyzer as well. BUG=b:193893457 TEST=./tools/clippy and VSCode show the same linter results Change-Id: I5deec4a24d71fec4692ed708664130f46c1311b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3278773 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2021-11-12 17:35:42 +00:00
"-Aclippy::or_fun_call",
clippy: fix devices crate Fixes clippy failures and enables clippy checks on the crate. BUG=b:192373803 TEST=./tools/presubmit Change-Id: I9022d40e997f94c03d3f936734116e3ea0505022 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3441348 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Reviewed-by: Noah Gold <nkgold@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2022-02-04 23:05:15 +00:00
"-Aclippy::result-unit-err",
tools/clippy: Move allow-list to .cargo/config.toml This applies the same clippy config to rust_analyzer as well. BUG=b:193893457 TEST=./tools/clippy and VSCode show the same linter results Change-Id: I5deec4a24d71fec4692ed708664130f46c1311b0 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3278773 Reviewed-by: Daniel Verkamp <dverkamp@chromium.org> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Dennis Kempin <denniskempin@google.com>
2021-11-12 17:35:42 +00:00
"-Aclippy::should_implement_trait",
"-Aclippy::single_char_pattern",
"-Aclippy::too_many_arguments",
"-Aclippy::trivially_copy_pass_by_ref",
"-Aclippy::type_complexity",
"-Aclippy::unreadable_literal",
"-Aclippy::useless_let_if_seq",
"-Aclippy::useless_transmute",
]
Enable some tests for armhf builds TEST=./tools/dev_container --hermetic bash -c "./tools/run_tests --target=vm:aarch64 --arch armhf" BUG=b/203152778 Change-Id: I76da1d029e9b11016b1ed9245c5b09095703fb63 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3328099 Reviewed-by: Dennis Kempin <denniskempin@google.com> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Anton Romanov <romanton@google.com>
2021-12-09 22:16:59 +00:00
[target.armv7-unknown-linux-gnueabihf]
linker = "arm-linux-gnueabihf-gcc"
Run some win64 tests with wine as part of presubmit docker container size increases to ~4.7-8 Gb TEST=./dev_container --hermetic ./tools/presubmit BUG=none Change-Id: I49dc03182f6ac5c29e0174618cc3864e073a2eb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3499264 Reviewed-by: Dennis Kempin <denniskempin@google.com> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Anton Romanov <romanton@google.com> Auto-Submit: Anton Romanov <romanton@google.com>
2022-03-03 01:29:26 +00:00
crosvm: Fix cross-compile in dev container - Fix devcontainer.json to be used by vscode correctly - Fix wayland-protocols path finding - Add missing SYSROOT env for pkg_config - Add missing linker settings TEST=native/armv7/aarch64 targets compile in dev container Change-Id: Id6fc43b2fd63d09bc46169d996387d5417ada93e Reviewed-on: https://chromium-review.googlesource.com/c/crosvm/crosvm/+/3766822 Commit-Queue: Dennis Kempin <denniskempin@google.com> Tested-by: Dennis Kempin <denniskempin@google.com> Auto-Submit: Zihan Chen <zihanchen@google.com> Reviewed-by: Dennis Kempin <denniskempin@google.com>
2022-07-19 01:07:53 +00:00
[target.aarch64-unknown-linux-gnu]
linker = "aarch64-linux-gnu-gcc"
Run some win64 tests with wine as part of presubmit docker container size increases to ~4.7-8 Gb TEST=./dev_container --hermetic ./tools/presubmit BUG=none Change-Id: I49dc03182f6ac5c29e0174618cc3864e073a2eb8 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/3499264 Reviewed-by: Dennis Kempin <denniskempin@google.com> Tested-by: kokoro <noreply+kokoro@google.com> Commit-Queue: Anton Romanov <romanton@google.com> Auto-Submit: Anton Romanov <romanton@google.com>
2022-03-03 01:29:26 +00:00
[target.x86_64-pc-windows-gnu]
runner = "wine64"
crosvm: Fix cross-compile in dev container - Fix devcontainer.json to be used by vscode correctly - Fix wayland-protocols path finding - Add missing SYSROOT env for pkg_config - Add missing linker settings TEST=native/armv7/aarch64 targets compile in dev container Change-Id: Id6fc43b2fd63d09bc46169d996387d5417ada93e Reviewed-on: https://chromium-review.googlesource.com/c/crosvm/crosvm/+/3766822 Commit-Queue: Dennis Kempin <denniskempin@google.com> Tested-by: Dennis Kempin <denniskempin@google.com> Auto-Submit: Zihan Chen <zihanchen@google.com> Reviewed-by: Dennis Kempin <denniskempin@google.com>
2022-07-19 01:07:53 +00:00
[env]
PKG_CONFIG_SYSROOT_DIR_armv7-unknown-linux-gnueabihf = "/usr/arm-linux-gnueabihf"
PKG_CONFIG_SYSROOT_DIR_aarch64-unknown-linux-gnu = "/usr/aarch64_linux_gnu"
PKG_CONFIG_SYSROOT_DIR_x86_64-pc-windows-gnu = "/usr/x86_64-w64-mingw32"
Reference in a new issue Copy permalink