From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 7E80A740032 for ; Sun, 18 Feb 2024 20:59:59 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=Qn7XHn8nfwP7/BS6DRElKDQqZEkhY3vCofSb1UktINw=; c=relaxed/simple; d=groups.io; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Transfer-Encoding; s=20140610; t=1708289998; v=1; b=bwtC0GjIyNFOYL8lPOYu6wzHQU31zJ975igpFaM1ADomh+Achx39Ojj74xONTXyI8k3gINfo RphyGxwV60V8t2CA2f3wedsw3nYuBEmjQKbmjvwA5QZIzmddWgp0QB+7mPoy9Ft8EIaJ7NNrcSB 8mvZv2wDVjy1/T2Zi/+RBosY= X-Received: by 127.0.0.2 with SMTP id YJRKYY7687511xcrnCZcp2ef; Sun, 18 Feb 2024 12:59:58 -0800 X-Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.18]) by mx.groups.io with SMTP id smtpd.web11.26742.1708289997485769267 for ; Sun, 18 Feb 2024 12:59:57 -0800 X-IronPort-AV: E=McAfee;i="6600,9927,10988"; a="2235008" X-IronPort-AV: E=Sophos;i="6.06,169,1705392000"; d="scan'208";a="2235008" X-Received: from orviesa009.jf.intel.com ([10.64.159.149]) by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Feb 2024 12:59:57 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.06,169,1705392000"; d="scan'208";a="4351420" X-Received: from mdkinney-mobl.amr.corp.intel.com ([10.209.68.113]) by orviesa009-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Feb 2024 12:59:57 -0800 From: "Michael D Kinney" To: devel@edk2.groups.io Cc: Rebecca Cran , Liming Gao , Bob Feng , Yuwei Chen , Michael Kubacki , Ard Biesheuvel , Leif Lindholm Subject: [edk2-devel] [Patch 0/4] PatchCheck Updates Date: Sun, 18 Feb 2024 12:59:47 -0800 Message-Id: <20240218205951.497-1-michael.d.kinney@intel.com> MIME-Version: 1.0 Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,michael.d.kinney@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: 3GaCoa2AGeKffsOpY2DnwYAlx7686176AA= Content-Transfer-Encoding: 8bit X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=bwtC0GjI; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=intel.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4679 REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4680 REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4693 REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4694 Fix a collection of PatchCheck issues and add a new PatchCheck featue to check for a commit that modifies files in multiple packages with option to disable the multiple package check as a command line option or a commit message tag. * Reject patches that match Author email "devel@edk2.groups.io" * Update the current check for " via Groups.Io" to perform a case insensitive match. It appears that groups.io has changed the format of this string to use all lower case. * Update logic in CommitMessageCheck class to return errors detected in commit message signatures instead of silently passing the check. * Genenerate an error if no Cc tags are present to make sure all patches Cc the required maintainers/reviewers. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Cc: Yuwei Chen Cc: Michael Kubacki Cc: Ard Biesheuvel Cc: Leif Lindholm Signed-off-by: Michael D Kinney Michael D Kinney (4): BaseTools/Scripts/PatchCheck: Update Author checks BaseTools/Scripts/PatchCheck: Return CommitMessageCheck errors BaseTools/Scripts/PatchCheck: Error if no Cc tags are present BaseTools/Scripts/PatchCheck: Error if commit modifies multiple packages BaseTools/Scripts/PatchCheck.py | 91 +++++++++++++++++++++++++++++++-- 1 file changed, 86 insertions(+), 5 deletions(-) -- 2.40.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#115568): https://edk2.groups.io/g/devel/message/115568 Mute This Topic: https://groups.io/mt/104434581/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=-=-=-=-=-=-=-=-=-=-=-