From ae911cb8ae3d07cd09740b55154eb27e8060d7b6 Mon Sep 17 00:00:00 2001 From: Siarhei Vishniakou Date: Mon, 26 Mar 2018 16:18:34 -0700 Subject: [PATCH] Suppress key validation logs If the validation of keylayouts / keycharmaps / keyconfigs succeeds, do not log anything to reduce logspam. The "-q" argument was misplaced in the original CL. Bug: 35672363 Test: touch frameworks/base/data/keyboards/Vendor_045e_Product_028e.kl && m Also, change the behaviour in tools/validatekeymaps/Main.cpp to always ignore "-q", see that the logs come back. Change-Id: I3cd2ef161942905c2d8ffce8fc93dd345868066d --- data/keyboards/Android.mk | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/data/keyboards/Android.mk b/data/keyboards/Android.mk index 2daef0cb0cf41..7949c77ab5582 100644 --- a/data/keyboards/Android.mk +++ b/data/keyboards/Android.mk @@ -28,7 +28,7 @@ LOCAL_BUILT_MODULE := $(intermediates)/stamp validatekeymaps := $(HOST_OUT_EXECUTABLES)/validatekeymaps$(HOST_EXECUTABLE_SUFFIX) $(LOCAL_BUILT_MODULE): PRIVATE_VALIDATEKEYMAPS := $(validatekeymaps) $(LOCAL_BUILT_MODULE) : $(framework_keylayouts) $(framework_keycharmaps) $(framework_keyconfigs) | $(validatekeymaps) - $(hide) -q $(PRIVATE_VALIDATEKEYMAPS) $^ + $(hide) $(PRIVATE_VALIDATEKEYMAPS) -q $^ $(hide) mkdir -p $(dir $@) && touch $@ # Run validatekeymaps uncondionally for platform build.