[PATCH] Revert ".pytool/EccCheck: Disable Ecc error code 10014 for open CI"

[PATCH] Revert ".pytool/EccCheck: Disable Ecc error code 10014 for open CI"

[Zhang, Shenglei]

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",
-- 
2.18.0.windows.1

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

[Laszlo Ersek]

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

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

[gaoliming]

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
> 
> 
> 

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

[gaoliming]

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
> >
> >
> >
> 
> 
> 
> 
>