Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 700b4d3e authored by Sam Delmerico's avatar Sam Delmerico
Browse files

pass TIDY_EXTERNAL_VENDOR envvar to Bazel 

Bug: 268668248
Test: verify that Bazel tidy actions don't have a large diff from Soong tidy actions:
  (add -v to Ninja invocation in ui/build/ninja.go [and disable fifo file])
  WITH_TIDY=1 DISABLE_ARTIFACT_PATH_REQUIREMENTS=true m com.android.adbd 2>&1 | tee soong_build-tidy.adbd &&
  WITH_TIDY=1 DISABLE_ARTIFACT_PATH_REQUIREMENTS=true m com.android.adbd --bazel-mode-staging 2>&1 | tee mixed_build-tidy.adbd &&
  grep -Po "prebuilts/clang/host/linux-x86/clang-r475365b/bin/clang-tidy.sh \K\S+" soong_build-tidy.adbd | sort -u > soong-clang-tidy.adbd &&
  grep -Po "prebuilts/clang/host/linux-x86/clang-r475365b/bin/clang-tidy.sh \K\S+" mixed_build-tidy.adbd | sort -u > mixed-clang-tidy.adbd &&
  vimdiff soong_build-tidy.adbd mixed_build-tidy.adbd
Change-Id: I22804d6d92ed56d9727eaf430318df0b041712a8
parent 4ed95e26

android/bazel_handler.go

+3 −0
Original line number Diff line number Diff line
@@ -45,11 +45,14 @@ var ( 
		CommandDeps: []string{"${bazelBuildRunfilesTool}"},

	}, "outDir")

	allowedBazelEnvironmentVars = []string{

		// clang-tidy

		"ALLOW_LOCAL_TIDY_TRUE",

		"DEFAULT_TIDY_HEADER_DIRS",

		"TIDY_TIMEOUT",

		"WITH_TIDY",

		"WITH_TIDY_FLAGS",

		"TIDY_EXTERNAL_VENDOR",



		"SKIP_ABI_CHECKS",

		"UNSAFE_DISABLE_APEX_ALLOWED_DEPS_CHECK",

		"AUTO_ZERO_INITIALIZE",