public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Abner Chang" <abner.chang@hpe.com>
To: gaoliming <gaoliming@byosoft.com.cn>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>,
	"'Feng, Bob C'" <bob.c.feng@intel.com>,
	"spbrogan@outlook.com" <spbrogan@outlook.com>,
	"shenglei.zhang@intel.com" <shenglei.zhang@intel.com>
Cc: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>,
	'Sean Brogan' <sean.brogan@microsoft.com>
Subject: Re: [edk2-devel] CI test on copyright
Date: Mon, 19 Oct 2020 05:08:30 +0000	[thread overview]
Message-ID: <CS1PR8401MB1144D3DF8525C7530B7B762AFF1E0@CS1PR8401MB1144.NAMPRD84.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <001f01d6a5d4$35eb4830$a1c1d890$@byosoft.com.cn>

Ah yes. I totally forget this. Thanks for the reminder.

> -----Original Message-----
> From: gaoliming [mailto:gaoliming@byosoft.com.cn]
> Sent: Monday, October 19, 2020 12:57 PM
> To: devel@edk2.groups.io; Chang, Abner (HPS SW/FW Technologist)
> <abner.chang@hpe.com>; 'Feng, Bob C' <bob.c.feng@intel.com>;
> spbrogan@outlook.com; shenglei.zhang@intel.com
> Cc: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>; 'Sean Brogan'
> <sean.brogan@microsoft.com>
> Subject: 回复: [edk2-devel] CI test on copyright
> 
> Abner:
>   You have submitted INVALID URI REMOVED
> 3A__bugzilla.tianocore.org_show-5Fbug.cgi-3Fid-
> 3D3011&d=DwIFaQ&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4Ulks
> kz6qU3NYRO03nHp9P7Z5q59A3E&m=vaB6hNbJbPOmQ8yaMFNWsRlvo07_u
> UgGuoezMKsV_Ho&s=Lq7nvu3edPECva-
> LC8iZY1I2_UjzvJh2zF1onHnzXw4&e=  for this issue. We can use it to track the
> fix.
> 
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: bounce+27952+66373+4905953+8761045@groups.io
> > <bounce+27952+66373+4905953+8761045@groups.io> 代表 Abner Chang
> > 发送时间: 2020年10月19日 11:26
> > 收件人: Feng, Bob C <bob.c.feng@intel.com>; devel@edk2.groups.io;
> > gaoliming <gaoliming@byosoft.com.cn>; spbrogan@outlook.com;
> > shenglei.zhang@intel.com
> > 抄送: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>; 'Sean Brogan'
> > <sean.brogan@microsoft.com>
> > 主题: Re: [edk2-devel] CI test on copyright
> >
> > Ok. I will submit an one.
> >
> > Thanks Bob and Liming.
> >
> > > -----Original Message-----
> > > From: Feng, Bob C [mailto:bob.c.feng@intel.com]
> > > Sent: Monday, October 19, 2020 11:01 AM
> > > To: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>;
> > > devel@edk2.groups.io; gaoliming <gaoliming@byosoft.com.cn>;
> > > spbrogan@outlook.com; shenglei.zhang@intel.com
> > > Cc: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>; 'Sean Brogan'
> > > <sean.brogan@microsoft.com>
> > > Subject: RE: [edk2-devel] CI test on copyright
> > >
> > > Yes.
> > >
> > > -----Original Message-----
> > > From: Chang, Abner (HPS SW/FW Technologist)
> <abner.chang@hpe.com>
> > > Sent: Monday, October 19, 2020 10:48 AM
> > > To: devel@edk2.groups.io; Feng, Bob C <bob.c.feng@intel.com>;
> > > gaoliming <gaoliming@byosoft.com.cn>; spbrogan@outlook.com;
> > > shenglei.zhang@intel.com
> > > Cc: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>; 'Sean Brogan'
> > > <sean.brogan@microsoft.com>
> > > Subject: RE: [edk2-devel] CI test on copyright
> > >
> > > Hi Bob,
> > > So the fix was only applied on c and h file as your mentioned? If so
> > > then we need a BZ to add the same rule on the files other then c and h.
> > >
> > > Thanks
> > > Abner
> > >
> > > > -----Original Message-----
> > > > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf
> > > > Of Bob Feng
> > > > Sent: Monday, October 19, 2020 10:25 AM
> > > > To: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io;
> > > > spbrogan@outlook.com; shenglei.zhang@intel.com; Chang, Abner (HPS
> > > > SW/FW Technologist) <abner.chang@hpe.com>
> > > > Cc: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>; 'Sean Brogan'
> > > > <sean.brogan@microsoft.com>
> > > > Subject: Re: [edk2-devel] CI test on copyright
> > > >
> > > > For the .c and .h, ECC add the rule to allow (C) Copyright format.
> > > > Looks this bug was not fixed completely.
> > > > INVALID URI REMOVED
> > > > 3A__bugzilla.tianocore.org_show-5Fbug.cgi-3Fid-
> > > >
> > >
> >
> 3D2057&d=DwIGaQ&c=C5b8zRQO1miGmBeVZ2LFWg&r=_SN6FZBN4Vgi4Ulk
> > > >
> > >
> >
> skz6qU3NYRO03nHp9P7Z5q59A3E&m=oWYb80tvgLGP9RZPAmJdv9pqtmSQb
> > > > DbP1z9tdxINhI0&s=KduVFX0xp_qOfbmCqcRhYJ1r-19xwoTA6V4-
> > > > mbW5cF4&e=
> > > >
> > > > Need file a new Bugzilla and fix this issue.
> > > >
> > > > Thanks,
> > > > Bob
> > > > -----Original Message-----
> > > > From: gaoliming <gaoliming@byosoft.com.cn>
> > > > Sent: Monday, October 19, 2020 9:28 AM
> > > > To: Feng, Bob C <bob.c.feng@intel.com>; devel@edk2.groups.io;
> > > > spbrogan@outlook.com; shenglei.zhang@intel.com;
> > > abner.chang@hpe.com
> > > > Cc: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>; 'Sean Brogan'
> > > > <sean.brogan@microsoft.com>
> > > > Subject: 回复: [edk2-devel] CI test on copyright
> > > >
> > > > Bob:
> > > >   Now, the problem is the inconsistent behavior. The same
> > > > copyright style ((C) Copyright 2020 Hewlett Packard Enterprise
> > > > Development LP ) in .c can pass ECC, but in .inf file can't pass
> > > > ECC. I want to know the
> > reason.
> > > >
> > > > Thanks
> > > > Liming
> > > > > -----邮件原件-----
> > > > > 发件人: Feng, Bob C <bob.c.feng@intel.com>
> > > > > 发送时间: 2020年10月19日 9:25
> > > > > 收件人: devel@edk2.groups.io; gaoliming@byosoft.com.cn;
> > > > > spbrogan@outlook.com; shenglei.zhang@intel.com;
> > > > abner.chang@hpe.com
> > > > > 抄送: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>; 'Sean Brogan'
> > > > > <sean.brogan@microsoft.com>
> > > > > 主题: RE: [edk2-devel] CI test on copyright
> > > > >
> > > > > In the ECC, the rule to check Copyright is to see if the
> > > > > Copyright is the first word of a line, and the Copyright should
> > > > > be followed by a (
> > > > >
> > > > > Liming, for your reference, the code in ECC
> > > > >
> > > > > ## _IsCopyrightLine
> > > > > # check whether current line is copyright line, the criteria is
> > > > > whether there is case insensitive keyword "Copyright"
> > > > > # followed by zero or more white space characters followed by a "("
> > > > > character # # @param LineContent:  the line need to be checked #
> > > > > @return: True if current line is copyright line, False else #
> > > > >
> > > > > -Bob
> > > > >
> > > > > -----Original Message-----
> > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > > gaoliming
> > > > > Sent: Monday, October 19, 2020 9:01 AM
> > > > > To: devel@edk2.groups.io; spbrogan@outlook.com;
> > > > > shenglei.zhang@intel.com; abner.chang@hpe.com
> > > > > Cc: 'Bret Barkelew' <Bret.Barkelew@microsoft.com>; 'Sean Brogan'
> > > > > <sean.brogan@microsoft.com>
> > > > > Subject: 回复: [edk2-devel] CI test on copyright
> > > > >
> > > > > I will check this issue first.
> > > > >
> > > > > Thanks
> > > > > Liming
> > > > >
> > > > > > -----邮件原件-----
> > > > > > 发件人: bounce+27952+66349+4905953+8761045@groups.io
> > > > > > <bounce+27952+66349+4905953+8761045@groups.io> 代表 Sean
> > > > > > 发送时间: 2020年10月17日 12:46
> > > > > > 收件人: devel@edk2.groups.io; shenglei.zhang@intel.com;
> > > > > > abner.chang@hpe.com
> > > > > > 抄送: Bret Barkelew <Bret.Barkelew@microsoft.com>; Sean Brogan
> > > > > > <sean.brogan@microsoft.com>
> > > > > > 主题: Re: [edk2-devel] CI test on copyright
> > > > > >
> > > > > > ECC stuff was done by Intel. I would defer to them.
> > > > > >
> > > > > > Adding shenglei.zhang@intel.com
> > > > > >
> > > > > > On 10/15/2020 9:21 AM, Abner Chang wrote:
> > > > > > > Hi Sean and Bret,
> > > > > > > I got the CI test error which says the first line in file
> > > > > > > header section
> > > > > must
> > > > > > have the copyright information, however the copyright is there
> > > > > > and looks
> > > > > to
> > > > > > me fine as below,
> > > > > > > (C) Copyright 2020 Hewlett Packard Enterprise Development
> > > > > > > LP<BR>
> > > > > > >
> > > > > > > [cid:image001.png@01D6A351.31688460]
> > > > > > >
> > > > > > >
> > > > > > > The interesting part is CI seems happy with below format of
> > > > > > > copyright,
> > > > > and
> > > > > > this issue only happens on INF file but not on the *.c and *.h.
> > > > > > > Copyright (C) 2020 Hewlett Packard Enterprise Development
> > > > > > > LP<BR>
> > > > > > >
> > > > > > > Any idea on this?
> > > > > > > Abner
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > >
> >
> >
> >
> > 
> >
> 
> 


      reply	other threads:[~2020-10-19  5:08 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-15 16:21 CI test on copyright Abner Chang
2020-10-15 16:24 ` Bret Barkelew
2020-10-15 16:32   ` Abner Chang
2020-10-15 16:34     ` Bret Barkelew
2020-10-16  1:07       ` [edk2-devel] " Abner Chang
2020-10-16  1:15         ` 回复: " gaoliming
2020-10-16  1:19           ` Abner Chang
     [not found]           ` <163E54449E0DFAB5.31077@groups.io>
2020-10-16  1:25             ` Abner Chang
2020-10-17  4:45 ` Sean
2020-10-19  1:01   ` 回复: " gaoliming
2020-10-19  1:24     ` Bob Feng
2020-10-19  1:28       ` 回复: " gaoliming
2020-10-19  2:24         ` Bob Feng
2020-10-19  2:48           ` Abner Chang
2020-10-19  3:01             ` Bob Feng
2020-10-19  3:25               ` Abner Chang
2020-10-19  4:56                 ` 回复: " gaoliming
2020-10-19  5:08                   ` Abner Chang [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CS1PR8401MB1144D3DF8525C7530B7B762AFF1E0@CS1PR8401MB1144.NAMPRD84.PROD.OUTLOOK.COM \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox