回复: [edk2-devel] [PATCH] Revert ".pytool/EccCheck: Disable Ecc error code 10014 for open CI"

["gaoliming" <gaoliming@byosoft.com.cn>]

Merge @06dc822d045c2bb42e497487935485302486e151

> -----邮件原件-----
> 发件人: bounce+27952+64999+4905953+8761045@groups.io
> <bounce+27952+64999+4905953+8761045@groups.io> 代表 gaoliming
> 发送时间: 2020年9月3日 16:18
> 收件人: devel@edk2.groups.io; lersek@redhat.com;
> shenglei.zhang@intel.com
> 抄送: 'Sean Brogan' <sean.brogan@microsoft.com>; 'Bret Barkelew'
> <Bret.Barkelew@microsoft.com>; 'Michael D Kinney'
> <michael.d.kinney@intel.com>
> 主题: 回复: [edk2-devel] [PATCH] Revert ".pytool/EccCheck: Disable Ecc
> error code 10014 for open CI"
> 
> Laszlo:
>   Thanks! I will merge it. Reviewed-by: Liming Gao
> <gaoliming@byosoft.com.cn>
> 
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: bounce+27952+64998+4905953+8761045@groups.io
> > <bounce+27952+64998+4905953+8761045@groups.io> 代表 Laszlo Ersek
> > 发送时间: 2020年9月3日 16:08
> > 收件人: devel@edk2.groups.io; shenglei.zhang@intel.com
> > 抄送: Sean Brogan <sean.brogan@microsoft.com>; Bret Barkelew
> > <Bret.Barkelew@microsoft.com>; Michael D Kinney
> > <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>
> > 主题: Re: [edk2-devel] [PATCH] Revert ".pytool/EccCheck: Disable Ecc
> error
> > code 10014 for open CI"
> >
> > On 09/03/20 08:56, Zhang, Shenglei wrote:
> > > This reverts commit d4e0b9607c9a64a8eff20724b2e35ea2cd5bd33f.
> > >
> > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2951
> > > Previously false positive Ecc issue whose error code is 10014 was reported
> > > under Linux OS. So we disabled it in EccCheck plugin for edk2 open CI.
> > > As the bug is fixed, we need to revert the change and re-enbale it.
> > >
> > > Cc: Sean Brogan <sean.brogan@microsoft.com>
> > > Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> > > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > > Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
> > > ---
> > >  .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",
> > >
> >
> > Reviewed-by: Laszlo Ersek <lersek@redhat.com>
> >
> > Liming, can you please merge this? If so, please fix a typo in the
> > commit message: "re-enbale" should be "re-enable".
> >
> > Thanks!
> > Laszlo
> >
> >
> >
> 
> 
> 
> 
>