From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 06744A0C43; Wed, 1 Sep 2021 14:31:35 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8A9F540140; Wed, 1 Sep 2021 14:31:34 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mails.dpdk.org (Postfix) with ESMTP id 92E4F4013F for ; Wed, 1 Sep 2021 14:31:33 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1630499492; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=xiIbaoL18KZSKFjGS6SK3asITlkuf7pdqFJDnulM1xM=; b=Ksb0POpMAgiCWLQcjmtuQVnxJqpn0K9lBOBPwLWuwWb7g5o3jE6/mCNUc5N6LruUXeT1f1 pwyW4mJwolz0DybkI8vebQBuTmWza5ie8WtsE9kCDQPio5hLp54ovePcgEHUHxfC+KLzGJ WAGWNt+8iZO2vsZXftW5BP+uqtO6Sd8= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-547-MSZ9a6M3OOGmTdUhYWfPOw-1; Wed, 01 Sep 2021 08:31:31 -0400 X-MC-Unique: MSZ9a6M3OOGmTdUhYWfPOw-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 3C32C107ACE3; Wed, 1 Sep 2021 12:31:30 +0000 (UTC) Received: from RHTPC1VM0NT (unknown [10.22.17.187]) by smtp.corp.redhat.com (Postfix) with ESMTPS id BB6E219C79; Wed, 1 Sep 2021 12:31:28 +0000 (UTC) From: Aaron Conole To: Ray Kinsella Cc: dev@dpdk.org, bruce.richardson@intel.com, stephen@networkplumber.org, ferruh.yigit@intel.com, thomas@monjalon.net, ktraynor@redhat.com References: <20210618163659.85933-1-mdr@ashroe.eu> <20210831145017.856776-1-mdr@ashroe.eu> Date: Wed, 01 Sep 2021 08:31:27 -0400 In-Reply-To: <20210831145017.856776-1-mdr@ashroe.eu> (Ray Kinsella's message of "Tue, 31 Aug 2021 15:50:14 +0100") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=aconole@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain Subject: Re: [dpdk-dev] [PATCH v10 0/3] devtools: scripts to count and track symbols X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Ray Kinsella writes: > Scripts to count and track the lifecycle of DPDK symbols. > > The symbol-tool script reports on the growth of symbols over releases > and list expired symbols. The notify-symbol-maintainers script > consumes the input from symbol-tool and generates email notifications > of expired symbols. > > v2: reworked to fix pylint errors > v3: sent with the correct in-reply-to > v4: fix typos picked up by the CI > v5: fix terminal_size & directory args > v6: added list-expired, to list expired experimental symbols > v7: fix typo in comments > v8: added tool to notify maintainers of expired symbols > v9: removed hardcoded emails addressed and script names > v10: added ability to identify and notify the original contributors > > Ray Kinsella (3): > devtools: script to track symbols over releases > devtools: script to send notifications of expired symbols > maintainers: add new abi scripts > > MAINTAINERS | 2 + > devtools/notify-symbol-maintainers.py | 256 ++++++++++++++ > devtools/symbol-tool.py | 482 ++++++++++++++++++++++++++ > 3 files changed, 740 insertions(+) > create mode 100755 devtools/notify-symbol-maintainers.py > create mode 100755 devtools/symbol-tool.py I get a whole mess of flake8 issues from this series (mostly 'backslash is redundant' and whitespace issues). I'm using flake8 because it pretty well enforces PEP8 style guide. I would like to see it addressed, but also I see that many of the python files in the DPDK tree don't actually pass. Example:: $ flake8 ./usertools/dpdk-devbind.py | wc -l 34 $ flake8 ./usertools/dpdk-devbind.py | sed 's@./usertools/dpdk-devbind.py[:0-9]* @@' | sort -u E128 continuation line under-indented for visual indent E302 expected 2 blank lines, found 1 E305 expected 2 blank lines after class or function definition, found 1 E501 line too long (105 > 79 characters) E501 line too long (80 > 79 characters) E501 line too long (82 > 79 characters) E501 line too long (83 > 79 characters) E501 line too long (84 > 79 characters) E501 line too long (85 > 79 characters) E501 line too long (86 > 79 characters) E501 line too long (91 > 79 characters) E502 the backslash is redundant between brackets E722 do not use bare 'except' Looks like we repeat the same kinds of errors everywhere (this is on multiple tools). Some of our in-tree python is better than others (like app/test/autotest.py which only has 1 flake). Maybe we can address this. Other comments inline on the patches.