diff options
| author | George Burgess IV <gbiv@google.com> | 2020-04-08 14:59:40 -0700 |
|---|---|---|
| committer | George Burgess <gbiv@chromium.org> | 2020-04-08 22:17:04 +0000 |
| commit | 0b3b75ebc2e89ed7055b3153a198cb29be2b8c71 (patch) | |
| tree | 53e2f6c1def6a42b6635fb1dec2e4eb0cb8a5026 /compiler_wrapper/disable_werror_flag_test.go | |
| parent | 505ab9530dc7a3a1d84b351f9ce682b4e8e33bc6 (diff) | |
| download | toolchain-utils-0b3b75ebc2e89ed7055b3153a198cb29be2b8c71.tar.gz | |
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 <manojgupta@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Diffstat (limited to 'compiler_wrapper/disable_werror_flag_test.go')
| -rw-r--r-- | compiler_wrapper/disable_werror_flag_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
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) } |