From 91de25fd88ed755be5d72db80c93da364d97e16f Mon Sep 17 00:00:00 2001 From: Alex Light Date: Wed, 28 Oct 2015 17:00:06 -0700 Subject: Make things generally quieter. * Give run-test a --quiet flag that causes it to only print on errors. * Give cpplint a --quiet flag that causes it to not print anything when there are no errors. * Add a ART_TEST_QUIET flag to build/Android.common_test.mk which makes run-test targets much quieter when true (the default). With this flag only failures will be printed. * Make build/Android.cpplint.mk pass the new cpplint --quiet flag so that only failures will be printed. Before: [ 96% 5715/5906] build test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-461-get-reference-vreg32 test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-448-multiple-returns32 RUNNING /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: building... /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: running... /usr/local/google/buildbot/src/googleplex-android/master-art-host/art/test/448-multiple-returns: succeeded! test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-448-multiple-returns32 PASSED After: [ 96% 5715/5906] build test-art-host-run-test-debug-prebuild-interpreter-relocate-ntrace-cms-checkjni-image-npictest-ndebuggable-461-get-reference-vreg32 Change-Id: Idf6fce7f48a619f83254b48861dbd7f8eb4ebdbf --- build/Android.cpplint.mk | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'build/Android.cpplint.mk') diff --git a/build/Android.cpplint.mk b/build/Android.cpplint.mk index 79f8f5eeaf..953cfc091c 100644 --- a/build/Android.cpplint.mk +++ b/build/Android.cpplint.mk @@ -18,6 +18,7 @@ include art/build/Android.common_build.mk ART_CPPLINT := art/tools/cpplint.py ART_CPPLINT_FILTER := --filter=-whitespace/line_length,-build/include,-readability/function,-readability/streams,-readability/todo,-runtime/references,-runtime/sizeof,-runtime/threadsafe_fn,-runtime/printf +ART_CPPLINT_FLAGS := --quiet ART_CPPLINT_SRC := $(shell find art -name "*.h" -o -name "*$(ART_CPP_EXTENSION)" | grep -v art/compiler/llvm/generated/ | grep -v art/runtime/elf\.h) # "mm cpplint-art" to verify we aren't regressing @@ -39,8 +40,8 @@ art_cpplint_file := $(1) art_cpplint_touch := $$(OUT_CPPLINT)/$$(subst /,__,$$(art_cpplint_file)) $$(art_cpplint_touch): $$(art_cpplint_file) $(ART_CPPLINT) art/build/Android.cpplint.mk - $(hide) $(ART_CPPLINT) $(ART_CPPLINT_FILTER) $$< - @mkdir -p $$(dir $$@) + $(hide) $(ART_CPPLINT) $(ART_CPPLINT_FLAGS) $(ART_CPPLINT_FILTER) $$< + $(hide) mkdir -p $$(dir $$@) $(hide) touch $$@ ART_CPPLINT_TARGETS += $$(art_cpplint_touch) -- cgit v1.2.3-59-g8ed1b