From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-1.mimecast.com (us-smtp-delivery-1.mimecast.com [207.211.31.81]) by mx.groups.io with SMTP id smtpd.web11.4883.1599044951533864286 for ; Wed, 02 Sep 2020 04:09:11 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=R0osbwg8; spf=pass (domain: redhat.com, ip: 207.211.31.81, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1599044950; 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: in-reply-to:in-reply-to:references:references; bh=4YtBsl//iKxykBN1Zr605E35KxZLUW+nNg8El1JqZUQ=; b=R0osbwg8sSIeyN//jk//dpaN/pY4rwawM87FEU8cBSpOKxZK59UzB050c2b9+pa2hOmC+H kMSFbEdMFrbIfeDinD8tUY+lDzdYgdEFkTdvz3ah4xfO2Ly7qRce8GVPKYHnt6FUX9lyFs BBEYOZNO2pvE5xeBHLYBYU2su5E3br8= 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-101-MTj9D12TNGykidc9JkOhoA-1; Wed, 02 Sep 2020 07:08:58 -0400 X-MC-Unique: MTj9D12TNGykidc9JkOhoA-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id D7BF964084; Wed, 2 Sep 2020 11:08:56 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-113-14.ams2.redhat.com [10.36.113.14]) by smtp.corp.redhat.com (Postfix) with ESMTP id B8C3261983; Wed, 2 Sep 2020 11:08:54 +0000 (UTC) Subject: =?UTF-8?B?UmU6IOWbnuWkjTogW2VkazItZGV2ZWxdIOWbnuWkjTogW1BBVENIIDEvMV0gLnB5dG9vbC9FY2NDaGVjazogRW5hYmxlIEVjYyBlcnJvciBjb2RlIDEwMDE0IGZvciBvcGVuIENJ?= To: devel@edk2.groups.io, gaoliming@byosoft.com.cn, 'Shenglei Zhang' , afish@apple.com, 'Leif Lindholm' Cc: 'Sean Brogan' , 'Bret Barkelew' , 'Michael D Kinney' References: <20200902083801.338608-1-shenglei.zhang@intel.com> <1630EC2C35B89AD7.28345@groups.io> <003d01d6810c$8aa46ab0$9fed4010$@byosoft.com.cn> From: "Laszlo Ersek" Message-ID: Date: Wed, 2 Sep 2020 13:08:53 +0200 MIME-Version: 1.0 In-Reply-To: <003d01d6810c$8aa46ab0$9fed4010$@byosoft.com.cn> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0.001 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Content-Language: en-US Hi Liming, Shenglei, On 09/02/20 11:36, gaoliming wrote: > Because the fix for ECC tool has been merged, I also request to catch this > change for stable202008 tag. I'm OK with that, given that this patch modifies .pytool (that is, CI) -- in other words, I'm OK because the patch affects our workflow only, and doesn't really affect the edk2 codebase itself. (Put differently, downstream parties that fork edk2 at edk2-stable202008, or rebase their current forks to edk2-stable202008, cannot be affected by this change in any way.) However, I suggest a different approach for the patch itself (the end result will be the same, code-wise). Please start the patch by running: $ git revert d4e0b9607c9a and then please extend the commit message. It should be clear from the commit log (subject and first paragraph) that this patch is a revert. Furthermore, TianoCore#2904 is in RESOLVED|FIXED status already. (And even the code comment, from the original patch, says: "need to be removed *after* BZ2904 is fixed" [emphasis mine] -- so this new patch, the revert, is not expected to be a part of BZ2904.) So I suggest opening a new TianoCore BZ as well, making it *dependent* on 2904. And the revert should reference the new BZ. Again, I'm fine with merging this patch before the stable tag, I'd just like the BZ reference and the revert statement to be clean. Thanks! Laszlo >> -----邮件原件----- >> 发件人: bounce+27952+64967+4905953+8761045@groups.io >> 代表 gaoliming >> 发送时间: 2020年9月2日 17:06 >> 收件人: 'Shenglei Zhang' ; devel@edk2.groups.io >> 抄送: 'Sean Brogan' ; 'Bret Barkelew' >> ; 'Michael D Kinney' >> >> 主题: [edk2-devel] 回复: [PATCH 1/1] .pytool/EccCheck: Enable Ecc error >> code 10014 for open CI >> >> Reviewed-by: Liming Gao >> >> Thanks >> Liming >>> -----邮件原件----- >>> 发件人: Shenglei Zhang >>> 发送时间: 2020年9月2日 16:38 >>> 收件人: devel@edk2.groups.io >>> 抄送: Sean Brogan ; Bret Barkelew >>> ; Michael D Kinney >>> ; Liming Gao >>> 主题: [PATCH 1/1] .pytool/EccCheck: Enable Ecc error code 10014 for open >> CI >>> >>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2904 >>> As the false positive issue whose error code is 10014 has been >>> fixed, we need to re-enable it on open CI. >>> >>> Cc: Sean Brogan >>> Cc: Bret Barkelew >>> Cc: Michael D Kinney >>> Cc: Liming Gao >>> Signed-off-by: Shenglei Zhang >>> --- >>> .pytool/Plugin/EccCheck/EccCheck.py | 1 - >>> 1 file changed, 1 deletion(-) >>> >>> diff --git a/.pytool/Plugin/EccCheck/EccCheck.py >>> b/.pytool/Plugin/EccCheck/EccCheck.py >>> index 3eaad0bf5623..eee1ff7a77b5 100644 >>> --- a/.pytool/Plugin/EccCheck/EccCheck.py >>> +++ b/.pytool/Plugin/EccCheck/EccCheck.py >>> @@ -301,7 +301,6 @@ class EccCheck(ICiBuildPlugin): >>> "10011", >>> "10012", >>> "10013", >>> - "10014", #need to be removed after >>> BZ2904 is fixed >>> "10015", >>> "10016", >>> "10017", >>> -- >>> 2.18.0.windows.1 >> >> >> >> >> > > > > > >