Re: [edk2-platforms][PATCH] Change edk2-platforms/master to BSD+Patent License

["Chiu, Chasel" <chasel.chiu@intel.com>]

Agree on deleting extra license files. Thanks!

Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>



> -----Original Message-----
> From: Kubacki, Michael A
> Sent: Thursday, May 16, 2019 12:45 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; Leif Lindholm
> <leif.lindholm@linaro.org>; Chiu, Chasel <chasel.chiu@intel.com>
> Cc: devel@edk2.groups.io; Ard Biesheuvel (ard.biesheuvel@linaro.org)
> <ard.biesheuvel@linaro.org>; Gillispie, Thad <thad.gillispie@intel.com>; Bu,
> Daocheng <daocheng.bu@intel.com>; Oram, Isaac W
> <isaac.w.oram@intel.com>; Piwko, Maciej <maciej.piwko@intel.com>; Lu, Shifei
> A <shifei.a.lu@intel.com>; Zhou, Bowen <bowen.zhou@intel.com>; Sinha, Ankit
> <ankit.sinha@intel.com>; Chaganty, Rangasai V
> <rangasai.v.chaganty@intel.com>
> Subject: RE: [edk2-platforms][PATCH] Change edk2-platforms/master to
> BSD+Patent License
> 
> After the license files are deleted.
> 
> Reviewed-by: Michael Kubacki <michael.a.kubacki@intel.com>
> 
> > -----Original Message-----
> > From: Kinney, Michael D
> > Sent: Wednesday, May 15, 2019 9:33 AM
> > To: Leif Lindholm <leif.lindholm@linaro.org>; Chiu, Chasel
> > <chasel.chiu@intel.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>
> > Cc: devel@edk2.groups.io; Ard Biesheuvel (ard.biesheuvel@linaro.org)
> > <ard.biesheuvel@linaro.org>; Kubacki, Michael A
> > <michael.a.kubacki@intel.com>; Gillispie, Thad
> > <thad.gillispie@intel.com>; Bu, Daocheng <daocheng.bu@intel.com>;
> > Oram, Isaac W <isaac.w.oram@intel.com>; Piwko, Maciej
> > <maciej.piwko@intel.com>; Lu, Shifei A <shifei.a.lu@intel.com>; Zhou,
> > Bowen <bowen.zhou@intel.com>; Sinha, Ankit <ankit.sinha@intel.com>;
> > Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>
> > Subject: RE: [edk2-platforms][PATCH] Change edk2-platforms/master to
> > BSD+Patent License
> >
> > Leif,
> >
> > Yes.  Deleting the extra license files is correct if the content Uses
> > the
> > BSD+Patent license in the License.txt in the root.
> >
> > I will fix this in V2.
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> > > Sent: Wednesday, May 15, 2019 4:17 AM
> > > To: Chiu, Chasel <chasel.chiu@intel.com>
> > > Cc: devel@edk2.groups.io; Kinney, Michael D
> > > <michael.d.kinney@intel.com>; Ard Biesheuvel
> > > (ard.biesheuvel@linaro.org)
> > > <ard.biesheuvel@linaro.org>; Kubacki, Michael A
> > > <michael.a.kubacki@intel.com>; Gillispie, Thad
> > > <thad.gillispie@intel.com>; Bu, Daocheng <daocheng.bu@intel.com>;
> > > Oram, Isaac W <isaac.w.oram@intel.com>; Piwko, Maciej
> > > <maciej.piwko@intel.com>; Lu, Shifei A <shifei.a.lu@intel.com>;
> > > Zhou, Bowen <bowen.zhou@intel.com>; Sinha, Ankit
> > > <ankit.sinha@intel.com>; Chaganty, Rangasai V
> > > <rangasai.v.chaganty@intel.com>
> > > Subject: Re: [edk2-platforms][PATCH] Change edk2- platforms/master
> > > to
> > > BSD+Patent License
> > >
> > > On Wed, May 15, 2019 at 07:11:52AM +0000, Chiu, Chasel
> > > wrote:
> > > >
> > > > Hi Mike,
> > > >
> > > > Seems we missed below files, would you please check?
> > > > 	.\Platform\Intel\KabylakeOpenBoardPkg\License.txt
> > > > 	.\Silicon\Intel\KabylakeSiliconPkg\License.txt
> > > >
> > > > With above 2 files updated,
> > >
> > > Would not deleting them be more appropriate?
> > >
> > > Best Regards,
> > >
> > > Leif
> > >
> > > > Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>
> > > > 	.\Platform\Intel\KabylakeOpenBoardPkg
> > > > 	.\Silicon\Intel\KabylakeSiliconPkg
> > > >
> > > > Thanks!
> > > > Chasel
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: devel@edk2.groups.io
> > > [mailto:devel@edk2.groups.io] On Behalf Of
> > > > > Michael D Kinney
> > > > > Sent: Wednesday, May 15, 2019 9:31 AM
> > > > > To: devel@edk2.groups.io
> > > > > Cc: leif.lindholm@linaro.org; Ard Biesheuvel
> > > (ard.biesheuvel@linaro.org)
> > > > > <ard.biesheuvel@linaro.org>; Kubacki, Michael A
> > > > > <michael.a.kubacki@intel.com>; Gillispie, Thad
> > > <thad.gillispie@intel.com>; Bu,
> > > > > Daocheng <daocheng.bu@intel.com>; Oram, Isaac W
> > > > > <isaac.w.oram@intel.com>; Piwko, Maciej
> > > <maciej.piwko@intel.com>; Chiu,
> > > > > Chasel <chasel.chiu@intel.com>; Lu, Shifei A
> > > <shifei.a.lu@intel.com>; Zhou,
> > > > > Bowen <bowen.zhou@intel.com>; Sinha, Ankit
> > > <ankit.sinha@intel.com>;
> > > > > Chaganty, Rangasai V
> > > <rangasai.v.chaganty@intel.com>
> > > > > Subject: [edk2-devel] [edk2-platforms][PATCH]
> > > Change edk2-platforms/master
> > > > > to BSD+Patent License
> > > > >
> > > > > Hello,
> > > > >
> > > > > BZ:
> > > https://bugzilla.tianocore.org/show_bug.cgi?id=1373
> > > > >
> > > > > This change is based on the following emails:
> > > > >   https://lists.01.org/pipermail/edk2-devel/2019-
> > > February/036260.html
> > > > >   https://lists.01.org/pipermail/edk2-devel/2018-
> > > October/030385.html
> > > > >
> > > > > RFCs with detailed process for the license change:
> > > > >   V3: https://lists.01.org/pipermail/edk2-
> > > devel/2019-March/038116.html
> > > > >   V2: https://lists.01.org/pipermail/edk2-
> > > devel/2019-March/037669.html
> > > > >   V1: https://lists.01.org/pipermail/edk2-
> > > devel/2019-March/037500.html
> > > > >
> > > > > I have posted a V1 branch for review for converting
> > > the edk2-platforms/master
> > > > > branch to the BSD + Patent license
> > > > >
> > > > > https://github.com/mdkinney/edk2-
> > > platforms/tree/Bug_1373_BsdPatentLicen
> > > > > se_V1
> > > > >
> > > > > The list of commits in the series are here:
> > > > >
> > > > > https://github.com/mdkinney/edk2-
> > > platforms/commits/Bug_1373_BsdPatent
> > > > > License_V1
> > > > >
> > > > > I have a couple questions out to maintainers for a
> > > few files in this repository.
> > > > > Once I have that feedback, I will post a
> > > > > V2 version with those updates.
> > > > >
> > > > > The edk2-platforms/master branch maintainers and
> > > package reviewers should
> > > > > provide review feedback for their packages.  The
> > > critical part of the review is:
> > > > > 1) Any changes that cause build breaks or logic
> > > changes.  These code
> > > > >    changes are intended to only modify license
> > > contents in comment
> > > > >    blocks.
> > > > > 2) Any file that has been changed to BSD+Patent,
> > > but should remain
> > > > >    with the current license.
> > > > > 3) Any file that that has not changed to
> > > BSD+Patent, but should be
> > > > >    changed to BSD+Patent.
> > > > >
> > > > > Thanks,
> > > > >
> > > > > Mike
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > 
> > > >