From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-1.mimecast.com (us-smtp-delivery-1.mimecast.com [205.139.110.61]) by mx.groups.io with SMTP id smtpd.web11.2067.1578306941560134463 for ; Mon, 06 Jan 2020 02:35:41 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=FR5yxqvt; spf=pass (domain: redhat.com, ip: 205.139.110.61, mailfrom: philmd@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1578306940; 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: content-transfer-encoding:content-transfer-encoding; bh=PWf4KGkwzbhyN1hsBDNzdFrq5YKZbCcI/Ihe06kDvD0=; b=FR5yxqvtvyQ0jUh2x+kDUYXGJO3eXdjvCfkpFcRh4lkQvjn83lY2MmxtIwI80OL/aIhG39 MOt2UKiMw906MYqTnEWgqzOsKRIP7A3sbeEfX9hXzRt0IpcODzJPpT46cwMrEK9f6TTl4w 8CHDsN4VkAduhnc1Gw7rkAecBqt1RX4= 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-292-xgoCsJUsPpGPfxcwbxZdog-1; Mon, 06 Jan 2020 05:35:36 -0500 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 0FACD100550E; Mon, 6 Jan 2020 10:35:36 +0000 (UTC) Received: from x1w.redhat.com (ovpn-204-53.brq.redhat.com [10.40.204.53]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 9C7EC5D9D6; Mon, 6 Jan 2020 10:35:31 +0000 (UTC) From: =?UTF-8?B?UGhpbGlwcGUgTWF0aGlldS1EYXVkw6k=?= To: devel@edk2.groups.io Cc: Michael D Kinney , Laszlo Ersek , Philippe Mathieu-Daude Subject: [PATCH v2 0/4] BaseTools/PatchCheck: Check committer/author email addresses Date: Mon, 6 Jan 2020 11:35:24 +0100 Message-Id: <20200106103528.5438-1-philmd@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-MC-Unique: xgoCsJUsPpGPfxcwbxZdog-1 X-Mimecast-Spam-Score: 0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Last month I cleaned the git-history incorrect email addresses: https://edk2.groups.io/g/devel/message/51834 but today I noticed more incorrect addresses got committed (see caa917491a4..33a3293651). To avoid having the same cleanup in the future, fix the problem once for all by having PatchCheck doing this automatically (on merge). Since we already have code to check email address in PatchCheck, factor the code out to reuse it, and add checks for committer/author addresses. Series only tested on Linux. Since v1 [*]: - Addressed Jordan Justen review comment [*] https://edk2.groups.io/g/devel/message/52656 Philippe Mathieu-Daude (4): BaseTools/PatchCheck.py: Extract email check code to EmailAddressCheck BaseTools/PatchCheck.py: Let EmailAddressCheck describe email checked BaseTools/PatchCheck.py: Check the patch author email address BaseTools/PatchCheck.py: Check the committer email address BaseTools/Scripts/PatchCheck.py | 99 +++++++++++++++++++++++---------- 1 file changed, 69 insertions(+), 30 deletions(-) --=20 2.21.1