From dbe4c85400b5a4b67538e03b88b1f69e0161c54a Mon Sep 17 00:00:00 2001 From: Ana Krivokapic Date: Thu, 19 Oct 2017 15:35:56 +0200 Subject: [PATCH] Explicitly set default to 0 for quiet argument in validate-files script Due to a bug in python3 which sets default to None for 'count' actions, and the fact that Fedora now uses python3 as default, running the pep8 portion of the tests would fail on Fedora. Change-Id: Ib570c3f8a21dc0a2af2373f3481f48623f05d852 --- tools/validate-files.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/validate-files.py b/tools/validate-files.py index 16545aeff..c8e7a90ef 100755 --- a/tools/validate-files.py +++ b/tools/validate-files.py @@ -37,6 +37,9 @@ def parse_args(): p.add_argument('--quiet', '-q', action='count', + # TODO(akrivoka): Python3 sets this default to None instead + # of 0. Remove this when this bug is fixed in Python3. + default=0, help='output warnings and errors (-q) or only errors (-qq)') p.add_argument('path_args',