From 0b3b75ebc2e89ed7055b3153a198cb29be2b8c71 Mon Sep 17 00:00:00 2001 From: George Burgess IV Date: Wed, 8 Apr 2020 14:59:40 -0700 Subject: compiler_wrapper: assume /dev/null is a configure test Linux does a lot of "is this flag supported" testing by passing /dev/null as an input file. Since this code is pretty heuristic-y anyway, let's see if we can get by with calling that a `configure`-like file. :) BUG=None TEST=emerge-eve chromeos-kernel-4_4; no more fatal warnings Change-Id: Ie1cd80b36e93a9269280345a1df006baa356a9f6 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2142404 Reviewed-by: Manoj Gupta Tested-by: George Burgess --- compiler_wrapper/disable_werror_flag_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'compiler_wrapper/disable_werror_flag_test.go') diff --git a/compiler_wrapper/disable_werror_flag_test.go b/compiler_wrapper/disable_werror_flag_test.go index 9d762599..53b0fb52 100644 --- a/compiler_wrapper/disable_werror_flag_test.go +++ b/compiler_wrapper/disable_werror_flag_test.go @@ -73,7 +73,7 @@ func TestDoubleBuildWithWNoErrorFlag(t *testing.T) { func TestKnownConfigureFileParsing(t *testing.T) { withTestContext(t, func(ctx *testContext) { - for _, f := range []string{"conftest.c", "conftest.cpp"} { + for _, f := range []string{"conftest.c", "conftest.cpp", "/dev/null"} { if !isLikelyAConfTest(ctx.cfg, ctx.newCommand(clangX86_64, f)) { t.Errorf("%q isn't considered a conf test file", f) } -- cgit v1.2.3