From patchwork Mon Jun 26 10:24:03 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Thomas Monjalon X-Patchwork-Id: 128992 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 1593542D59; Mon, 26 Jun 2023 12:24:12 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 03A5242D33; Mon, 26 Jun 2023 12:24:12 +0200 (CEST) Received: from out5-smtp.messagingengine.com (out5-smtp.messagingengine.com [66.111.4.29]) by mails.dpdk.org (Postfix) with ESMTP id 42E1341149; Mon, 26 Jun 2023 12:24:11 +0200 (CEST) Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id E5FCF5C00F8; Mon, 26 Jun 2023 06:24:10 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute6.internal (MEProxy); Mon, 26 Jun 2023 06:24:10 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:cc:content-transfer-encoding:content-type:date:date:from:from :in-reply-to:message-id:mime-version:reply-to:sender:subject :subject:to:to; s=fm2; t=1687775050; x=1687861450; bh=Tp3ffDTurS VVijj0+h5MXLbTF1VjMLWByrWXyojJQ/0=; b=mJTsa75YFc5b1D2VFKGLmPnrja M9sSjtbu5qXKhGEUgH8ZVVGMtUMhkyffTU447+/znATCVhN7105Rw1Fn+JX6IqjS qUp+d/ReQYjxE787gF4kNugrL/FMp6tLv3zV6y8aTmTy+l6p/3JkS31ecyXMHZal 32dydDUZi+fyNFJRDh6yN8dvORSk6qiI3oGpqC6CdPzfxDPG0mjZLeMtyDyss7nw eyQesZfgw2BJIvLXfXClQ5YnNcGtMcBYJ8sCVJTL+pV8SjtCfbI+jc9QMFySBfua +MR4W2k5PbFiZVaCMxI++coNVS47kIhFMt7UJBvgZrhTEd0oXpqMfLEo+U2Q== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:date:date:feedback-id:feedback-id:from:from :in-reply-to:message-id:mime-version:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; t=1687775050; x=1687861450; bh=Tp3ffDTurSVVi jj0+h5MXLbTF1VjMLWByrWXyojJQ/0=; b=RS7Ci6rzbloWTSb+gc/71Mn7CSj1x g1AfBsaJqG6fxTmtNsaU+GXv1wWxawWl4EmA0gkb8ycmtR/gE7g2wqNUZRfAh4U6 Sflzp/zcrbGFpZnFgNi3j0DkjVTMwRZVNq8+rOmpmFNKxmMz2ZfaToXaPoH3JQUf KFGLjh8wwpljA2Tu2Q/ZYOqN5Wmn+Ud/f1ojASjbaCj2F+EYpGK+qVGyUx6jwjtc s+D7jWwiI7921/iQWga02go+3Z8T4IeVT83gb10hAUofHbPqkpYuDSRsn8A4H9tZ HR4YufpWYtgF2IFoALZGOA5FH5BFwwf90LyUpKdpWAx0TRfq1h9gjCjAA== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrgeehfedgvdegucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvvefufffkofgggfestdekredtredttdenucfhrhhomhepvfhhohhmrghs ucfoohhnjhgrlhhonhcuoehthhhomhgrshesmhhonhhjrghlohhnrdhnvghtqeenucggtf frrghtthgvrhhnpeevveefveeiiefhtedukeegkeelieegffeukeevudetueffieduveet iedukeefleenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhroh hmpehthhhomhgrshesmhhonhhjrghlohhnrdhnvght X-ME-Proxy: Feedback-ID: i47234305:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Mon, 26 Jun 2023 06:24:09 -0400 (EDT) From: Thomas Monjalon To: dev@dpdk.org Cc: stable@dpdk.org, Bruce Richardson , David Marchand Subject: [PATCH] devtools: fix mailmap check for parentheses Date: Mon, 26 Jun 2023 12:24:03 +0200 Message-ID: <20230626102403.2531582-1-thomas@monjalon.net> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 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 When checking names having parentheses, the grep matching was failing. It is fixed by escaping the open parenthesis. Also, the mailmap path was relative to the root directory. The path is made absolute. Fixes: e83d41f0694d ("mailmap: add list of contributors") Fixes: 83812de4f2f3 ("devtools: move mailmap check after patch applied") Cc: stable@dpdk.org Signed-off-by: Thomas Monjalon Acked-by: Stephen Hemminger --- devtools/check-git-log.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/devtools/check-git-log.sh b/devtools/check-git-log.sh index af751e49ab..89544a2cc5 100755 --- a/devtools/check-git-log.sh +++ b/devtools/check-git-log.sh @@ -264,8 +264,10 @@ names=$(git log --format='From: %an <%ae>%n%b' --reverse $range | sed -rn 's,.*: (.*<.*@.*>),\1,p' | sort -u) bad=$(for contributor in $names ; do + contributor=${contributor//(/\\(} ! grep -qE "^$contributor($| <)" $selfdir/../.mailmap || continue - if grep -q "^${contributor%% <*} <" .mailmap ; then + name=${contributor%% <*} + if grep -q "^$name <" $selfdir/../.mailmap ; then printf "\t$contributor is not the primary email address\n" else printf "\t$contributor is unknown in .mailmap\n"