public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH V2] Change EDK II to BSD+Patent License
@ 2019-03-23  2:25 Kinney, Michael D
  2019-03-25 12:11 ` Laszlo Ersek
                   ` (3 more replies)
  0 siblings, 4 replies; 9+ messages in thread
From: Kinney, Michael D @ 2019-03-23  2:25 UTC (permalink / raw)
  To: edk2-devel@lists.01.org, Kinney, Michael D

Hello,

New in V2
=========
* Remove Cc lines from commit messages
* Remove branch reference from commit messages
* Change license in 2 files missed in OvmfPkg
* Update OvmfPkg/License.txt to BSD+Patent as the default license
* Move the portions of Contributions.txt in the root of edk2 to
  Readme.md in the root of edk2 that describe how to contribute
  along with the commit message format.
* Add to Readme.md in the root of edk2 that Signed-off-by means that
  the contributor certifies compliance to the Developer's Certificate
  of Origin 1.1.  https://developercertificate.org
=========

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 the patch series for review on the following branch using
edk2-stable201903 as the base for the patch series.  

  https://github.com/mdkinney/edk2/tree/Bug_1373_BsdPatentLicense_V2

The commits in patch series can be viewed here:

  https://github.com/mdkinney/edk2/commits/Bug_1373_BsdPatentLicense_V2

The patch series has one patch per package along with a few patches
to update the license information in the root of the edk2 repository
as described in the RFC V3.

Due to the size of the patch series, I prefer to not send the
patch emails.  Instead, please perform code reviews using content
from the branch.

All EDK II package 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. 

Feedback and Reviewed-by emails should identify the patch the feedback
applies using the patch summary listed below.  The goal is to complete
all reviews to support the commit of these patches on April 9, 2019.
 
2a7d2c56bc edk2: Remove Contributions.txt and update Readme.md
f9d59ccdc5 OvmfPkg: Change License.txt from 2-Clause BSD to BSD+Patent
ce3fbf929e StdLibPrivateInternalFiles: Replace BSD License with BSD+Patent License
aa8a3692c7 StdLib: Replace BSD License with BSD+Patent License
2dfbe1e1ee AppPkg: Replace BSD License with BSD+Patent License
b2161f6dd8 Vlv2TbltDevicePkg: Replace BSD License with BSD+Patent License
3688c33755 Vlv2DeviceRefCodePkg: Replace BSD License with BSD+Patent License
8170308c98 UefiCpuPkg: Replace BSD License with BSD+Patent License
4b68832cdc StandaloneMmPkg: Replace BSD License with BSD+Patent License
327dc18122 SourceLevelDebugPkg: Replace BSD License with BSD+Patent License
6c4c506a5e SignedCapsulePkg: Replace BSD License with BSD+Patent License
2fdd514aff ShellPkg: Replace BSD License with BSD+Patent License
c667724cb9 ShellBinPkg: Replace BSD License with BSD+Patent License
76f9e555b8 SecurityPkg: Replace BSD License with BSD+Patent License
89a588c1be QuarkSocPkg: Replace BSD License with BSD+Patent License
454f280e0c QuarkPlatformPkg: Replace BSD License with BSD+Patent License
935319af35 PcAtChipsetPkg: Replace BSD License with BSD+Patent License
e39d07266d OvmfPkg: Replace BSD License with BSD+Patent License
24d6ba8a25 OptionRomPkg: Replace BSD License with BSD+Patent License
5c4a074eed Omap35xxPkg: Replace BSD License with BSD+Patent License
dde19815b3 Nt32Pkg: Replace BSD License with BSD+Patent License
acf2e84beb NetworkPkg: Replace BSD License with BSD+Patent License
3e404f30b6 MdePkg: Replace BSD License with BSD+Patent License
6e01bf6764 MdeModulePkg: Replace BSD License with BSD+Patent License
36322b1b8c IntelSiliconPkg: Replace BSD License with BSD+Patent License
f5f9fa8b95 IntelFspWrapperPkg: Replace BSD License with BSD+Patent License
39725dc5e3 IntelFspPkg: Replace BSD License with BSD+Patent License
ba8a9e3e1d IntelFsp2WrapperPkg: Replace BSD License with BSD+Patent License
a61295eeda IntelFsp2Pkg: Replace BSD License with BSD+Patent License
cd7bd24c6d IntelFrameworkPkg: Replace BSD License with BSD+Patent License
e9f7710bb6 IntelFrameworkModulePkg: Replace BSD License with BSD+Patent License
487e96a6b4 FmpDevicePkg: Replace BSD License with BSD+Patent License
3cfd1b98e4 FatPkg: Replace BSD License with BSD+Patent License
f9fc278ea1 EmulatorPkg: Replace BSD License with BSD+Patent License
1a26764621 EmbeddedPkg: Replace BSD License with BSD+Patent License
aa402ac906 EdkCompatibilityPkg: Replace BSD License with BSD+Patent License
3c6df91d7c DynamicTablesPkg: Replace BSD License with BSD+Patent License
6389a5b4d5 CryptoPkg: Replace BSD License with BSD+Patent License
0065fa2d9f CorebootPayloadPkg: Replace BSD License with BSD+Patent License
26d7dbf868 CorebootModulePkg: Replace BSD License with BSD+Patent License
b1ebd76234 BeagleBoardPkg: Replace BSD License with BSD+Patent License
f23540ea65 ArmVirtPkg: Replace BSD License with BSD+Patent License
054b667071 ArmPlatformPkg: Replace BSD License with BSD+Patent License
5128ec1897 ArmPkg: Replace BSD License with BSD+Patent License
3b7fd23df9 BaseTools: Replace BSD License with BSD+Patent License
aa5e7ad3ef edk2: Replace BSD License with BSD+Patent License
fdcf6f00c7 edk2: Change License.txt from 2-Clause BSD to BSD+Patent
831e2096e8 edk2: Add License-History.txt

Best regards,

Mike

> -----Original Message-----
> From: Kinney, Michael D
> Sent: Friday, March 22, 2019 6:49 PM
> To: edk2-devel@lists.01.org; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Subject: [RFC v3] Change EDK II to BSD+Patent License
> 
> Hello,
> 
> Based on review of the RFC V2, there are some updates
> required to
> Readme.md in the root of the edk2 repository.
> 
> Changes for V3
> ===============
> * Move the portions of Contributions.txt in the root of
> edk2 to
>   Readme.md in the root of edk2 that describe how to
> contribute
>   along with the commit message format.
> 
> * Add to Readme.md in the root of edk2 that Signed-off-by
> means that
>   the contributor certifies compliance to the Developer's
> Certificate
>   of Origin 1.1.  https://developercertificate.org
> 
> Changes for V2
> ===============
> * Replace 2-Clause BSD License in file headers with SPDX-
> License-Identifier
>   statement.  This reduces the size of the file headers
> and the size
>   of the patches for this change.  Based on the following
> post:
> 
>   https://01.org/blogs/jc415/2018/open-source-hacks-one-
> question-interviews-open-source-experts-how-use-spdx-
> headers
> 
> * Update License.txt in root of edk2 before changing file
> headers.
> * Fix minor typos
> ===============
> 
> This RFC follows up on the proposal from Mark Doran to
> change the
> EDK II Project to a BSD+Patent License.
> 
> 	https://lists.01.org/pipermail/edk2-devel/2019-
> February/036260.html
> 
> The review period for this license change is 30 days.  If
> there is no
> unresolved feedback on April 9, 2019, then commits of the
> license change
> patches will begin on April 9, 2019.
> 
>   ** Please provide feedback on the proposal by Monday
> April 8, 2019. **
> 
> Feedback can be sent to edk2-devel at lists.01.org, the
> EDK II community
> manager or any of the EDK II stewards.
> 
>   * Stephano Cetola <stephano.cetola at intel.com>
> Community Manager
>   * Leif Lindholm   <leif.lindholm at linaro.org>
> Steward
>   * Andrew Fish     <afish at apple.com>
> Steward
>   * Laszlo Ersek    <lersek at redhat.com>
> Steward
>   * Michael Kinney  <michael.d.kinney at intel.com>
> Steward
> 
> The goal is to convert all of the files in the edk2
> repository that are
> currently covered by the 2-Clause BSD License and the
> TianoCore
> Contribution Agreement to a BSD+Patent License.
> 
> I will be following up with pointers to public GitHub
> branches that
> contain the set of changes to the edk2 repository for
> review.
> 
> The proposal is to perform this change to edk2/master in
> the steps listed
> below. The license change will not be applied to any of
> the other existing
> branches in the edk2 repository.
> 
> 1) Add a License-History.txt file to the root of the edk2
> repository that
>    contains the 2-Clause BSD License and the TianoCore
> Contribution
>    Agreement along with the details on the change to the
> BSD+Patent License.
> 
> 2) Change License.txt in the root of the edk2 repository
> from a 2-Clause
>    BSD License to the BSD+Patent License. The following
> is the link to the
>    BSD+Patent License and the new License.txt file
> contents.
> 
>    https://opensource.org/licenses/BSDplusPatent
> 
> 
> =========================================================
> =============
>    Redistribution and use in source and binary forms,
> with or without
>    modification, are permitted provided that the
> following conditions are met:
> 
>    1. Redistributions of source code must retain the
> above copyright notice,
>       this list of conditions and the following
> disclaimer.
> 
>    2. Redistributions in binary form must reproduce the
> above copyright notice,
>       this list of conditions and the following
> disclaimer in the documentation
>       and/or other materials provided with the
> distribution.
> 
>    Subject to the terms and conditions of this license,
> each copyright holder
>    and contributor hereby grants to those receiving
> rights under this license
>    a perpetual, worldwide, non-exclusive, no-charge,
> royalty-free, irrevocable
>    (except for failure to satisfy the conditions of this
> license) patent
>    license to make, have made, use, offer to sell, sell,
> import, and otherwise
>    transfer this software, where such license applies
> only to those patent
>    claims, already acquired or hereafter acquired,
> licensable by such copyright
>    holder or contributor that are necessarily infringed
> by:
> 
>    (a) their Contribution(s) (the licensed copyrights of
> copyright holders and
>        non-copyrightable additions of contributors, in
> source or binary form)
>        alone; or
> 
>    (b) combination of their Contribution(s) with the work
> of authorship to
>        which such Contribution(s) was added by such
> copyright holder or
>        contributor, if, at the time the Contribution is
> added, such addition
>        causes such combination to be necessarily
> infringed. The patent license
>        shall not apply to any other combinations which
> include the
>        Contribution.
> 
>    Except as expressly stated above, no rights or
> licenses from any copyright
>    holder or contributor is granted under this license,
> whether expressly, by
>    implication, estoppel or otherwise.
> 
>    DISCLAIMER
> 
>    THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
> CONTRIBUTORS "AS IS"
>    AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT
> NOT LIMITED TO, THE
>    IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> A PARTICULAR PURPOSE
>    ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> HOLDERS OR CONTRIBUTORS BE
>    LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
> EXEMPLARY, OR
>    CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
> PROCUREMENT OF
>    SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
> PROFITS; OR BUSINESS
>    INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
> LIABILITY, WHETHER IN
>    CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
> NEGLIGENCE OR OTHERWISE)
>    ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE,
> EVEN IF ADVISED OF THE
>    POSSIBILITY OF SUCH DAMAGE.
> 
> =========================================================
> =============
> 
> 3) Change all files currently covered by a 2-Clause BSD
> License and the
>    TianoCore Contribution Agreement to a BSD+Patent
> License using the
>    following SPDX-License-Identifier statement:
> 
>        SPDX-License-Identifier: BSD-2-Clause-Patent
> 
>    The use of SPDX-License-Identifier statement is based
> on the following:
> 
>         https://01.org/blogs/jc415/2018/open-source-
> hacks-one-question-interviews-open-source-experts-how-
> use-spdx-headers
> 
> 4) Update Readme.md in the root of the edk2 repository to
> state that content
>    is covered by a BSD+Patent License.  Also state that
> the BSD+Patent License
>    is the preferred license for the EDK II project.
> 
>   a) Move the portions of Contributions.txt in the root
> of the edk2 repository
>      Readme.md in the root of edk2 repository that
> describe how to contribute
>      along with the commit message format.
> 
>   b) Add the following to Readme.md in the root of edk2
> repository:
> 
>      # Developer Certificate of Origin
> 
>      Your change description should use the standard
> format for a
>      commit message, and must include your `Signed-off-
> by` signature.
> 
>      In order to keep track of who did what, all patches
> contributed must
>      include a statement that to the best of the
> contributor's knowledge
>      they have the right to contribute it under the
> specified license.
> 
>      The test for this is as specified in the
> [Developer's Certificate of
>      Origin (DCO)
> 1.1](https://developercertificate.org/). The contributor
>      certifies compliance by adding a line saying
> 
>        Signed-off-by: Developer Name
> <developer@example.org>
> 
>      where `Developer Name` is the contributor's real
> name, and the email
>      address is one the developer is reachable through at
> the time of
>      contributing.
> 
>      ```
>      Developer's Certificate of Origin 1.1
> 
>      By making a contribution to this project, I certify
> that:
> 
>       (a) The contribution was created in whole or in
> part by me and I
>          have the right to submit it under the open
> source license
>          indicated in the file; or
> 
>       (b) The contribution is based upon previous work
> that, to the best
>          of my knowledge, is covered under an appropriate
> open source
>          license and I have the right under that license
> to submit that
>          work with modifications, whether created in
> whole or in part
>          by me, under the same open source license
> (unless I am
>          permitted to submit under a different license),
> as indicated
>          in the file; or
> 
>       (c) The contribution was provided directly to me by
> some other
>          person who certified (a), (b) or (c) and I have
> not modified
>          it.
> 
>       (d) I understand and agree that this project and
> the contribution
>          are public and that a record of the contribution
> (including all
>          personal information I submit with it, including
> my sign-off) is
>          maintained indefinitely and may be redistributed
> consistent with
>          this project or the open source license(s)
> involved.
>      ```
> 
> 5) Remove the Contributions.txt file from the root of the
> edk2 repository
>    that contains the TianoCore Contribution Agreement.
> 
> 6) Update all documentation to state that content
> submitted under the
>    BSD+Patent License no longer requires the Tianocore
> Contribution
>    Agreement which means the following line is not
> required in commit
>    messages for changes to files that are covered by a
> BSD+Patent License.
> 
>        Contributed-under: TianoCore Contribution
> Agreement 1.1
> 
> 7) Create Wiki page(s) that provide the details of the
> BSD+Patent License
>    change and provides the status of the license change
> for each TianoCore
>    repository and package.
> 
> Once the conversion of the edk2 repository is complete,
> work will begin
> on the other repositories in the TianoCore project.
> 
> Best regards,
> 
> Mike



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH V2] Change EDK II to BSD+Patent License
  2019-03-23  2:25 [PATCH V2] Change EDK II to BSD+Patent License Kinney, Michael D
@ 2019-03-25 12:11 ` Laszlo Ersek
  2019-03-25 17:24   ` Kinney, Michael D
  2019-03-26 18:08 ` Leif Lindholm
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 9+ messages in thread
From: Laszlo Ersek @ 2019-03-25 12:11 UTC (permalink / raw)
  To: Kinney, Michael D, edk2-devel@lists.01.org

Hi Mike,

On 03/23/19 03:25, Kinney, Michael D wrote:
> Hello,
> 
> New in V2
> =========
> * Remove Cc lines from commit messages
> * Remove branch reference from commit messages
> * Change license in 2 files missed in OvmfPkg
> * Update OvmfPkg/License.txt to BSD+Patent as the default license
> * Move the portions of Contributions.txt in the root of edk2 to
>   Readme.md in the root of edk2 that describe how to contribute
>   along with the commit message format.
> * Add to Readme.md in the root of edk2 that Signed-off-by means that
>   the contributor certifies compliance to the Developer's Certificate
>   of Origin 1.1.  https://developercertificate.org
> =========

[...]

> The patch series has one patch per package along with a few patches
> to update the license information in the root of the edk2 repository
> as described in the RFC V3.

[...]

> f9d59ccdc5 OvmfPkg: Change License.txt from 2-Clause BSD to BSD+Patent

[...]

> e39d07266d OvmfPkg: Replace BSD License with BSD+Patent License

The series now has two patches for OvmfPkg. Did you intend to squash
these into one (to match the original intent quoted above, i.e. one
patch per pkg), or did you intend to keep both patches separate?

If you meant to squash them, can you please do that and push a v3?

If the patches should be separate, I'll review both patches as they are
(plus recheck ArmVirtPkg).

Thanks!
Laszlo


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH V2] Change EDK II to BSD+Patent License
  2019-03-25 12:11 ` Laszlo Ersek
@ 2019-03-25 17:24   ` Kinney, Michael D
  0 siblings, 0 replies; 9+ messages in thread
From: Kinney, Michael D @ 2019-03-25 17:24 UTC (permalink / raw)
  To: Laszlo Ersek, edk2-devel@lists.01.org, Kinney, Michael D

Hi Laszlo,

The update to OvmfPkg/License.txt was manual, similar
to Licence.txt and Readme.md in the root of the edk2
repository, so I do not plan to squash.

Thanks,

Mike

> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek@redhat.com]
> Sent: Monday, March 25, 2019 5:12 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>;
> edk2-devel@lists.01.org
> Subject: Re: [edk2] [PATCH V2] Change EDK II to
> BSD+Patent License
> 
> Hi Mike,
> 
> On 03/23/19 03:25, Kinney, Michael D wrote:
> > Hello,
> >
> > New in V2
> > =========
> > * Remove Cc lines from commit messages
> > * Remove branch reference from commit messages
> > * Change license in 2 files missed in OvmfPkg
> > * Update OvmfPkg/License.txt to BSD+Patent as the
> default license
> > * Move the portions of Contributions.txt in the root
> of edk2 to
> >   Readme.md in the root of edk2 that describe how to
> contribute
> >   along with the commit message format.
> > * Add to Readme.md in the root of edk2 that Signed-
> off-by means that
> >   the contributor certifies compliance to the
> Developer's Certificate
> >   of Origin 1.1.  https://developercertificate.org
> > =========
> 
> [...]
> 
> > The patch series has one patch per package along with
> a few patches
> > to update the license information in the root of the
> edk2 repository
> > as described in the RFC V3.
> 
> [...]
> 
> > f9d59ccdc5 OvmfPkg: Change License.txt from 2-Clause
> BSD to BSD+Patent
> 
> [...]
> 
> > e39d07266d OvmfPkg: Replace BSD License with
> BSD+Patent License
> 
> The series now has two patches for OvmfPkg. Did you
> intend to squash
> these into one (to match the original intent quoted
> above, i.e. one
> patch per pkg), or did you intend to keep both patches
> separate?
> 
> If you meant to squash them, can you please do that and
> push a v3?
> 
> If the patches should be separate, I'll review both
> patches as they are
> (plus recheck ArmVirtPkg).
> 
> Thanks!
> Laszlo

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH V2] Change EDK II to BSD+Patent License
  2019-03-23  2:25 [PATCH V2] Change EDK II to BSD+Patent License Kinney, Michael D
  2019-03-25 12:11 ` Laszlo Ersek
@ 2019-03-26 18:08 ` Leif Lindholm
  2019-03-26 18:21   ` Kinney, Michael D
  2019-04-03 11:43 ` Laszlo Ersek
       [not found] ` <74D8A39837DF1E4DA445A8C0B3885C503F5C2D92@shsmsx102.ccr.corp.intel.com>
  3 siblings, 1 reply; 9+ messages in thread
From: Leif Lindholm @ 2019-03-26 18:08 UTC (permalink / raw)
  To: Kinney, Michael D; +Cc: edk2-devel@lists.01.org

Hi Mike,

First of all - now the March table tag was made (and I'm back from
holiday), I had planned to do the move of BeagleBoardPkg and
Omap35xxPkg to edk2-platforms.

Would you prefer me to put that on hold, or should we drop those
changes from this set and worry about those if/when we get around to
relicensing edk2-platforms too?

For the changes to ArmPkg, ArmPlatformPkg, EmbeddedPkg(,
BeagleBoardPkg, Omap35xxPkg):
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

For the changes to edk2:
License.txt - could the commit message describe where the new text is
	      from (as an implicit way of explaining why the
	      layout/bulleting has changed in the portion that is
	      otherwise content-wise identical)?
            - (I'm sorry, I should just keep quiet, but...)
              The copyright lines at the top of the Licence.txt file
              have been bugging me since day 1. Can we drop them?
	      Clearly none of these organisations hold copyright over
	      either the old or the new license.

I'll just add that my wording for the Signed-off-by was just a
meant as a starting point and I'd be happy to see it improved.

But from my end, all edk2: patches other than
"edk2: Change License.txt from 2-Clause BSD to BSD+Patent":
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>

/
    Leif

On Sat, Mar 23, 2019 at 02:25:15AM +0000, Kinney, Michael D wrote:
> Hello,
> 
> New in V2
> =========
> * Remove Cc lines from commit messages
> * Remove branch reference from commit messages
> * Change license in 2 files missed in OvmfPkg
> * Update OvmfPkg/License.txt to BSD+Patent as the default license
> * Move the portions of Contributions.txt in the root of edk2 to
>   Readme.md in the root of edk2 that describe how to contribute
>   along with the commit message format.
> * Add to Readme.md in the root of edk2 that Signed-off-by means that
>   the contributor certifies compliance to the Developer's Certificate
>   of Origin 1.1.  https://developercertificate.org
> =========
> 
> 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 the patch series for review on the following branch using
> edk2-stable201903 as the base for the patch series.  
> 
>   https://github.com/mdkinney/edk2/tree/Bug_1373_BsdPatentLicense_V2
> 
> The commits in patch series can be viewed here:
> 
>   https://github.com/mdkinney/edk2/commits/Bug_1373_BsdPatentLicense_V2
> 
> The patch series has one patch per package along with a few patches
> to update the license information in the root of the edk2 repository
> as described in the RFC V3.
> 
> Due to the size of the patch series, I prefer to not send the
> patch emails.  Instead, please perform code reviews using content
> from the branch.
> 
> All EDK II package 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. 
> 
> Feedback and Reviewed-by emails should identify the patch the feedback
> applies using the patch summary listed below.  The goal is to complete
> all reviews to support the commit of these patches on April 9, 2019.
>  
> 2a7d2c56bc edk2: Remove Contributions.txt and update Readme.md
> f9d59ccdc5 OvmfPkg: Change License.txt from 2-Clause BSD to BSD+Patent
> ce3fbf929e StdLibPrivateInternalFiles: Replace BSD License with BSD+Patent License
> aa8a3692c7 StdLib: Replace BSD License with BSD+Patent License
> 2dfbe1e1ee AppPkg: Replace BSD License with BSD+Patent License
> b2161f6dd8 Vlv2TbltDevicePkg: Replace BSD License with BSD+Patent License
> 3688c33755 Vlv2DeviceRefCodePkg: Replace BSD License with BSD+Patent License
> 8170308c98 UefiCpuPkg: Replace BSD License with BSD+Patent License
> 4b68832cdc StandaloneMmPkg: Replace BSD License with BSD+Patent License
> 327dc18122 SourceLevelDebugPkg: Replace BSD License with BSD+Patent License
> 6c4c506a5e SignedCapsulePkg: Replace BSD License with BSD+Patent License
> 2fdd514aff ShellPkg: Replace BSD License with BSD+Patent License
> c667724cb9 ShellBinPkg: Replace BSD License with BSD+Patent License
> 76f9e555b8 SecurityPkg: Replace BSD License with BSD+Patent License
> 89a588c1be QuarkSocPkg: Replace BSD License with BSD+Patent License
> 454f280e0c QuarkPlatformPkg: Replace BSD License with BSD+Patent License
> 935319af35 PcAtChipsetPkg: Replace BSD License with BSD+Patent License
> e39d07266d OvmfPkg: Replace BSD License with BSD+Patent License
> 24d6ba8a25 OptionRomPkg: Replace BSD License with BSD+Patent License
> 5c4a074eed Omap35xxPkg: Replace BSD License with BSD+Patent License
> dde19815b3 Nt32Pkg: Replace BSD License with BSD+Patent License
> acf2e84beb NetworkPkg: Replace BSD License with BSD+Patent License
> 3e404f30b6 MdePkg: Replace BSD License with BSD+Patent License
> 6e01bf6764 MdeModulePkg: Replace BSD License with BSD+Patent License
> 36322b1b8c IntelSiliconPkg: Replace BSD License with BSD+Patent License
> f5f9fa8b95 IntelFspWrapperPkg: Replace BSD License with BSD+Patent License
> 39725dc5e3 IntelFspPkg: Replace BSD License with BSD+Patent License
> ba8a9e3e1d IntelFsp2WrapperPkg: Replace BSD License with BSD+Patent License
> a61295eeda IntelFsp2Pkg: Replace BSD License with BSD+Patent License
> cd7bd24c6d IntelFrameworkPkg: Replace BSD License with BSD+Patent License
> e9f7710bb6 IntelFrameworkModulePkg: Replace BSD License with BSD+Patent License
> 487e96a6b4 FmpDevicePkg: Replace BSD License with BSD+Patent License
> 3cfd1b98e4 FatPkg: Replace BSD License with BSD+Patent License
> f9fc278ea1 EmulatorPkg: Replace BSD License with BSD+Patent License
> 1a26764621 EmbeddedPkg: Replace BSD License with BSD+Patent License
> aa402ac906 EdkCompatibilityPkg: Replace BSD License with BSD+Patent License
> 3c6df91d7c DynamicTablesPkg: Replace BSD License with BSD+Patent License
> 6389a5b4d5 CryptoPkg: Replace BSD License with BSD+Patent License
> 0065fa2d9f CorebootPayloadPkg: Replace BSD License with BSD+Patent License
> 26d7dbf868 CorebootModulePkg: Replace BSD License with BSD+Patent License
> b1ebd76234 BeagleBoardPkg: Replace BSD License with BSD+Patent License
> f23540ea65 ArmVirtPkg: Replace BSD License with BSD+Patent License
> 054b667071 ArmPlatformPkg: Replace BSD License with BSD+Patent License
> 5128ec1897 ArmPkg: Replace BSD License with BSD+Patent License
> 3b7fd23df9 BaseTools: Replace BSD License with BSD+Patent License
> aa5e7ad3ef edk2: Replace BSD License with BSD+Patent License
> fdcf6f00c7 edk2: Change License.txt from 2-Clause BSD to BSD+Patent
> 831e2096e8 edk2: Add License-History.txt
> 
> Best regards,
> 
> Mike
> 
> > -----Original Message-----
> > From: Kinney, Michael D
> > Sent: Friday, March 22, 2019 6:49 PM
> > To: edk2-devel@lists.01.org; Kinney, Michael D
> > <michael.d.kinney@intel.com>
> > Subject: [RFC v3] Change EDK II to BSD+Patent License
> > 
> > Hello,
> > 
> > Based on review of the RFC V2, there are some updates
> > required to
> > Readme.md in the root of the edk2 repository.
> > 
> > Changes for V3
> > ===============
> > * Move the portions of Contributions.txt in the root of
> > edk2 to
> >   Readme.md in the root of edk2 that describe how to
> > contribute
> >   along with the commit message format.
> > 
> > * Add to Readme.md in the root of edk2 that Signed-off-by
> > means that
> >   the contributor certifies compliance to the Developer's
> > Certificate
> >   of Origin 1.1.  https://developercertificate.org
> > 
> > Changes for V2
> > ===============
> > * Replace 2-Clause BSD License in file headers with SPDX-
> > License-Identifier
> >   statement.  This reduces the size of the file headers
> > and the size
> >   of the patches for this change.  Based on the following
> > post:
> > 
> >   https://01.org/blogs/jc415/2018/open-source-hacks-one-
> > question-interviews-open-source-experts-how-use-spdx-
> > headers
> > 
> > * Update License.txt in root of edk2 before changing file
> > headers.
> > * Fix minor typos
> > ===============
> > 
> > This RFC follows up on the proposal from Mark Doran to
> > change the
> > EDK II Project to a BSD+Patent License.
> > 
> > 	https://lists.01.org/pipermail/edk2-devel/2019-
> > February/036260.html
> > 
> > The review period for this license change is 30 days.  If
> > there is no
> > unresolved feedback on April 9, 2019, then commits of the
> > license change
> > patches will begin on April 9, 2019.
> > 
> >   ** Please provide feedback on the proposal by Monday
> > April 8, 2019. **
> > 
> > Feedback can be sent to edk2-devel at lists.01.org, the
> > EDK II community
> > manager or any of the EDK II stewards.
> > 
> >   * Stephano Cetola <stephano.cetola at intel.com>
> > Community Manager
> >   * Leif Lindholm   <leif.lindholm at linaro.org>
> > Steward
> >   * Andrew Fish     <afish at apple.com>
> > Steward
> >   * Laszlo Ersek    <lersek at redhat.com>
> > Steward
> >   * Michael Kinney  <michael.d.kinney at intel.com>
> > Steward
> > 
> > The goal is to convert all of the files in the edk2
> > repository that are
> > currently covered by the 2-Clause BSD License and the
> > TianoCore
> > Contribution Agreement to a BSD+Patent License.
> > 
> > I will be following up with pointers to public GitHub
> > branches that
> > contain the set of changes to the edk2 repository for
> > review.
> > 
> > The proposal is to perform this change to edk2/master in
> > the steps listed
> > below. The license change will not be applied to any of
> > the other existing
> > branches in the edk2 repository.
> > 
> > 1) Add a License-History.txt file to the root of the edk2
> > repository that
> >    contains the 2-Clause BSD License and the TianoCore
> > Contribution
> >    Agreement along with the details on the change to the
> > BSD+Patent License.
> > 
> > 2) Change License.txt in the root of the edk2 repository
> > from a 2-Clause
> >    BSD License to the BSD+Patent License. The following
> > is the link to the
> >    BSD+Patent License and the new License.txt file
> > contents.
> > 
> >    https://opensource.org/licenses/BSDplusPatent
> > 
> > 
> > =========================================================
> > =============
> >    Redistribution and use in source and binary forms,
> > with or without
> >    modification, are permitted provided that the
> > following conditions are met:
> > 
> >    1. Redistributions of source code must retain the
> > above copyright notice,
> >       this list of conditions and the following
> > disclaimer.
> > 
> >    2. Redistributions in binary form must reproduce the
> > above copyright notice,
> >       this list of conditions and the following
> > disclaimer in the documentation
> >       and/or other materials provided with the
> > distribution.
> > 
> >    Subject to the terms and conditions of this license,
> > each copyright holder
> >    and contributor hereby grants to those receiving
> > rights under this license
> >    a perpetual, worldwide, non-exclusive, no-charge,
> > royalty-free, irrevocable
> >    (except for failure to satisfy the conditions of this
> > license) patent
> >    license to make, have made, use, offer to sell, sell,
> > import, and otherwise
> >    transfer this software, where such license applies
> > only to those patent
> >    claims, already acquired or hereafter acquired,
> > licensable by such copyright
> >    holder or contributor that are necessarily infringed
> > by:
> > 
> >    (a) their Contribution(s) (the licensed copyrights of
> > copyright holders and
> >        non-copyrightable additions of contributors, in
> > source or binary form)
> >        alone; or
> > 
> >    (b) combination of their Contribution(s) with the work
> > of authorship to
> >        which such Contribution(s) was added by such
> > copyright holder or
> >        contributor, if, at the time the Contribution is
> > added, such addition
> >        causes such combination to be necessarily
> > infringed. The patent license
> >        shall not apply to any other combinations which
> > include the
> >        Contribution.
> > 
> >    Except as expressly stated above, no rights or
> > licenses from any copyright
> >    holder or contributor is granted under this license,
> > whether expressly, by
> >    implication, estoppel or otherwise.
> > 
> >    DISCLAIMER
> > 
> >    THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
> > CONTRIBUTORS "AS IS"
> >    AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT
> > NOT LIMITED TO, THE
> >    IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> > A PARTICULAR PURPOSE
> >    ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> > HOLDERS OR CONTRIBUTORS BE
> >    LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
> > EXEMPLARY, OR
> >    CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
> > PROCUREMENT OF
> >    SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
> > PROFITS; OR BUSINESS
> >    INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
> > LIABILITY, WHETHER IN
> >    CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
> > NEGLIGENCE OR OTHERWISE)
> >    ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE,
> > EVEN IF ADVISED OF THE
> >    POSSIBILITY OF SUCH DAMAGE.
> > 
> > =========================================================
> > =============
> > 
> > 3) Change all files currently covered by a 2-Clause BSD
> > License and the
> >    TianoCore Contribution Agreement to a BSD+Patent
> > License using the
> >    following SPDX-License-Identifier statement:
> > 
> >        SPDX-License-Identifier: BSD-2-Clause-Patent
> > 
> >    The use of SPDX-License-Identifier statement is based
> > on the following:
> > 
> >         https://01.org/blogs/jc415/2018/open-source-
> > hacks-one-question-interviews-open-source-experts-how-
> > use-spdx-headers
> > 
> > 4) Update Readme.md in the root of the edk2 repository to
> > state that content
> >    is covered by a BSD+Patent License.  Also state that
> > the BSD+Patent License
> >    is the preferred license for the EDK II project.
> > 
> >   a) Move the portions of Contributions.txt in the root
> > of the edk2 repository
> >      Readme.md in the root of edk2 repository that
> > describe how to contribute
> >      along with the commit message format.
> > 
> >   b) Add the following to Readme.md in the root of edk2
> > repository:
> > 
> >      # Developer Certificate of Origin
> > 
> >      Your change description should use the standard
> > format for a
> >      commit message, and must include your `Signed-off-
> > by` signature.
> > 
> >      In order to keep track of who did what, all patches
> > contributed must
> >      include a statement that to the best of the
> > contributor's knowledge
> >      they have the right to contribute it under the
> > specified license.
> > 
> >      The test for this is as specified in the
> > [Developer's Certificate of
> >      Origin (DCO)
> > 1.1](https://developercertificate.org/). The contributor
> >      certifies compliance by adding a line saying
> > 
> >        Signed-off-by: Developer Name
> > <developer@example.org>
> > 
> >      where `Developer Name` is the contributor's real
> > name, and the email
> >      address is one the developer is reachable through at
> > the time of
> >      contributing.
> > 
> >      ```
> >      Developer's Certificate of Origin 1.1
> > 
> >      By making a contribution to this project, I certify
> > that:
> > 
> >       (a) The contribution was created in whole or in
> > part by me and I
> >          have the right to submit it under the open
> > source license
> >          indicated in the file; or
> > 
> >       (b) The contribution is based upon previous work
> > that, to the best
> >          of my knowledge, is covered under an appropriate
> > open source
> >          license and I have the right under that license
> > to submit that
> >          work with modifications, whether created in
> > whole or in part
> >          by me, under the same open source license
> > (unless I am
> >          permitted to submit under a different license),
> > as indicated
> >          in the file; or
> > 
> >       (c) The contribution was provided directly to me by
> > some other
> >          person who certified (a), (b) or (c) and I have
> > not modified
> >          it.
> > 
> >       (d) I understand and agree that this project and
> > the contribution
> >          are public and that a record of the contribution
> > (including all
> >          personal information I submit with it, including
> > my sign-off) is
> >          maintained indefinitely and may be redistributed
> > consistent with
> >          this project or the open source license(s)
> > involved.
> >      ```
> > 
> > 5) Remove the Contributions.txt file from the root of the
> > edk2 repository
> >    that contains the TianoCore Contribution Agreement.
> > 
> > 6) Update all documentation to state that content
> > submitted under the
> >    BSD+Patent License no longer requires the Tianocore
> > Contribution
> >    Agreement which means the following line is not
> > required in commit
> >    messages for changes to files that are covered by a
> > BSD+Patent License.
> > 
> >        Contributed-under: TianoCore Contribution
> > Agreement 1.1
> > 
> > 7) Create Wiki page(s) that provide the details of the
> > BSD+Patent License
> >    change and provides the status of the license change
> > for each TianoCore
> >    repository and package.
> > 
> > Once the conversion of the edk2 repository is complete,
> > work will begin
> > on the other repositories in the TianoCore project.
> > 
> > Best regards,
> > 
> > Mike
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH V2] Change EDK II to BSD+Patent License
  2019-03-26 18:08 ` Leif Lindholm
@ 2019-03-26 18:21   ` Kinney, Michael D
  2019-03-26 18:54     ` Leif Lindholm
  0 siblings, 1 reply; 9+ messages in thread
From: Kinney, Michael D @ 2019-03-26 18:21 UTC (permalink / raw)
  To: Leif Lindholm, Kinney, Michael D; +Cc: edk2-devel@lists.01.org

Hi Leif,

Thanks for the reviews.  Responses below.

Mike

> -----Original Message-----
> From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> Sent: Tuesday, March 26, 2019 11:09 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: edk2-devel@lists.01.org
> Subject: Re: [edk2] [PATCH V2] Change EDK II to
> BSD+Patent License
> 
> Hi Mike,
> 
> First of all - now the March table tag was made (and I'm
> back from
> holiday), I had planned to do the move of BeagleBoardPkg
> and
> Omap35xxPkg to edk2-platforms.
> 
> Would you prefer me to put that on hold, or should we
> drop those
> changes from this set and worry about those if/when we
> get around to
> relicensing edk2-platforms too?

I do not have a strong opinion on when those packages
move to edk2-platforms.

I am holding off on other package moves I am involved in
until after the license change.

> 
> For the changes to ArmPkg, ArmPlatformPkg, EmbeddedPkg(,
> BeagleBoardPkg, Omap35xxPkg):
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> 

Thank you!

> For the changes to edk2:
> License.txt - could the commit message describe where
> the new text is
> 	      from (as an implicit way of explaining why the
> 	      layout/bulleting has changed in the portion
> that is
> 	      otherwise content-wise identical)?

I do not follow what you want updated here.  Which 
commit and what would you like the message changed to?

>             - (I'm sorry, I should just keep quiet,
> but...)
>               The copyright lines at the top of the
> Licence.txt file
>               have been bugging me since day 1. Can we
> drop them?
> 	      Clearly none of these organisations hold
> copyright over
> 	      either the old or the new license.
> 

The copyrights at the top of that file were inherited
from the packages when License.txt used to be in each
package.  If you think it is appropriate to remove them
I am happy to do that as part of this series.  Does the
same comment apply to License.txt in OvmfPkg?

> I'll just add that my wording for the Signed-off-by was
> just a
> meant as a starting point and I'd be happy to see it
> improved.

Please let me know if you have any suggestions here.

> 
> But from my end, all edk2: patches other than
> "edk2: Change License.txt from 2-Clause BSD to
> BSD+Patent":
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> 
> /
>     Leif
> 
> On Sat, Mar 23, 2019 at 02:25:15AM +0000, Kinney,
> Michael D wrote:
> > Hello,
> >
> > New in V2
> > =========
> > * Remove Cc lines from commit messages
> > * Remove branch reference from commit messages
> > * Change license in 2 files missed in OvmfPkg
> > * Update OvmfPkg/License.txt to BSD+Patent as the
> default license
> > * Move the portions of Contributions.txt in the root
> of edk2 to
> >   Readme.md in the root of edk2 that describe how to
> contribute
> >   along with the commit message format.
> > * Add to Readme.md in the root of edk2 that Signed-
> off-by means that
> >   the contributor certifies compliance to the
> Developer's Certificate
> >   of Origin 1.1.  https://developercertificate.org
> > =========
> >
> > 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 the patch series for review on the
> following branch using
> > edk2-stable201903 as the base for the patch series.
> >
> >
> https://github.com/mdkinney/edk2/tree/Bug_1373_BsdPatent
> License_V2
> >
> > The commits in patch series can be viewed here:
> >
> >
> https://github.com/mdkinney/edk2/commits/Bug_1373_BsdPat
> entLicense_V2
> >
> > The patch series has one patch per package along with
> a few patches
> > to update the license information in the root of the
> edk2 repository
> > as described in the RFC V3.
> >
> > Due to the size of the patch series, I prefer to not
> send the
> > patch emails.  Instead, please perform code reviews
> using content
> > from the branch.
> >
> > All EDK II package 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.
> >
> > Feedback and Reviewed-by emails should identify the
> patch the feedback
> > applies using the patch summary listed below.  The
> goal is to complete
> > all reviews to support the commit of these patches on
> April 9, 2019.
> >
> > 2a7d2c56bc edk2: Remove Contributions.txt and update
> Readme.md
> > f9d59ccdc5 OvmfPkg: Change License.txt from 2-Clause
> BSD to BSD+Patent
> > ce3fbf929e StdLibPrivateInternalFiles: Replace BSD
> License with BSD+Patent License
> > aa8a3692c7 StdLib: Replace BSD License with BSD+Patent
> License
> > 2dfbe1e1ee AppPkg: Replace BSD License with BSD+Patent
> License
> > b2161f6dd8 Vlv2TbltDevicePkg: Replace BSD License with
> BSD+Patent License
> > 3688c33755 Vlv2DeviceRefCodePkg: Replace BSD License
> with BSD+Patent License
> > 8170308c98 UefiCpuPkg: Replace BSD License with
> BSD+Patent License
> > 4b68832cdc StandaloneMmPkg: Replace BSD License with
> BSD+Patent License
> > 327dc18122 SourceLevelDebugPkg: Replace BSD License
> with BSD+Patent License
> > 6c4c506a5e SignedCapsulePkg: Replace BSD License with
> BSD+Patent License
> > 2fdd514aff ShellPkg: Replace BSD License with
> BSD+Patent License
> > c667724cb9 ShellBinPkg: Replace BSD License with
> BSD+Patent License
> > 76f9e555b8 SecurityPkg: Replace BSD License with
> BSD+Patent License
> > 89a588c1be QuarkSocPkg: Replace BSD License with
> BSD+Patent License
> > 454f280e0c QuarkPlatformPkg: Replace BSD License with
> BSD+Patent License
> > 935319af35 PcAtChipsetPkg: Replace BSD License with
> BSD+Patent License
> > e39d07266d OvmfPkg: Replace BSD License with
> BSD+Patent License
> > 24d6ba8a25 OptionRomPkg: Replace BSD License with
> BSD+Patent License
> > 5c4a074eed Omap35xxPkg: Replace BSD License with
> BSD+Patent License
> > dde19815b3 Nt32Pkg: Replace BSD License with
> BSD+Patent License
> > acf2e84beb NetworkPkg: Replace BSD License with
> BSD+Patent License
> > 3e404f30b6 MdePkg: Replace BSD License with BSD+Patent
> License
> > 6e01bf6764 MdeModulePkg: Replace BSD License with
> BSD+Patent License
> > 36322b1b8c IntelSiliconPkg: Replace BSD License with
> BSD+Patent License
> > f5f9fa8b95 IntelFspWrapperPkg: Replace BSD License
> with BSD+Patent License
> > 39725dc5e3 IntelFspPkg: Replace BSD License with
> BSD+Patent License
> > ba8a9e3e1d IntelFsp2WrapperPkg: Replace BSD License
> with BSD+Patent License
> > a61295eeda IntelFsp2Pkg: Replace BSD License with
> BSD+Patent License
> > cd7bd24c6d IntelFrameworkPkg: Replace BSD License with
> BSD+Patent License
> > e9f7710bb6 IntelFrameworkModulePkg: Replace BSD
> License with BSD+Patent License
> > 487e96a6b4 FmpDevicePkg: Replace BSD License with
> BSD+Patent License
> > 3cfd1b98e4 FatPkg: Replace BSD License with BSD+Patent
> License
> > f9fc278ea1 EmulatorPkg: Replace BSD License with
> BSD+Patent License
> > 1a26764621 EmbeddedPkg: Replace BSD License with
> BSD+Patent License
> > aa402ac906 EdkCompatibilityPkg: Replace BSD License
> with BSD+Patent License
> > 3c6df91d7c DynamicTablesPkg: Replace BSD License with
> BSD+Patent License
> > 6389a5b4d5 CryptoPkg: Replace BSD License with
> BSD+Patent License
> > 0065fa2d9f CorebootPayloadPkg: Replace BSD License
> with BSD+Patent License
> > 26d7dbf868 CorebootModulePkg: Replace BSD License with
> BSD+Patent License
> > b1ebd76234 BeagleBoardPkg: Replace BSD License with
> BSD+Patent License
> > f23540ea65 ArmVirtPkg: Replace BSD License with
> BSD+Patent License
> > 054b667071 ArmPlatformPkg: Replace BSD License with
> BSD+Patent License
> > 5128ec1897 ArmPkg: Replace BSD License with BSD+Patent
> License
> > 3b7fd23df9 BaseTools: Replace BSD License with
> BSD+Patent License
> > aa5e7ad3ef edk2: Replace BSD License with BSD+Patent
> License
> > fdcf6f00c7 edk2: Change License.txt from 2-Clause BSD
> to BSD+Patent
> > 831e2096e8 edk2: Add License-History.txt
> >
> > Best regards,
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: Kinney, Michael D
> > > Sent: Friday, March 22, 2019 6:49 PM
> > > To: edk2-devel@lists.01.org; Kinney, Michael D
> > > <michael.d.kinney@intel.com>
> > > Subject: [RFC v3] Change EDK II to BSD+Patent
> License
> > >
> > > Hello,
> > >
> > > Based on review of the RFC V2, there are some
> updates
> > > required to
> > > Readme.md in the root of the edk2 repository.
> > >
> > > Changes for V3
> > > ===============
> > > * Move the portions of Contributions.txt in the root
> of
> > > edk2 to
> > >   Readme.md in the root of edk2 that describe how to
> > > contribute
> > >   along with the commit message format.
> > >
> > > * Add to Readme.md in the root of edk2 that Signed-
> off-by
> > > means that
> > >   the contributor certifies compliance to the
> Developer's
> > > Certificate
> > >   of Origin 1.1.  https://developercertificate.org
> > >
> > > Changes for V2
> > > ===============
> > > * Replace 2-Clause BSD License in file headers with
> SPDX-
> > > License-Identifier
> > >   statement.  This reduces the size of the file
> headers
> > > and the size
> > >   of the patches for this change.  Based on the
> following
> > > post:
> > >
> > >   https://01.org/blogs/jc415/2018/open-source-hacks-
> one-
> > > question-interviews-open-source-experts-how-use-
> spdx-
> > > headers
> > >
> > > * Update License.txt in root of edk2 before changing
> file
> > > headers.
> > > * Fix minor typos
> > > ===============
> > >
> > > This RFC follows up on the proposal from Mark Doran
> to
> > > change the
> > > EDK II Project to a BSD+Patent License.
> > >
> > > 	https://lists.01.org/pipermail/edk2-devel/2019-
> > > February/036260.html
> > >
> > > The review period for this license change is 30
> days.  If
> > > there is no
> > > unresolved feedback on April 9, 2019, then commits
> of the
> > > license change
> > > patches will begin on April 9, 2019.
> > >
> > >   ** Please provide feedback on the proposal by
> Monday
> > > April 8, 2019. **
> > >
> > > Feedback can be sent to edk2-devel at lists.01.org,
> the
> > > EDK II community
> > > manager or any of the EDK II stewards.
> > >
> > >   * Stephano Cetola <stephano.cetola at intel.com>
> > > Community Manager
> > >   * Leif Lindholm   <leif.lindholm at linaro.org>
> > > Steward
> > >   * Andrew Fish     <afish at apple.com>
> > > Steward
> > >   * Laszlo Ersek    <lersek at redhat.com>
> > > Steward
> > >   * Michael Kinney  <michael.d.kinney at intel.com>
> > > Steward
> > >
> > > The goal is to convert all of the files in the edk2
> > > repository that are
> > > currently covered by the 2-Clause BSD License and
> the
> > > TianoCore
> > > Contribution Agreement to a BSD+Patent License.
> > >
> > > I will be following up with pointers to public
> GitHub
> > > branches that
> > > contain the set of changes to the edk2 repository
> for
> > > review.
> > >
> > > The proposal is to perform this change to
> edk2/master in
> > > the steps listed
> > > below. The license change will not be applied to any
> of
> > > the other existing
> > > branches in the edk2 repository.
> > >
> > > 1) Add a License-History.txt file to the root of the
> edk2
> > > repository that
> > >    contains the 2-Clause BSD License and the
> TianoCore
> > > Contribution
> > >    Agreement along with the details on the change to
> the
> > > BSD+Patent License.
> > >
> > > 2) Change License.txt in the root of the edk2
> repository
> > > from a 2-Clause
> > >    BSD License to the BSD+Patent License. The
> following
> > > is the link to the
> > >    BSD+Patent License and the new License.txt file
> > > contents.
> > >
> > >    https://opensource.org/licenses/BSDplusPatent
> > >
> > >
> > >
> ========================================================
> =
> > > =============
> > >    Redistribution and use in source and binary
> forms,
> > > with or without
> > >    modification, are permitted provided that the
> > > following conditions are met:
> > >
> > >    1. Redistributions of source code must retain the
> > > above copyright notice,
> > >       this list of conditions and the following
> > > disclaimer.
> > >
> > >    2. Redistributions in binary form must reproduce
> the
> > > above copyright notice,
> > >       this list of conditions and the following
> > > disclaimer in the documentation
> > >       and/or other materials provided with the
> > > distribution.
> > >
> > >    Subject to the terms and conditions of this
> license,
> > > each copyright holder
> > >    and contributor hereby grants to those receiving
> > > rights under this license
> > >    a perpetual, worldwide, non-exclusive, no-charge,
> > > royalty-free, irrevocable
> > >    (except for failure to satisfy the conditions of
> this
> > > license) patent
> > >    license to make, have made, use, offer to sell,
> sell,
> > > import, and otherwise
> > >    transfer this software, where such license
> applies
> > > only to those patent
> > >    claims, already acquired or hereafter acquired,
> > > licensable by such copyright
> > >    holder or contributor that are necessarily
> infringed
> > > by:
> > >
> > >    (a) their Contribution(s) (the licensed
> copyrights of
> > > copyright holders and
> > >        non-copyrightable additions of contributors,
> in
> > > source or binary form)
> > >        alone; or
> > >
> > >    (b) combination of their Contribution(s) with the
> work
> > > of authorship to
> > >        which such Contribution(s) was added by such
> > > copyright holder or
> > >        contributor, if, at the time the Contribution
> is
> > > added, such addition
> > >        causes such combination to be necessarily
> > > infringed. The patent license
> > >        shall not apply to any other combinations
> which
> > > include the
> > >        Contribution.
> > >
> > >    Except as expressly stated above, no rights or
> > > licenses from any copyright
> > >    holder or contributor is granted under this
> license,
> > > whether expressly, by
> > >    implication, estoppel or otherwise.
> > >
> > >    DISCLAIMER
> > >
> > >    THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT
> HOLDERS AND
> > > CONTRIBUTORS "AS IS"
> > >    AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING,
> BUT
> > > NOT LIMITED TO, THE
> > >    IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
> FOR
> > > A PARTICULAR PURPOSE
> > >    ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> > > HOLDERS OR CONTRIBUTORS BE
> > >    LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> SPECIAL,
> > > EXEMPLARY, OR
> > >    CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED
> TO,
> > > PROCUREMENT OF
> > >    SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA,
> OR
> > > PROFITS; OR BUSINESS
> > >    INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
> > > LIABILITY, WHETHER IN
> > >    CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
> > > NEGLIGENCE OR OTHERWISE)
> > >    ARISING IN ANY WAY OUT OF THE USE OF THIS
> SOFTWARE,
> > > EVEN IF ADVISED OF THE
> > >    POSSIBILITY OF SUCH DAMAGE.
> > >
> > >
> ========================================================
> =
> > > =============
> > >
> > > 3) Change all files currently covered by a 2-Clause
> BSD
> > > License and the
> > >    TianoCore Contribution Agreement to a BSD+Patent
> > > License using the
> > >    following SPDX-License-Identifier statement:
> > >
> > >        SPDX-License-Identifier: BSD-2-Clause-Patent
> > >
> > >    The use of SPDX-License-Identifier statement is
> based
> > > on the following:
> > >
> > >         https://01.org/blogs/jc415/2018/open-source-
> > > hacks-one-question-interviews-open-source-experts-
> how-
> > > use-spdx-headers
> > >
> > > 4) Update Readme.md in the root of the edk2
> repository to
> > > state that content
> > >    is covered by a BSD+Patent License.  Also state
> that
> > > the BSD+Patent License
> > >    is the preferred license for the EDK II project.
> > >
> > >   a) Move the portions of Contributions.txt in the
> root
> > > of the edk2 repository
> > >      Readme.md in the root of edk2 repository that
> > > describe how to contribute
> > >      along with the commit message format.
> > >
> > >   b) Add the following to Readme.md in the root of
> edk2
> > > repository:
> > >
> > >      # Developer Certificate of Origin
> > >
> > >      Your change description should use the standard
> > > format for a
> > >      commit message, and must include your `Signed-
> off-
> > > by` signature.
> > >
> > >      In order to keep track of who did what, all
> patches
> > > contributed must
> > >      include a statement that to the best of the
> > > contributor's knowledge
> > >      they have the right to contribute it under the
> > > specified license.
> > >
> > >      The test for this is as specified in the
> > > [Developer's Certificate of
> > >      Origin (DCO)
> > > 1.1](https://developercertificate.org/). The
> contributor
> > >      certifies compliance by adding a line saying
> > >
> > >        Signed-off-by: Developer Name
> > > <developer@example.org>
> > >
> > >      where `Developer Name` is the contributor's
> real
> > > name, and the email
> > >      address is one the developer is reachable
> through at
> > > the time of
> > >      contributing.
> > >
> > >      ```
> > >      Developer's Certificate of Origin 1.1
> > >
> > >      By making a contribution to this project, I
> certify
> > > that:
> > >
> > >       (a) The contribution was created in whole or
> in
> > > part by me and I
> > >          have the right to submit it under the open
> > > source license
> > >          indicated in the file; or
> > >
> > >       (b) The contribution is based upon previous
> work
> > > that, to the best
> > >          of my knowledge, is covered under an
> appropriate
> > > open source
> > >          license and I have the right under that
> license
> > > to submit that
> > >          work with modifications, whether created in
> > > whole or in part
> > >          by me, under the same open source license
> > > (unless I am
> > >          permitted to submit under a different
> license),
> > > as indicated
> > >          in the file; or
> > >
> > >       (c) The contribution was provided directly to
> me by
> > > some other
> > >          person who certified (a), (b) or (c) and I
> have
> > > not modified
> > >          it.
> > >
> > >       (d) I understand and agree that this project
> and
> > > the contribution
> > >          are public and that a record of the
> contribution
> > > (including all
> > >          personal information I submit with it,
> including
> > > my sign-off) is
> > >          maintained indefinitely and may be
> redistributed
> > > consistent with
> > >          this project or the open source license(s)
> > > involved.
> > >      ```
> > >
> > > 5) Remove the Contributions.txt file from the root
> of the
> > > edk2 repository
> > >    that contains the TianoCore Contribution
> Agreement.
> > >
> > > 6) Update all documentation to state that content
> > > submitted under the
> > >    BSD+Patent License no longer requires the
> Tianocore
> > > Contribution
> > >    Agreement which means the following line is not
> > > required in commit
> > >    messages for changes to files that are covered by
> a
> > > BSD+Patent License.
> > >
> > >        Contributed-under: TianoCore Contribution
> > > Agreement 1.1
> > >
> > > 7) Create Wiki page(s) that provide the details of
> the
> > > BSD+Patent License
> > >    change and provides the status of the license
> change
> > > for each TianoCore
> > >    repository and package.
> > >
> > > Once the conversion of the edk2 repository is
> complete,
> > > work will begin
> > > on the other repositories in the TianoCore project.
> > >
> > > Best regards,
> > >
> > > Mike
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH V2] Change EDK II to BSD+Patent License
  2019-03-26 18:21   ` Kinney, Michael D
@ 2019-03-26 18:54     ` Leif Lindholm
  2019-04-02 20:42       ` Kinney, Michael D
  0 siblings, 1 reply; 9+ messages in thread
From: Leif Lindholm @ 2019-03-26 18:54 UTC (permalink / raw)
  To: Kinney, Michael D; +Cc: edk2-devel@lists.01.org

On Tue, Mar 26, 2019 at 06:21:43PM +0000, Kinney, Michael D wrote:
> Hi Leif,
> 
> Thanks for the reviews.  Responses below.
> 
> Mike
> 
> > -----Original Message-----
> > From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> > Sent: Tuesday, March 26, 2019 11:09 AM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > Cc: edk2-devel@lists.01.org
> > Subject: Re: [edk2] [PATCH V2] Change EDK II to
> > BSD+Patent License
> > 
> > Hi Mike,
> > 
> > First of all - now the March table tag was made (and I'm
> > back from
> > holiday), I had planned to do the move of BeagleBoardPkg
> > and
> > Omap35xxPkg to edk2-platforms.
> > 
> > Would you prefer me to put that on hold, or should we
> > drop those
> > changes from this set and worry about those if/when we
> > get around to
> > relicensing edk2-platforms too?
> 
> I do not have a strong opinion on when those packages
> move to edk2-platforms.
> 
> I am holding off on other package moves I am involved in
> until after the license change.
> 
> > 
> > For the changes to ArmPkg, ArmPlatformPkg, EmbeddedPkg(,
> > BeagleBoardPkg, Omap35xxPkg):
> > Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> > 
> 
> Thank you!
> 
> > For the changes to edk2:
> > License.txt - could the commit message describe where
> > the new text is
> > 	      from (as an implicit way of explaining why the
> > 	      layout/bulleting has changed in the portion
> > that is
> > 	      otherwise content-wise identical)?
> 
> I do not follow what you want updated here.  Which 
> commit and what would you like the message changed to?

Right, I could have been more clear. The patch in question is
"edk2: Change License.txt from 2-Clause BSD to BSD+Patent"

I'm referring to the fact that a diff between
https://opensource.org/licenses/BSD-2-Clause and
https://opensource.org/licenses/BSDplusPatent
shows substantially less than this patch does - for layout and
bulleting format reasons.

So, as a clarification regarding why the diff appears greater than the
actual difference between the licenses (which would simply be an
insertion), you could note in the commit message that the .

(An alternative course of action would be to insert a preceding patch
aligning the layout and bulleting format of License.txt with the
opensource.org version.)

> >             - (I'm sorry, I should just keep quiet,
> > but...)
> >               The copyright lines at the top of the
> > Licence.txt file
> >               have been bugging me since day 1. Can we
> > drop them?
> > 	      Clearly none of these organisations hold
> > copyright over
> > 	      either the old or the new license.
> > 
> 
> The copyrights at the top of that file were inherited
> from the packages when License.txt used to be in each
> package.  If you think it is appropriate to remove them
> I am happy to do that as part of this series.  Does the
> same comment apply to License.txt in OvmfPkg?

I think I could use input from Laszlo here, but looking at other
BSD-licensed projects, they tend to have the LICENSE/COPYING/whatever
file contain a row referring to the project/foundation. Which I guess
in our case would be TianoCore. Some also add "and contributors" to
that one line.

(If including the statement in the top-level file is necessary,
"TianoCore and contributors" certainly sounds the most appropriate to
me.)

The way top-level Licence.txt has been used so far has ... given the
impression it was intended to be used as shorthand for identifying all
copyrights held within the project (and their dates). Which I don't
think is actually very useful.

> > I'll just add that my wording for the Signed-off-by was
> > just a
> > meant as a starting point and I'd be happy to see it
> > improved.
> 
> Please let me know if you have any suggestions here.

Well, I just feel it could do with some review. It's not *wrong* or
anything.

Regards,

Leif

> > 
> > But from my end, all edk2: patches other than
> > "edk2: Change License.txt from 2-Clause BSD to
> > BSD+Patent":
> > Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> > 
> > /
> >     Leif
> > 
> > On Sat, Mar 23, 2019 at 02:25:15AM +0000, Kinney,
> > Michael D wrote:
> > > Hello,
> > >
> > > New in V2
> > > =========
> > > * Remove Cc lines from commit messages
> > > * Remove branch reference from commit messages
> > > * Change license in 2 files missed in OvmfPkg
> > > * Update OvmfPkg/License.txt to BSD+Patent as the
> > default license
> > > * Move the portions of Contributions.txt in the root
> > of edk2 to
> > >   Readme.md in the root of edk2 that describe how to
> > contribute
> > >   along with the commit message format.
> > > * Add to Readme.md in the root of edk2 that Signed-
> > off-by means that
> > >   the contributor certifies compliance to the
> > Developer's Certificate
> > >   of Origin 1.1.  https://developercertificate.org
> > > =========
> > >
> > > 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 the patch series for review on the
> > following branch using
> > > edk2-stable201903 as the base for the patch series.
> > >
> > >
> > https://github.com/mdkinney/edk2/tree/Bug_1373_BsdPatent
> > License_V2
> > >
> > > The commits in patch series can be viewed here:
> > >
> > >
> > https://github.com/mdkinney/edk2/commits/Bug_1373_BsdPat
> > entLicense_V2
> > >
> > > The patch series has one patch per package along with
> > a few patches
> > > to update the license information in the root of the
> > edk2 repository
> > > as described in the RFC V3.
> > >
> > > Due to the size of the patch series, I prefer to not
> > send the
> > > patch emails.  Instead, please perform code reviews
> > using content
> > > from the branch.
> > >
> > > All EDK II package 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.
> > >
> > > Feedback and Reviewed-by emails should identify the
> > patch the feedback
> > > applies using the patch summary listed below.  The
> > goal is to complete
> > > all reviews to support the commit of these patches on
> > April 9, 2019.
> > >
> > > 2a7d2c56bc edk2: Remove Contributions.txt and update
> > Readme.md
> > > f9d59ccdc5 OvmfPkg: Change License.txt from 2-Clause
> > BSD to BSD+Patent
> > > ce3fbf929e StdLibPrivateInternalFiles: Replace BSD
> > License with BSD+Patent License
> > > aa8a3692c7 StdLib: Replace BSD License with BSD+Patent
> > License
> > > 2dfbe1e1ee AppPkg: Replace BSD License with BSD+Patent
> > License
> > > b2161f6dd8 Vlv2TbltDevicePkg: Replace BSD License with
> > BSD+Patent License
> > > 3688c33755 Vlv2DeviceRefCodePkg: Replace BSD License
> > with BSD+Patent License
> > > 8170308c98 UefiCpuPkg: Replace BSD License with
> > BSD+Patent License
> > > 4b68832cdc StandaloneMmPkg: Replace BSD License with
> > BSD+Patent License
> > > 327dc18122 SourceLevelDebugPkg: Replace BSD License
> > with BSD+Patent License
> > > 6c4c506a5e SignedCapsulePkg: Replace BSD License with
> > BSD+Patent License
> > > 2fdd514aff ShellPkg: Replace BSD License with
> > BSD+Patent License
> > > c667724cb9 ShellBinPkg: Replace BSD License with
> > BSD+Patent License
> > > 76f9e555b8 SecurityPkg: Replace BSD License with
> > BSD+Patent License
> > > 89a588c1be QuarkSocPkg: Replace BSD License with
> > BSD+Patent License
> > > 454f280e0c QuarkPlatformPkg: Replace BSD License with
> > BSD+Patent License
> > > 935319af35 PcAtChipsetPkg: Replace BSD License with
> > BSD+Patent License
> > > e39d07266d OvmfPkg: Replace BSD License with
> > BSD+Patent License
> > > 24d6ba8a25 OptionRomPkg: Replace BSD License with
> > BSD+Patent License
> > > 5c4a074eed Omap35xxPkg: Replace BSD License with
> > BSD+Patent License
> > > dde19815b3 Nt32Pkg: Replace BSD License with
> > BSD+Patent License
> > > acf2e84beb NetworkPkg: Replace BSD License with
> > BSD+Patent License
> > > 3e404f30b6 MdePkg: Replace BSD License with BSD+Patent
> > License
> > > 6e01bf6764 MdeModulePkg: Replace BSD License with
> > BSD+Patent License
> > > 36322b1b8c IntelSiliconPkg: Replace BSD License with
> > BSD+Patent License
> > > f5f9fa8b95 IntelFspWrapperPkg: Replace BSD License
> > with BSD+Patent License
> > > 39725dc5e3 IntelFspPkg: Replace BSD License with
> > BSD+Patent License
> > > ba8a9e3e1d IntelFsp2WrapperPkg: Replace BSD License
> > with BSD+Patent License
> > > a61295eeda IntelFsp2Pkg: Replace BSD License with
> > BSD+Patent License
> > > cd7bd24c6d IntelFrameworkPkg: Replace BSD License with
> > BSD+Patent License
> > > e9f7710bb6 IntelFrameworkModulePkg: Replace BSD
> > License with BSD+Patent License
> > > 487e96a6b4 FmpDevicePkg: Replace BSD License with
> > BSD+Patent License
> > > 3cfd1b98e4 FatPkg: Replace BSD License with BSD+Patent
> > License
> > > f9fc278ea1 EmulatorPkg: Replace BSD License with
> > BSD+Patent License
> > > 1a26764621 EmbeddedPkg: Replace BSD License with
> > BSD+Patent License
> > > aa402ac906 EdkCompatibilityPkg: Replace BSD License
> > with BSD+Patent License
> > > 3c6df91d7c DynamicTablesPkg: Replace BSD License with
> > BSD+Patent License
> > > 6389a5b4d5 CryptoPkg: Replace BSD License with
> > BSD+Patent License
> > > 0065fa2d9f CorebootPayloadPkg: Replace BSD License
> > with BSD+Patent License
> > > 26d7dbf868 CorebootModulePkg: Replace BSD License with
> > BSD+Patent License
> > > b1ebd76234 BeagleBoardPkg: Replace BSD License with
> > BSD+Patent License
> > > f23540ea65 ArmVirtPkg: Replace BSD License with
> > BSD+Patent License
> > > 054b667071 ArmPlatformPkg: Replace BSD License with
> > BSD+Patent License
> > > 5128ec1897 ArmPkg: Replace BSD License with BSD+Patent
> > License
> > > 3b7fd23df9 BaseTools: Replace BSD License with
> > BSD+Patent License
> > > aa5e7ad3ef edk2: Replace BSD License with BSD+Patent
> > License
> > > fdcf6f00c7 edk2: Change License.txt from 2-Clause BSD
> > to BSD+Patent
> > > 831e2096e8 edk2: Add License-History.txt
> > >
> > > Best regards,
> > >
> > > Mike
> > >
> > > > -----Original Message-----
> > > > From: Kinney, Michael D
> > > > Sent: Friday, March 22, 2019 6:49 PM
> > > > To: edk2-devel@lists.01.org; Kinney, Michael D
> > > > <michael.d.kinney@intel.com>
> > > > Subject: [RFC v3] Change EDK II to BSD+Patent
> > License
> > > >
> > > > Hello,
> > > >
> > > > Based on review of the RFC V2, there are some
> > updates
> > > > required to
> > > > Readme.md in the root of the edk2 repository.
> > > >
> > > > Changes for V3
> > > > ===============
> > > > * Move the portions of Contributions.txt in the root
> > of
> > > > edk2 to
> > > >   Readme.md in the root of edk2 that describe how to
> > > > contribute
> > > >   along with the commit message format.
> > > >
> > > > * Add to Readme.md in the root of edk2 that Signed-
> > off-by
> > > > means that
> > > >   the contributor certifies compliance to the
> > Developer's
> > > > Certificate
> > > >   of Origin 1.1.  https://developercertificate.org
> > > >
> > > > Changes for V2
> > > > ===============
> > > > * Replace 2-Clause BSD License in file headers with
> > SPDX-
> > > > License-Identifier
> > > >   statement.  This reduces the size of the file
> > headers
> > > > and the size
> > > >   of the patches for this change.  Based on the
> > following
> > > > post:
> > > >
> > > >   https://01.org/blogs/jc415/2018/open-source-hacks-
> > one-
> > > > question-interviews-open-source-experts-how-use-
> > spdx-
> > > > headers
> > > >
> > > > * Update License.txt in root of edk2 before changing
> > file
> > > > headers.
> > > > * Fix minor typos
> > > > ===============
> > > >
> > > > This RFC follows up on the proposal from Mark Doran
> > to
> > > > change the
> > > > EDK II Project to a BSD+Patent License.
> > > >
> > > > 	https://lists.01.org/pipermail/edk2-devel/2019-
> > > > February/036260.html
> > > >
> > > > The review period for this license change is 30
> > days.  If
> > > > there is no
> > > > unresolved feedback on April 9, 2019, then commits
> > of the
> > > > license change
> > > > patches will begin on April 9, 2019.
> > > >
> > > >   ** Please provide feedback on the proposal by
> > Monday
> > > > April 8, 2019. **
> > > >
> > > > Feedback can be sent to edk2-devel at lists.01.org,
> > the
> > > > EDK II community
> > > > manager or any of the EDK II stewards.
> > > >
> > > >   * Stephano Cetola <stephano.cetola at intel.com>
> > > > Community Manager
> > > >   * Leif Lindholm   <leif.lindholm at linaro.org>
> > > > Steward
> > > >   * Andrew Fish     <afish at apple.com>
> > > > Steward
> > > >   * Laszlo Ersek    <lersek at redhat.com>
> > > > Steward
> > > >   * Michael Kinney  <michael.d.kinney at intel.com>
> > > > Steward
> > > >
> > > > The goal is to convert all of the files in the edk2
> > > > repository that are
> > > > currently covered by the 2-Clause BSD License and
> > the
> > > > TianoCore
> > > > Contribution Agreement to a BSD+Patent License.
> > > >
> > > > I will be following up with pointers to public
> > GitHub
> > > > branches that
> > > > contain the set of changes to the edk2 repository
> > for
> > > > review.
> > > >
> > > > The proposal is to perform this change to
> > edk2/master in
> > > > the steps listed
> > > > below. The license change will not be applied to any
> > of
> > > > the other existing
> > > > branches in the edk2 repository.
> > > >
> > > > 1) Add a License-History.txt file to the root of the
> > edk2
> > > > repository that
> > > >    contains the 2-Clause BSD License and the
> > TianoCore
> > > > Contribution
> > > >    Agreement along with the details on the change to
> > the
> > > > BSD+Patent License.
> > > >
> > > > 2) Change License.txt in the root of the edk2
> > repository
> > > > from a 2-Clause
> > > >    BSD License to the BSD+Patent License. The
> > following
> > > > is the link to the
> > > >    BSD+Patent License and the new License.txt file
> > > > contents.
> > > >
> > > >    https://opensource.org/licenses/BSDplusPatent
> > > >
> > > >
> > > >
> > ========================================================
> > =
> > > > =============
> > > >    Redistribution and use in source and binary
> > forms,
> > > > with or without
> > > >    modification, are permitted provided that the
> > > > following conditions are met:
> > > >
> > > >    1. Redistributions of source code must retain the
> > > > above copyright notice,
> > > >       this list of conditions and the following
> > > > disclaimer.
> > > >
> > > >    2. Redistributions in binary form must reproduce
> > the
> > > > above copyright notice,
> > > >       this list of conditions and the following
> > > > disclaimer in the documentation
> > > >       and/or other materials provided with the
> > > > distribution.
> > > >
> > > >    Subject to the terms and conditions of this
> > license,
> > > > each copyright holder
> > > >    and contributor hereby grants to those receiving
> > > > rights under this license
> > > >    a perpetual, worldwide, non-exclusive, no-charge,
> > > > royalty-free, irrevocable
> > > >    (except for failure to satisfy the conditions of
> > this
> > > > license) patent
> > > >    license to make, have made, use, offer to sell,
> > sell,
> > > > import, and otherwise
> > > >    transfer this software, where such license
> > applies
> > > > only to those patent
> > > >    claims, already acquired or hereafter acquired,
> > > > licensable by such copyright
> > > >    holder or contributor that are necessarily
> > infringed
> > > > by:
> > > >
> > > >    (a) their Contribution(s) (the licensed
> > copyrights of
> > > > copyright holders and
> > > >        non-copyrightable additions of contributors,
> > in
> > > > source or binary form)
> > > >        alone; or
> > > >
> > > >    (b) combination of their Contribution(s) with the
> > work
> > > > of authorship to
> > > >        which such Contribution(s) was added by such
> > > > copyright holder or
> > > >        contributor, if, at the time the Contribution
> > is
> > > > added, such addition
> > > >        causes such combination to be necessarily
> > > > infringed. The patent license
> > > >        shall not apply to any other combinations
> > which
> > > > include the
> > > >        Contribution.
> > > >
> > > >    Except as expressly stated above, no rights or
> > > > licenses from any copyright
> > > >    holder or contributor is granted under this
> > license,
> > > > whether expressly, by
> > > >    implication, estoppel or otherwise.
> > > >
> > > >    DISCLAIMER
> > > >
> > > >    THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT
> > HOLDERS AND
> > > > CONTRIBUTORS "AS IS"
> > > >    AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING,
> > BUT
> > > > NOT LIMITED TO, THE
> > > >    IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
> > FOR
> > > > A PARTICULAR PURPOSE
> > > >    ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> > > > HOLDERS OR CONTRIBUTORS BE
> > > >    LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
> > SPECIAL,
> > > > EXEMPLARY, OR
> > > >    CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED
> > TO,
> > > > PROCUREMENT OF
> > > >    SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA,
> > OR
> > > > PROFITS; OR BUSINESS
> > > >    INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
> > > > LIABILITY, WHETHER IN
> > > >    CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
> > > > NEGLIGENCE OR OTHERWISE)
> > > >    ARISING IN ANY WAY OUT OF THE USE OF THIS
> > SOFTWARE,
> > > > EVEN IF ADVISED OF THE
> > > >    POSSIBILITY OF SUCH DAMAGE.
> > > >
> > > >
> > ========================================================
> > =
> > > > =============
> > > >
> > > > 3) Change all files currently covered by a 2-Clause
> > BSD
> > > > License and the
> > > >    TianoCore Contribution Agreement to a BSD+Patent
> > > > License using the
> > > >    following SPDX-License-Identifier statement:
> > > >
> > > >        SPDX-License-Identifier: BSD-2-Clause-Patent
> > > >
> > > >    The use of SPDX-License-Identifier statement is
> > based
> > > > on the following:
> > > >
> > > >         https://01.org/blogs/jc415/2018/open-source-
> > > > hacks-one-question-interviews-open-source-experts-
> > how-
> > > > use-spdx-headers
> > > >
> > > > 4) Update Readme.md in the root of the edk2
> > repository to
> > > > state that content
> > > >    is covered by a BSD+Patent License.  Also state
> > that
> > > > the BSD+Patent License
> > > >    is the preferred license for the EDK II project.
> > > >
> > > >   a) Move the portions of Contributions.txt in the
> > root
> > > > of the edk2 repository
> > > >      Readme.md in the root of edk2 repository that
> > > > describe how to contribute
> > > >      along with the commit message format.
> > > >
> > > >   b) Add the following to Readme.md in the root of
> > edk2
> > > > repository:
> > > >
> > > >      # Developer Certificate of Origin
> > > >
> > > >      Your change description should use the standard
> > > > format for a
> > > >      commit message, and must include your `Signed-
> > off-
> > > > by` signature.
> > > >
> > > >      In order to keep track of who did what, all
> > patches
> > > > contributed must
> > > >      include a statement that to the best of the
> > > > contributor's knowledge
> > > >      they have the right to contribute it under the
> > > > specified license.
> > > >
> > > >      The test for this is as specified in the
> > > > [Developer's Certificate of
> > > >      Origin (DCO)
> > > > 1.1](https://developercertificate.org/). The
> > contributor
> > > >      certifies compliance by adding a line saying
> > > >
> > > >        Signed-off-by: Developer Name
> > > > <developer@example.org>
> > > >
> > > >      where `Developer Name` is the contributor's
> > real
> > > > name, and the email
> > > >      address is one the developer is reachable
> > through at
> > > > the time of
> > > >      contributing.
> > > >
> > > >      ```
> > > >      Developer's Certificate of Origin 1.1
> > > >
> > > >      By making a contribution to this project, I
> > certify
> > > > that:
> > > >
> > > >       (a) The contribution was created in whole or
> > in
> > > > part by me and I
> > > >          have the right to submit it under the open
> > > > source license
> > > >          indicated in the file; or
> > > >
> > > >       (b) The contribution is based upon previous
> > work
> > > > that, to the best
> > > >          of my knowledge, is covered under an
> > appropriate
> > > > open source
> > > >          license and I have the right under that
> > license
> > > > to submit that
> > > >          work with modifications, whether created in
> > > > whole or in part
> > > >          by me, under the same open source license
> > > > (unless I am
> > > >          permitted to submit under a different
> > license),
> > > > as indicated
> > > >          in the file; or
> > > >
> > > >       (c) The contribution was provided directly to
> > me by
> > > > some other
> > > >          person who certified (a), (b) or (c) and I
> > have
> > > > not modified
> > > >          it.
> > > >
> > > >       (d) I understand and agree that this project
> > and
> > > > the contribution
> > > >          are public and that a record of the
> > contribution
> > > > (including all
> > > >          personal information I submit with it,
> > including
> > > > my sign-off) is
> > > >          maintained indefinitely and may be
> > redistributed
> > > > consistent with
> > > >          this project or the open source license(s)
> > > > involved.
> > > >      ```
> > > >
> > > > 5) Remove the Contributions.txt file from the root
> > of the
> > > > edk2 repository
> > > >    that contains the TianoCore Contribution
> > Agreement.
> > > >
> > > > 6) Update all documentation to state that content
> > > > submitted under the
> > > >    BSD+Patent License no longer requires the
> > Tianocore
> > > > Contribution
> > > >    Agreement which means the following line is not
> > > > required in commit
> > > >    messages for changes to files that are covered by
> > a
> > > > BSD+Patent License.
> > > >
> > > >        Contributed-under: TianoCore Contribution
> > > > Agreement 1.1
> > > >
> > > > 7) Create Wiki page(s) that provide the details of
> > the
> > > > BSD+Patent License
> > > >    change and provides the status of the license
> > change
> > > > for each TianoCore
> > > >    repository and package.
> > > >
> > > > Once the conversion of the edk2 repository is
> > complete,
> > > > work will begin
> > > > on the other repositories in the TianoCore project.
> > > >
> > > > Best regards,
> > > >
> > > > Mike
> > >
> > > _______________________________________________
> > > edk2-devel mailing list
> > > edk2-devel@lists.01.org
> > > https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH V2] Change EDK II to BSD+Patent License
  2019-03-26 18:54     ` Leif Lindholm
@ 2019-04-02 20:42       ` Kinney, Michael D
  0 siblings, 0 replies; 9+ messages in thread
From: Kinney, Michael D @ 2019-04-02 20:42 UTC (permalink / raw)
  To: Leif Lindholm, Kinney, Michael D; +Cc: edk2-devel@lists.01.org

Hi Leif,

1) I will update commit message for the Licence.txt
   change to explain why the diff appears larger   
   than it should.  The links you provided do not
   provide a preferred ASCII text format.  The
   differences are due line break choices.

2) I will update the Copyright in the new version
   of Licence.txt to use "TianoCore and contributors"

Thanks,

Mike


> -----Original Message-----
> From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> Sent: Tuesday, March 26, 2019 11:54 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>
> Cc: edk2-devel@lists.01.org
> Subject: Re: [edk2] [PATCH V2] Change EDK II to
> BSD+Patent License
> 
> On Tue, Mar 26, 2019 at 06:21:43PM +0000, Kinney,
> Michael D wrote:
> > Hi Leif,
> >
> > Thanks for the reviews.  Responses below.
> >
> > Mike
> >
> > > -----Original Message-----
> > > From: Leif Lindholm
> [mailto:leif.lindholm@linaro.org]
> > > Sent: Tuesday, March 26, 2019 11:09 AM
> > > To: Kinney, Michael D <michael.d.kinney@intel.com>
> > > Cc: edk2-devel@lists.01.org
> > > Subject: Re: [edk2] [PATCH V2] Change EDK II to
> > > BSD+Patent License
> > >
> > > Hi Mike,
> > >
> > > First of all - now the March table tag was made
> (and I'm
> > > back from
> > > holiday), I had planned to do the move of
> BeagleBoardPkg
> > > and
> > > Omap35xxPkg to edk2-platforms.
> > >
> > > Would you prefer me to put that on hold, or should
> we
> > > drop those
> > > changes from this set and worry about those
> if/when we
> > > get around to
> > > relicensing edk2-platforms too?
> >
> > I do not have a strong opinion on when those
> packages
> > move to edk2-platforms.
> >
> > I am holding off on other package moves I am
> involved in
> > until after the license change.
> >
> > >
> > > For the changes to ArmPkg, ArmPlatformPkg,
> EmbeddedPkg(,
> > > BeagleBoardPkg, Omap35xxPkg):
> > > Reviewed-by: Leif Lindholm
> <leif.lindholm@linaro.org>
> > >
> >
> > Thank you!
> >
> > > For the changes to edk2:
> > > License.txt - could the commit message describe
> where
> > > the new text is
> > > 	      from (as an implicit way of explaining why
> the
> > > 	      layout/bulleting has changed in the
> portion
> > > that is
> > > 	      otherwise content-wise identical)?
> >
> > I do not follow what you want updated here.  Which
> > commit and what would you like the message changed
> to?
> 
> Right, I could have been more clear. The patch in
> question is
> "edk2: Change License.txt from 2-Clause BSD to
> BSD+Patent"
> 
> I'm referring to the fact that a diff between
> https://opensource.org/licenses/BSD-2-Clause and
> https://opensource.org/licenses/BSDplusPatent
> shows substantially less than this patch does - for
> layout and
> bulleting format reasons.
> 
> So, as a clarification regarding why the diff appears
> greater than the
> actual difference between the licenses (which would
> simply be an
> insertion), you could note in the commit message that
> the .
> 
> (An alternative course of action would be to insert a
> preceding patch
> aligning the layout and bulleting format of
> License.txt with the
> opensource.org version.)
> 
> > >             - (I'm sorry, I should just keep
> quiet,
> > > but...)
> > >               The copyright lines at the top of
> the
> > > Licence.txt file
> > >               have been bugging me since day 1.
> Can we
> > > drop them?
> > > 	      Clearly none of these organisations hold
> > > copyright over
> > > 	      either the old or the new license.
> > >
> >
> > The copyrights at the top of that file were
> inherited
> > from the packages when License.txt used to be in
> each
> > package.  If you think it is appropriate to remove
> them
> > I am happy to do that as part of this series.  Does
> the
> > same comment apply to License.txt in OvmfPkg?
> 
> I think I could use input from Laszlo here, but
> looking at other
> BSD-licensed projects, they tend to have the
> LICENSE/COPYING/whatever
> file contain a row referring to the
> project/foundation. Which I guess
> in our case would be TianoCore. Some also add "and
> contributors" to
> that one line.
> 
> (If including the statement in the top-level file is
> necessary,
> "TianoCore and contributors" certainly sounds the most
> appropriate to
> me.)
> 
> The way top-level Licence.txt has been used so far has
> ... given the
> impression it was intended to be used as shorthand for
> identifying all
> copyrights held within the project (and their dates).
> Which I don't
> think is actually very useful.
> 
> > > I'll just add that my wording for the Signed-off-
> by was
> > > just a
> > > meant as a starting point and I'd be happy to see
> it
> > > improved.
> >
> > Please let me know if you have any suggestions here.
> 
> Well, I just feel it could do with some review. It's
> not *wrong* or
> anything.
> 
> Regards,
> 
> Leif
> 
> > >
> > > But from my end, all edk2: patches other than
> > > "edk2: Change License.txt from 2-Clause BSD to
> > > BSD+Patent":
> > > Reviewed-by: Leif Lindholm
> <leif.lindholm@linaro.org>
> > >
> > > /
> > >     Leif
> > >
> > > On Sat, Mar 23, 2019 at 02:25:15AM +0000, Kinney,
> > > Michael D wrote:
> > > > Hello,
> > > >
> > > > New in V2
> > > > =========
> > > > * Remove Cc lines from commit messages
> > > > * Remove branch reference from commit messages
> > > > * Change license in 2 files missed in OvmfPkg
> > > > * Update OvmfPkg/License.txt to BSD+Patent as
> the
> > > default license
> > > > * Move the portions of Contributions.txt in the
> root
> > > of edk2 to
> > > >   Readme.md in the root of edk2 that describe
> how to
> > > contribute
> > > >   along with the commit message format.
> > > > * Add to Readme.md in the root of edk2 that
> Signed-
> > > off-by means that
> > > >   the contributor certifies compliance to the
> > > Developer's Certificate
> > > >   of Origin 1.1.
> https://developercertificate.org
> > > > =========
> > > >
> > > > 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 the patch series for review on the
> > > following branch using
> > > > edk2-stable201903 as the base for the patch
> series.
> > > >
> > > >
> > >
> https://github.com/mdkinney/edk2/tree/Bug_1373_BsdPate
> nt
> > > License_V2
> > > >
> > > > The commits in patch series can be viewed here:
> > > >
> > > >
> > >
> https://github.com/mdkinney/edk2/commits/Bug_1373_BsdP
> at
> > > entLicense_V2
> > > >
> > > > The patch series has one patch per package along
> with
> > > a few patches
> > > > to update the license information in the root of
> the
> > > edk2 repository
> > > > as described in the RFC V3.
> > > >
> > > > Due to the size of the patch series, I prefer to
> not
> > > send the
> > > > patch emails.  Instead, please perform code
> reviews
> > > using content
> > > > from the branch.
> > > >
> > > > All EDK II package 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.
> > > >
> > > > Feedback and Reviewed-by emails should identify
> the
> > > patch the feedback
> > > > applies using the patch summary listed below.
> The
> > > goal is to complete
> > > > all reviews to support the commit of these
> patches on
> > > April 9, 2019.
> > > >
> > > > 2a7d2c56bc edk2: Remove Contributions.txt and
> update
> > > Readme.md
> > > > f9d59ccdc5 OvmfPkg: Change License.txt from 2-
> Clause
> > > BSD to BSD+Patent
> > > > ce3fbf929e StdLibPrivateInternalFiles: Replace
> BSD
> > > License with BSD+Patent License
> > > > aa8a3692c7 StdLib: Replace BSD License with
> BSD+Patent
> > > License
> > > > 2dfbe1e1ee AppPkg: Replace BSD License with
> BSD+Patent
> > > License
> > > > b2161f6dd8 Vlv2TbltDevicePkg: Replace BSD
> License with
> > > BSD+Patent License
> > > > 3688c33755 Vlv2DeviceRefCodePkg: Replace BSD
> License
> > > with BSD+Patent License
> > > > 8170308c98 UefiCpuPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 4b68832cdc StandaloneMmPkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > 327dc18122 SourceLevelDebugPkg: Replace BSD
> License
> > > with BSD+Patent License
> > > > 6c4c506a5e SignedCapsulePkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > 2fdd514aff ShellPkg: Replace BSD License with
> > > BSD+Patent License
> > > > c667724cb9 ShellBinPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 76f9e555b8 SecurityPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 89a588c1be QuarkSocPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 454f280e0c QuarkPlatformPkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > 935319af35 PcAtChipsetPkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > e39d07266d OvmfPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 24d6ba8a25 OptionRomPkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > 5c4a074eed Omap35xxPkg: Replace BSD License with
> > > BSD+Patent License
> > > > dde19815b3 Nt32Pkg: Replace BSD License with
> > > BSD+Patent License
> > > > acf2e84beb NetworkPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 3e404f30b6 MdePkg: Replace BSD License with
> BSD+Patent
> > > License
> > > > 6e01bf6764 MdeModulePkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > 36322b1b8c IntelSiliconPkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > f5f9fa8b95 IntelFspWrapperPkg: Replace BSD
> License
> > > with BSD+Patent License
> > > > 39725dc5e3 IntelFspPkg: Replace BSD License with
> > > BSD+Patent License
> > > > ba8a9e3e1d IntelFsp2WrapperPkg: Replace BSD
> License
> > > with BSD+Patent License
> > > > a61295eeda IntelFsp2Pkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > cd7bd24c6d IntelFrameworkPkg: Replace BSD
> License with
> > > BSD+Patent License
> > > > e9f7710bb6 IntelFrameworkModulePkg: Replace BSD
> > > License with BSD+Patent License
> > > > 487e96a6b4 FmpDevicePkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > 3cfd1b98e4 FatPkg: Replace BSD License with
> BSD+Patent
> > > License
> > > > f9fc278ea1 EmulatorPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 1a26764621 EmbeddedPkg: Replace BSD License with
> > > BSD+Patent License
> > > > aa402ac906 EdkCompatibilityPkg: Replace BSD
> License
> > > with BSD+Patent License
> > > > 3c6df91d7c DynamicTablesPkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > 6389a5b4d5 CryptoPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 0065fa2d9f CorebootPayloadPkg: Replace BSD
> License
> > > with BSD+Patent License
> > > > 26d7dbf868 CorebootModulePkg: Replace BSD
> License with
> > > BSD+Patent License
> > > > b1ebd76234 BeagleBoardPkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > f23540ea65 ArmVirtPkg: Replace BSD License with
> > > BSD+Patent License
> > > > 054b667071 ArmPlatformPkg: Replace BSD License
> with
> > > BSD+Patent License
> > > > 5128ec1897 ArmPkg: Replace BSD License with
> BSD+Patent
> > > License
> > > > 3b7fd23df9 BaseTools: Replace BSD License with
> > > BSD+Patent License
> > > > aa5e7ad3ef edk2: Replace BSD License with
> BSD+Patent
> > > License
> > > > fdcf6f00c7 edk2: Change License.txt from 2-
> Clause BSD
> > > to BSD+Patent
> > > > 831e2096e8 edk2: Add License-History.txt
> > > >
> > > > Best regards,
> > > >
> > > > Mike
> > > >
> > > > > -----Original Message-----
> > > > > From: Kinney, Michael D
> > > > > Sent: Friday, March 22, 2019 6:49 PM
> > > > > To: edk2-devel@lists.01.org; Kinney, Michael D
> > > > > <michael.d.kinney@intel.com>
> > > > > Subject: [RFC v3] Change EDK II to BSD+Patent
> > > License
> > > > >
> > > > > Hello,
> > > > >
> > > > > Based on review of the RFC V2, there are some
> > > updates
> > > > > required to
> > > > > Readme.md in the root of the edk2 repository.
> > > > >
> > > > > Changes for V3
> > > > > ===============
> > > > > * Move the portions of Contributions.txt in
> the root
> > > of
> > > > > edk2 to
> > > > >   Readme.md in the root of edk2 that describe
> how to
> > > > > contribute
> > > > >   along with the commit message format.
> > > > >
> > > > > * Add to Readme.md in the root of edk2 that
> Signed-
> > > off-by
> > > > > means that
> > > > >   the contributor certifies compliance to the
> > > Developer's
> > > > > Certificate
> > > > >   of Origin 1.1.
> https://developercertificate.org
> > > > >
> > > > > Changes for V2
> > > > > ===============
> > > > > * Replace 2-Clause BSD License in file headers
> with
> > > SPDX-
> > > > > License-Identifier
> > > > >   statement.  This reduces the size of the
> file
> > > headers
> > > > > and the size
> > > > >   of the patches for this change.  Based on
> the
> > > following
> > > > > post:
> > > > >
> > > > >   https://01.org/blogs/jc415/2018/open-source-
> hacks-
> > > one-
> > > > > question-interviews-open-source-experts-how-
> use-
> > > spdx-
> > > > > headers
> > > > >
> > > > > * Update License.txt in root of edk2 before
> changing
> > > file
> > > > > headers.
> > > > > * Fix minor typos
> > > > > ===============
> > > > >
> > > > > This RFC follows up on the proposal from Mark
> Doran
> > > to
> > > > > change the
> > > > > EDK II Project to a BSD+Patent License.
> > > > >
> > > > > 	https://lists.01.org/pipermail/edk2-
> devel/2019-
> > > > > February/036260.html
> > > > >
> > > > > The review period for this license change is
> 30
> > > days.  If
> > > > > there is no
> > > > > unresolved feedback on April 9, 2019, then
> commits
> > > of the
> > > > > license change
> > > > > patches will begin on April 9, 2019.
> > > > >
> > > > >   ** Please provide feedback on the proposal
> by
> > > Monday
> > > > > April 8, 2019. **
> > > > >
> > > > > Feedback can be sent to edk2-devel at
> lists.01.org,
> > > the
> > > > > EDK II community
> > > > > manager or any of the EDK II stewards.
> > > > >
> > > > >   * Stephano Cetola <stephano.cetola at
> intel.com>
> > > > > Community Manager
> > > > >   * Leif Lindholm   <leif.lindholm at
> linaro.org>
> > > > > Steward
> > > > >   * Andrew Fish     <afish at apple.com>
> > > > > Steward
> > > > >   * Laszlo Ersek    <lersek at redhat.com>
> > > > > Steward
> > > > >   * Michael Kinney  <michael.d.kinney at
> intel.com>
> > > > > Steward
> > > > >
> > > > > The goal is to convert all of the files in the
> edk2
> > > > > repository that are
> > > > > currently covered by the 2-Clause BSD License
> and
> > > the
> > > > > TianoCore
> > > > > Contribution Agreement to a BSD+Patent
> License.
> > > > >
> > > > > I will be following up with pointers to public
> > > GitHub
> > > > > branches that
> > > > > contain the set of changes to the edk2
> repository
> > > for
> > > > > review.
> > > > >
> > > > > The proposal is to perform this change to
> > > edk2/master in
> > > > > the steps listed
> > > > > below. The license change will not be applied
> to any
> > > of
> > > > > the other existing
> > > > > branches in the edk2 repository.
> > > > >
> > > > > 1) Add a License-History.txt file to the root
> of the
> > > edk2
> > > > > repository that
> > > > >    contains the 2-Clause BSD License and the
> > > TianoCore
> > > > > Contribution
> > > > >    Agreement along with the details on the
> change to
> > > the
> > > > > BSD+Patent License.
> > > > >
> > > > > 2) Change License.txt in the root of the edk2
> > > repository
> > > > > from a 2-Clause
> > > > >    BSD License to the BSD+Patent License. The
> > > following
> > > > > is the link to the
> > > > >    BSD+Patent License and the new License.txt
> file
> > > > > contents.
> > > > >
> > > > >
> https://opensource.org/licenses/BSDplusPatent
> > > > >
> > > > >
> > > > >
> > >
> ======================================================
> ==
> > > =
> > > > > =============
> > > > >    Redistribution and use in source and binary
> > > forms,
> > > > > with or without
> > > > >    modification, are permitted provided that
> the
> > > > > following conditions are met:
> > > > >
> > > > >    1. Redistributions of source code must
> retain the
> > > > > above copyright notice,
> > > > >       this list of conditions and the
> following
> > > > > disclaimer.
> > > > >
> > > > >    2. Redistributions in binary form must
> reproduce
> > > the
> > > > > above copyright notice,
> > > > >       this list of conditions and the
> following
> > > > > disclaimer in the documentation
> > > > >       and/or other materials provided with the
> > > > > distribution.
> > > > >
> > > > >    Subject to the terms and conditions of this
> > > license,
> > > > > each copyright holder
> > > > >    and contributor hereby grants to those
> receiving
> > > > > rights under this license
> > > > >    a perpetual, worldwide, non-exclusive, no-
> charge,
> > > > > royalty-free, irrevocable
> > > > >    (except for failure to satisfy the
> conditions of
> > > this
> > > > > license) patent
> > > > >    license to make, have made, use, offer to
> sell,
> > > sell,
> > > > > import, and otherwise
> > > > >    transfer this software, where such license
> > > applies
> > > > > only to those patent
> > > > >    claims, already acquired or hereafter
> acquired,
> > > > > licensable by such copyright
> > > > >    holder or contributor that are necessarily
> > > infringed
> > > > > by:
> > > > >
> > > > >    (a) their Contribution(s) (the licensed
> > > copyrights of
> > > > > copyright holders and
> > > > >        non-copyrightable additions of
> contributors,
> > > in
> > > > > source or binary form)
> > > > >        alone; or
> > > > >
> > > > >    (b) combination of their Contribution(s)
> with the
> > > work
> > > > > of authorship to
> > > > >        which such Contribution(s) was added by
> such
> > > > > copyright holder or
> > > > >        contributor, if, at the time the
> Contribution
> > > is
> > > > > added, such addition
> > > > >        causes such combination to be
> necessarily
> > > > > infringed. The patent license
> > > > >        shall not apply to any other
> combinations
> > > which
> > > > > include the
> > > > >        Contribution.
> > > > >
> > > > >    Except as expressly stated above, no rights
> or
> > > > > licenses from any copyright
> > > > >    holder or contributor is granted under this
> > > license,
> > > > > whether expressly, by
> > > > >    implication, estoppel or otherwise.
> > > > >
> > > > >    DISCLAIMER
> > > > >
> > > > >    THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT
> > > HOLDERS AND
> > > > > CONTRIBUTORS "AS IS"
> > > > >    AND ANY EXPRESS OR IMPLIED WARRANTIES,
> INCLUDING,
> > > BUT
> > > > > NOT LIMITED TO, THE
> > > > >    IMPLIED WARRANTIES OF MERCHANTABILITY AND
> FITNESS
> > > FOR
> > > > > A PARTICULAR PURPOSE
> > > > >    ARE DISCLAIMED. IN NO EVENT SHALL THE
> COPYRIGHT
> > > > > HOLDERS OR CONTRIBUTORS BE
> > > > >    LIABLE FOR ANY DIRECT, INDIRECT,
> INCIDENTAL,
> > > SPECIAL,
> > > > > EXEMPLARY, OR
> > > > >    CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
> LIMITED
> > > TO,
> > > > > PROCUREMENT OF
> > > > >    SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
> DATA,
> > > OR
> > > > > PROFITS; OR BUSINESS
> > > > >    INTERRUPTION) HOWEVER CAUSED AND ON ANY
> THEORY OF
> > > > > LIABILITY, WHETHER IN
> > > > >    CONTRACT, STRICT LIABILITY, OR TORT
> (INCLUDING
> > > > > NEGLIGENCE OR OTHERWISE)
> > > > >    ARISING IN ANY WAY OUT OF THE USE OF THIS
> > > SOFTWARE,
> > > > > EVEN IF ADVISED OF THE
> > > > >    POSSIBILITY OF SUCH DAMAGE.
> > > > >
> > > > >
> > >
> ======================================================
> ==
> > > =
> > > > > =============
> > > > >
> > > > > 3) Change all files currently covered by a 2-
> Clause
> > > BSD
> > > > > License and the
> > > > >    TianoCore Contribution Agreement to a
> BSD+Patent
> > > > > License using the
> > > > >    following SPDX-License-Identifier
> statement:
> > > > >
> > > > >        SPDX-License-Identifier: BSD-2-Clause-
> Patent
> > > > >
> > > > >    The use of SPDX-License-Identifier
> statement is
> > > based
> > > > > on the following:
> > > > >
> > > > >         https://01.org/blogs/jc415/2018/open-
> source-
> > > > > hacks-one-question-interviews-open-source-
> experts-
> > > how-
> > > > > use-spdx-headers
> > > > >
> > > > > 4) Update Readme.md in the root of the edk2
> > > repository to
> > > > > state that content
> > > > >    is covered by a BSD+Patent License.  Also
> state
> > > that
> > > > > the BSD+Patent License
> > > > >    is the preferred license for the EDK II
> project.
> > > > >
> > > > >   a) Move the portions of Contributions.txt in
> the
> > > root
> > > > > of the edk2 repository
> > > > >      Readme.md in the root of edk2 repository
> that
> > > > > describe how to contribute
> > > > >      along with the commit message format.
> > > > >
> > > > >   b) Add the following to Readme.md in the
> root of
> > > edk2
> > > > > repository:
> > > > >
> > > > >      # Developer Certificate of Origin
> > > > >
> > > > >      Your change description should use the
> standard
> > > > > format for a
> > > > >      commit message, and must include your
> `Signed-
> > > off-
> > > > > by` signature.
> > > > >
> > > > >      In order to keep track of who did what,
> all
> > > patches
> > > > > contributed must
> > > > >      include a statement that to the best of
> the
> > > > > contributor's knowledge
> > > > >      they have the right to contribute it
> under the
> > > > > specified license.
> > > > >
> > > > >      The test for this is as specified in the
> > > > > [Developer's Certificate of
> > > > >      Origin (DCO)
> > > > > 1.1](https://developercertificate.org/). The
> > > contributor
> > > > >      certifies compliance by adding a line
> saying
> > > > >
> > > > >        Signed-off-by: Developer Name
> > > > > <developer@example.org>
> > > > >
> > > > >      where `Developer Name` is the
> contributor's
> > > real
> > > > > name, and the email
> > > > >      address is one the developer is reachable
> > > through at
> > > > > the time of
> > > > >      contributing.
> > > > >
> > > > >      ```
> > > > >      Developer's Certificate of Origin 1.1
> > > > >
> > > > >      By making a contribution to this project,
> I
> > > certify
> > > > > that:
> > > > >
> > > > >       (a) The contribution was created in
> whole or
> > > in
> > > > > part by me and I
> > > > >          have the right to submit it under the
> open
> > > > > source license
> > > > >          indicated in the file; or
> > > > >
> > > > >       (b) The contribution is based upon
> previous
> > > work
> > > > > that, to the best
> > > > >          of my knowledge, is covered under an
> > > appropriate
> > > > > open source
> > > > >          license and I have the right under
> that
> > > license
> > > > > to submit that
> > > > >          work with modifications, whether
> created in
> > > > > whole or in part
> > > > >          by me, under the same open source
> license
> > > > > (unless I am
> > > > >          permitted to submit under a different
> > > license),
> > > > > as indicated
> > > > >          in the file; or
> > > > >
> > > > >       (c) The contribution was provided
> directly to
> > > me by
> > > > > some other
> > > > >          person who certified (a), (b) or (c)
> and I
> > > have
> > > > > not modified
> > > > >          it.
> > > > >
> > > > >       (d) I understand and agree that this
> project
> > > and
> > > > > the contribution
> > > > >          are public and that a record of the
> > > contribution
> > > > > (including all
> > > > >          personal information I submit with
> it,
> > > including
> > > > > my sign-off) is
> > > > >          maintained indefinitely and may be
> > > redistributed
> > > > > consistent with
> > > > >          this project or the open source
> license(s)
> > > > > involved.
> > > > >      ```
> > > > >
> > > > > 5) Remove the Contributions.txt file from the
> root
> > > of the
> > > > > edk2 repository
> > > > >    that contains the TianoCore Contribution
> > > Agreement.
> > > > >
> > > > > 6) Update all documentation to state that
> content
> > > > > submitted under the
> > > > >    BSD+Patent License no longer requires the
> > > Tianocore
> > > > > Contribution
> > > > >    Agreement which means the following line is
> not
> > > > > required in commit
> > > > >    messages for changes to files that are
> covered by
> > > a
> > > > > BSD+Patent License.
> > > > >
> > > > >        Contributed-under: TianoCore
> Contribution
> > > > > Agreement 1.1
> > > > >
> > > > > 7) Create Wiki page(s) that provide the
> details of
> > > the
> > > > > BSD+Patent License
> > > > >    change and provides the status of the
> license
> > > change
> > > > > for each TianoCore
> > > > >    repository and package.
> > > > >
> > > > > Once the conversion of the edk2 repository is
> > > complete,
> > > > > work will begin
> > > > > on the other repositories in the TianoCore
> project.
> > > > >
> > > > > Best regards,
> > > > >
> > > > > Mike
> > > >
> > > > _______________________________________________
> > > > edk2-devel mailing list
> > > > edk2-devel@lists.01.org
> > > > https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH V2] Change EDK II to BSD+Patent License
  2019-03-23  2:25 [PATCH V2] Change EDK II to BSD+Patent License Kinney, Michael D
  2019-03-25 12:11 ` Laszlo Ersek
  2019-03-26 18:08 ` Leif Lindholm
@ 2019-04-03 11:43 ` Laszlo Ersek
       [not found] ` <74D8A39837DF1E4DA445A8C0B3885C503F5C2D92@shsmsx102.ccr.corp.intel.com>
  3 siblings, 0 replies; 9+ messages in thread
From: Laszlo Ersek @ 2019-04-03 11:43 UTC (permalink / raw)
  To: Kinney, Michael D; +Cc: edk2-devel@lists.01.org

Hi Mike,

On 03/23/19 03:25, Kinney, Michael D wrote:
> Hello,
>
> New in V2
> =========
> * Remove Cc lines from commit messages
> * Remove branch reference from commit messages
> * Change license in 2 files missed in OvmfPkg
> * Update OvmfPkg/License.txt to BSD+Patent as the default license
> * Move the portions of Contributions.txt in the root of edk2 to
>   Readme.md in the root of edk2 that describe how to contribute
>   along with the commit message format.
> * Add to Readme.md in the root of edk2 that Signed-off-by means that
>   the contributor certifies compliance to the Developer's Certificate
>   of Origin 1.1.  https://developercertificate.org
> =========
>
> 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 the patch series for review on the following branch
> using edk2-stable201903 as the base for the patch series.
>
>   https://github.com/mdkinney/edk2/tree/Bug_1373_BsdPatentLicense_V2
>
> The commits in patch series can be viewed here:
>
>   https://github.com/mdkinney/edk2/commits/Bug_1373_BsdPatentLicense_V2
>
> The patch series has one patch per package along with a few patches
> to update the license information in the root of the edk2 repository
> as described in the RFC V3.
>
> Due to the size of the patch series, I prefer to not send the
> patch emails.  Instead, please perform code reviews using content
> from the branch.
>
> All EDK II package 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.
>
> Feedback and Reviewed-by emails should identify the patch the feedback
> applies using the patch summary listed below.  The goal is to complete
> all reviews to support the commit of these patches on April 9, 2019.

Given that we've now entered the file addition/removal freeze:

  http://mid.mail-archive.com/E92EE9817A31E24EB0585FDF735412F5B9C772EE@ORSMSX112.amr.corp.intel.com

and that in that email you identify this (v2) series as "current", I'm
now reviewing v2.

Your v2 series was based on upstream commit 89910a39dcfd. The master
branch has since advanced to 7ed72121b753 however. Therefore, for this
review, I've rebased the three v2 patches listed below on top of commit
7ed72121b753 (i.e. on top the current master HEAD).


> f23540ea65 ArmVirtPkg: Replace BSD License with BSD+Patent License

I reviewed this patch most recently as part of your v1 series:

  http://mid.mail-archive.com/f2a32071-868a-e4fa-dcca-41bf28ba93aa@redhat.com

Content-wise, I'm in luck with the v2 review of this patch, because the
ArmVirtPkg directory tree is identical
- between your v1 patch set
- and your v2 patch set, rebased on top of current master.

Comparing the commit messages, I find:

- the RFC link list has been extended with the v3 RFC,
- the reference to your branch on github has been removed,
- the Cc: list in the commit message has been deleted.

Hence all my remarks have been observed. For this patch:

Reviewed-by: Laszlo Ersek <lersek@redhat.com>


> e39d07266d OvmfPkg: Replace BSD License with BSD+Patent License

(While rebasing this patch from your v2 to current master, I ran into a
conflict, but that was easy to resolve and it's going to be covered
below anyway.)

Relative to v1:

(1) RFC link list updated, OK

(2) "Branch for review" removed, OK (v1/1.2)

(3) Cc: list removed, OK (v1/1.1)

(4) The "create-release.py" hunk doesn't apply any longer, so please
    remove it in v3. This conflict is due to us removing
    "create-release.py" altogether, for
    <https://bugzilla.tianocore.org/show_bug.cgi?id=1653>. (v1/2.1.1)

(5) At commit 7ed72121b753 (= current master HEAD), OvmfPkg contains the
    following new files:

      OvmfPkg/SioBusDxe/ComponentName.c
      OvmfPkg/SioBusDxe/SioBusDxe.c
      OvmfPkg/SioBusDxe/SioBusDxe.h
      OvmfPkg/SioBusDxe/SioBusDxe.inf
      OvmfPkg/SioBusDxe/SioBusDxe.uni
      OvmfPkg/SioBusDxe/SioService.c
      OvmfPkg/SioBusDxe/SioService.h

    All of them reference the URL
    <http://opensource.org/licenses/bsd-license.php>, so please convert
    them too, in v3.

(6) The set of small/trivial files, without any copyright notices, is
    unchanged; OK (v1/2.2.1)

(7) We now have a separate BZ for the MIT-covered files,
    <https://bugzilla.tianocore.org/show_bug.cgi?id=1654>. The set of
    those files is unchanged, OK. (v1/2.2.2)

(8) "OvmfPkg/License.txt" is converted in a new patch (below), I'll
    review it separately. So, for this patch, it's OK (v1/2.2.3)

(9) The files

      OvmfPkg/XenBusDxe/GrantTable.c
      OvmfPkg/XenPvBlkDxe/BlockFront.c

    are now correctly converted, OK. (v1/2.2.4)

Thus, in version 3, please fix (4) and (5).


> f9d59ccdc5 OvmfPkg: Change License.txt from 2-Clause BSD to BSD+Patent

For this patch:

Reviewed-by: Laszlo Ersek <lersek@redhat.com>


Thanks!
Laszlo


^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [edk2] [PATCH V2] Change EDK II to BSD+Patent License
       [not found]   ` <74D8A39837DF1E4DA445A8C0B3885C503F5C2DFA@shsmsx102.ccr.corp.intel.com>
@ 2019-04-04  2:00     ` Yao, Jiewen
  0 siblings, 0 replies; 9+ messages in thread
From: Yao, Jiewen @ 2019-04-04  2:00 UTC (permalink / raw)
  To: Kinney, Michael D, devel@edk2.groups.io

Resend to the new mail list.

> > -----Original Message-----
> > From: Yao, Jiewen [mailto:jiewen.yao@intel.com]
> > Sent: Thursday, April 4, 2019 9:54 AM
> > To: Kinney, Michael D <michael.d.kinney@intel.com>;
> > edk2-devel@lists.01.org; Kinney, Michael D <michael.d.kinney@intel.com>
> > Subject: RE: [edk2] [PATCH V2] Change EDK II to BSD+Patent License
> >
> > SignedCapsulePkg: Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
> > SecurityPkg: Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
> > StandaloneMmPkg: Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>
> >
> > Thank you
> > Yao Jiewen
> >
> > > -----Original Message-----
> > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf
> Of
> > > Kinney, Michael D
> > > Sent: Saturday, March 23, 2019 10:25 AM
> > > To: edk2-devel@lists.01.org; Kinney, Michael D
> > > <michael.d.kinney@intel.com>
> > > Subject: [edk2] [PATCH V2] Change EDK II to BSD+Patent License
> > >
> > > Hello,
> > >
> > > New in V2
> > > =========
> > > * Remove Cc lines from commit messages
> > > * Remove branch reference from commit messages
> > > * Change license in 2 files missed in OvmfPkg
> > > * Update OvmfPkg/License.txt to BSD+Patent as the default license
> > > * Move the portions of Contributions.txt in the root of edk2 to
> > >   Readme.md in the root of edk2 that describe how to contribute
> > >   along with the commit message format.
> > > * Add to Readme.md in the root of edk2 that Signed-off-by means that
> > >   the contributor certifies compliance to the Developer's Certificate
> > >   of Origin 1.1.  https://developercertificate.org
> > > =========
> > >
> > > 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 the patch series for review on the following branch using
> > > edk2-stable201903 as the base for the patch series.
> > >
> > >
> > https://github.com/mdkinney/edk2/tree/Bug_1373_BsdPatentLicense_V2
> > >
> > > The commits in patch series can be viewed here:
> > >
> > >
> > >
> >
> https://github.com/mdkinney/edk2/commits/Bug_1373_BsdPatentLicense_
> > > V2
> > >
> > > The patch series has one patch per package along with a few patches
> > > to update the license information in the root of the edk2 repository
> > > as described in the RFC V3.
> > >
> > > Due to the size of the patch series, I prefer to not send the
> > > patch emails.  Instead, please perform code reviews using content
> > > from the branch.
> > >
> > > All EDK II package 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.
> > >
> > > Feedback and Reviewed-by emails should identify the patch the feedback
> > > applies using the patch summary listed below.  The goal is to complete
> > > all reviews to support the commit of these patches on April 9, 2019.
> > >
> > > 2a7d2c56bc edk2: Remove Contributions.txt and update Readme.md
> > > f9d59ccdc5 OvmfPkg: Change License.txt from 2-Clause BSD to
> > BSD+Patent
> > > ce3fbf929e StdLibPrivateInternalFiles: Replace BSD License with
> > BSD+Patent
> > > License
> > > aa8a3692c7 StdLib: Replace BSD License with BSD+Patent License
> > > 2dfbe1e1ee AppPkg: Replace BSD License with BSD+Patent License
> > > b2161f6dd8 Vlv2TbltDevicePkg: Replace BSD License with BSD+Patent
> > > License
> > > 3688c33755 Vlv2DeviceRefCodePkg: Replace BSD License with
> BSD+Patent
> > > License
> > > 8170308c98 UefiCpuPkg: Replace BSD License with BSD+Patent License
> > > 4b68832cdc StandaloneMmPkg: Replace BSD License with BSD+Patent
> > > License
> > > 327dc18122 SourceLevelDebugPkg: Replace BSD License with
> BSD+Patent
> > > License
> > > 6c4c506a5e SignedCapsulePkg: Replace BSD License with BSD+Patent
> > > License
> > > 2fdd514aff ShellPkg: Replace BSD License with BSD+Patent License
> > > c667724cb9 ShellBinPkg: Replace BSD License with BSD+Patent License
> > > 76f9e555b8 SecurityPkg: Replace BSD License with BSD+Patent License
> > > 89a588c1be QuarkSocPkg: Replace BSD License with BSD+Patent License
> > > 454f280e0c QuarkPlatformPkg: Replace BSD License with BSD+Patent
> > > License
> > > 935319af35 PcAtChipsetPkg: Replace BSD License with BSD+Patent
> License
> > > e39d07266d OvmfPkg: Replace BSD License with BSD+Patent License
> > > 24d6ba8a25 OptionRomPkg: Replace BSD License with BSD+Patent
> License
> > > 5c4a074eed Omap35xxPkg: Replace BSD License with BSD+Patent
> License
> > > dde19815b3 Nt32Pkg: Replace BSD License with BSD+Patent License
> > > acf2e84beb NetworkPkg: Replace BSD License with BSD+Patent License
> > > 3e404f30b6 MdePkg: Replace BSD License with BSD+Patent License
> > > 6e01bf6764 MdeModulePkg: Replace BSD License with BSD+Patent
> License
> > > 36322b1b8c IntelSiliconPkg: Replace BSD License with BSD+Patent
> License
> > > f5f9fa8b95 IntelFspWrapperPkg: Replace BSD License with BSD+Patent
> > > License
> > > 39725dc5e3 IntelFspPkg: Replace BSD License with BSD+Patent License
> > > ba8a9e3e1d IntelFsp2WrapperPkg: Replace BSD License with BSD+Patent
> > > License
> > > a61295eeda IntelFsp2Pkg: Replace BSD License with BSD+Patent License
> > > cd7bd24c6d IntelFrameworkPkg: Replace BSD License with BSD+Patent
> > > License
> > > e9f7710bb6 IntelFrameworkModulePkg: Replace BSD License with
> > > BSD+Patent License
> > > 487e96a6b4 FmpDevicePkg: Replace BSD License with BSD+Patent
> License
> > > 3cfd1b98e4 FatPkg: Replace BSD License with BSD+Patent License
> > > f9fc278ea1 EmulatorPkg: Replace BSD License with BSD+Patent License
> > > 1a26764621 EmbeddedPkg: Replace BSD License with BSD+Patent
> License
> > > aa402ac906 EdkCompatibilityPkg: Replace BSD License with BSD+Patent
> > > License
> > > 3c6df91d7c DynamicTablesPkg: Replace BSD License with BSD+Patent
> > > License
> > > 6389a5b4d5 CryptoPkg: Replace BSD License with BSD+Patent License
> > > 0065fa2d9f CorebootPayloadPkg: Replace BSD License with BSD+Patent
> > > License
> > > 26d7dbf868 CorebootModulePkg: Replace BSD License with BSD+Patent
> > > License
> > > b1ebd76234 BeagleBoardPkg: Replace BSD License with BSD+Patent
> > License
> > > f23540ea65 ArmVirtPkg: Replace BSD License with BSD+Patent License
> > > 054b667071 ArmPlatformPkg: Replace BSD License with BSD+Patent
> > License
> > > 5128ec1897 ArmPkg: Replace BSD License with BSD+Patent License
> > > 3b7fd23df9 BaseTools: Replace BSD License with BSD+Patent License
> > > aa5e7ad3ef edk2: Replace BSD License with BSD+Patent License
> > > fdcf6f00c7 edk2: Change License.txt from 2-Clause BSD to BSD+Patent
> > > 831e2096e8 edk2: Add License-History.txt
> > >
> > > Best regards,
> > >
> > > Mike
> > >
> > > > -----Original Message-----
> > > > From: Kinney, Michael D
> > > > Sent: Friday, March 22, 2019 6:49 PM
> > > > To: edk2-devel@lists.01.org; Kinney, Michael D
> > > > <michael.d.kinney@intel.com>
> > > > Subject: [RFC v3] Change EDK II to BSD+Patent License
> > > >
> > > > Hello,
> > > >
> > > > Based on review of the RFC V2, there are some updates
> > > > required to
> > > > Readme.md in the root of the edk2 repository.
> > > >
> > > > Changes for V3
> > > > ===============
> > > > * Move the portions of Contributions.txt in the root of
> > > > edk2 to
> > > >   Readme.md in the root of edk2 that describe how to
> > > > contribute
> > > >   along with the commit message format.
> > > >
> > > > * Add to Readme.md in the root of edk2 that Signed-off-by
> > > > means that
> > > >   the contributor certifies compliance to the Developer's
> > > > Certificate
> > > >   of Origin 1.1.  https://developercertificate.org
> > > >
> > > > Changes for V2
> > > > ===============
> > > > * Replace 2-Clause BSD License in file headers with SPDX-
> > > > License-Identifier
> > > >   statement.  This reduces the size of the file headers
> > > > and the size
> > > >   of the patches for this change.  Based on the following
> > > > post:
> > > >
> > > >   https://01.org/blogs/jc415/2018/open-source-hacks-one-
> > > > question-interviews-open-source-experts-how-use-spdx-
> > > > headers
> > > >
> > > > * Update License.txt in root of edk2 before changing file
> > > > headers.
> > > > * Fix minor typos
> > > > ===============
> > > >
> > > > This RFC follows up on the proposal from Mark Doran to
> > > > change the
> > > > EDK II Project to a BSD+Patent License.
> > > >
> > > > 	https://lists.01.org/pipermail/edk2-devel/2019-
> > > > February/036260.html
> > > >
> > > > The review period for this license change is 30 days.  If
> > > > there is no
> > > > unresolved feedback on April 9, 2019, then commits of the
> > > > license change
> > > > patches will begin on April 9, 2019.
> > > >
> > > >   ** Please provide feedback on the proposal by Monday
> > > > April 8, 2019. **
> > > >
> > > > Feedback can be sent to edk2-devel at lists.01.org, the
> > > > EDK II community
> > > > manager or any of the EDK II stewards.
> > > >
> > > >   * Stephano Cetola <stephano.cetola at intel.com>
> > > > Community Manager
> > > >   * Leif Lindholm   <leif.lindholm at linaro.org>
> > > > Steward
> > > >   * Andrew Fish     <afish at apple.com>
> > > > Steward
> > > >   * Laszlo Ersek    <lersek at redhat.com>
> > > > Steward
> > > >   * Michael Kinney  <michael.d.kinney at intel.com>
> > > > Steward
> > > >
> > > > The goal is to convert all of the files in the edk2
> > > > repository that are
> > > > currently covered by the 2-Clause BSD License and the
> > > > TianoCore
> > > > Contribution Agreement to a BSD+Patent License.
> > > >
> > > > I will be following up with pointers to public GitHub
> > > > branches that
> > > > contain the set of changes to the edk2 repository for
> > > > review.
> > > >
> > > > The proposal is to perform this change to edk2/master in
> > > > the steps listed
> > > > below. The license change will not be applied to any of
> > > > the other existing
> > > > branches in the edk2 repository.
> > > >
> > > > 1) Add a License-History.txt file to the root of the edk2
> > > > repository that
> > > >    contains the 2-Clause BSD License and the TianoCore
> > > > Contribution
> > > >    Agreement along with the details on the change to the
> > > > BSD+Patent License.
> > > >
> > > > 2) Change License.txt in the root of the edk2 repository
> > > > from a 2-Clause
> > > >    BSD License to the BSD+Patent License. The following
> > > > is the link to the
> > > >    BSD+Patent License and the new License.txt file
> > > > contents.
> > > >
> > > >    https://opensource.org/licenses/BSDplusPatent
> > > >
> > > >
> > > > =========================================================
> > > > =============
> > > >    Redistribution and use in source and binary forms,
> > > > with or without
> > > >    modification, are permitted provided that the
> > > > following conditions are met:
> > > >
> > > >    1. Redistributions of source code must retain the
> > > > above copyright notice,
> > > >       this list of conditions and the following
> > > > disclaimer.
> > > >
> > > >    2. Redistributions in binary form must reproduce the
> > > > above copyright notice,
> > > >       this list of conditions and the following
> > > > disclaimer in the documentation
> > > >       and/or other materials provided with the
> > > > distribution.
> > > >
> > > >    Subject to the terms and conditions of this license,
> > > > each copyright holder
> > > >    and contributor hereby grants to those receiving
> > > > rights under this license
> > > >    a perpetual, worldwide, non-exclusive, no-charge,
> > > > royalty-free, irrevocable
> > > >    (except for failure to satisfy the conditions of this
> > > > license) patent
> > > >    license to make, have made, use, offer to sell, sell,
> > > > import, and otherwise
> > > >    transfer this software, where such license applies
> > > > only to those patent
> > > >    claims, already acquired or hereafter acquired,
> > > > licensable by such copyright
> > > >    holder or contributor that are necessarily infringed
> > > > by:
> > > >
> > > >    (a) their Contribution(s) (the licensed copyrights of
> > > > copyright holders and
> > > >        non-copyrightable additions of contributors, in
> > > > source or binary form)
> > > >        alone; or
> > > >
> > > >    (b) combination of their Contribution(s) with the work
> > > > of authorship to
> > > >        which such Contribution(s) was added by such
> > > > copyright holder or
> > > >        contributor, if, at the time the Contribution is
> > > > added, such addition
> > > >        causes such combination to be necessarily
> > > > infringed. The patent license
> > > >        shall not apply to any other combinations which
> > > > include the
> > > >        Contribution.
> > > >
> > > >    Except as expressly stated above, no rights or
> > > > licenses from any copyright
> > > >    holder or contributor is granted under this license,
> > > > whether expressly, by
> > > >    implication, estoppel or otherwise.
> > > >
> > > >    DISCLAIMER
> > > >
> > > >    THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
> > > > CONTRIBUTORS "AS IS"
> > > >    AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT
> > > > NOT LIMITED TO, THE
> > > >    IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
> > > > A PARTICULAR PURPOSE
> > > >    ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
> > > > HOLDERS OR CONTRIBUTORS BE
> > > >    LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
> > > > EXEMPLARY, OR
> > > >    CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
> > > > PROCUREMENT OF
> > > >    SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
> > > > PROFITS; OR BUSINESS
> > > >    INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
> > > > LIABILITY, WHETHER IN
> > > >    CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
> > > > NEGLIGENCE OR OTHERWISE)
> > > >    ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE,
> > > > EVEN IF ADVISED OF THE
> > > >    POSSIBILITY OF SUCH DAMAGE.
> > > >
> > > > =========================================================
> > > > =============
> > > >
> > > > 3) Change all files currently covered by a 2-Clause BSD
> > > > License and the
> > > >    TianoCore Contribution Agreement to a BSD+Patent
> > > > License using the
> > > >    following SPDX-License-Identifier statement:
> > > >
> > > >        SPDX-License-Identifier: BSD-2-Clause-Patent
> > > >
> > > >    The use of SPDX-License-Identifier statement is based
> > > > on the following:
> > > >
> > > >         https://01.org/blogs/jc415/2018/open-source-
> > > > hacks-one-question-interviews-open-source-experts-how-
> > > > use-spdx-headers
> > > >
> > > > 4) Update Readme.md in the root of the edk2 repository to
> > > > state that content
> > > >    is covered by a BSD+Patent License.  Also state that
> > > > the BSD+Patent License
> > > >    is the preferred license for the EDK II project.
> > > >
> > > >   a) Move the portions of Contributions.txt in the root
> > > > of the edk2 repository
> > > >      Readme.md in the root of edk2 repository that
> > > > describe how to contribute
> > > >      along with the commit message format.
> > > >
> > > >   b) Add the following to Readme.md in the root of edk2
> > > > repository:
> > > >
> > > >      # Developer Certificate of Origin
> > > >
> > > >      Your change description should use the standard
> > > > format for a
> > > >      commit message, and must include your `Signed-off-
> > > > by` signature.
> > > >
> > > >      In order to keep track of who did what, all patches
> > > > contributed must
> > > >      include a statement that to the best of the
> > > > contributor's knowledge
> > > >      they have the right to contribute it under the
> > > > specified license.
> > > >
> > > >      The test for this is as specified in the
> > > > [Developer's Certificate of
> > > >      Origin (DCO)
> > > > 1.1](https://developercertificate.org/). The contributor
> > > >      certifies compliance by adding a line saying
> > > >
> > > >        Signed-off-by: Developer Name
> > > > <developer@example.org>
> > > >
> > > >      where `Developer Name` is the contributor's real
> > > > name, and the email
> > > >      address is one the developer is reachable through at
> > > > the time of
> > > >      contributing.
> > > >
> > > >      ```
> > > >      Developer's Certificate of Origin 1.1
> > > >
> > > >      By making a contribution to this project, I certify
> > > > that:
> > > >
> > > >       (a) The contribution was created in whole or in
> > > > part by me and I
> > > >          have the right to submit it under the open
> > > > source license
> > > >          indicated in the file; or
> > > >
> > > >       (b) The contribution is based upon previous work
> > > > that, to the best
> > > >          of my knowledge, is covered under an appropriate
> > > > open source
> > > >          license and I have the right under that license
> > > > to submit that
> > > >          work with modifications, whether created in
> > > > whole or in part
> > > >          by me, under the same open source license
> > > > (unless I am
> > > >          permitted to submit under a different license),
> > > > as indicated
> > > >          in the file; or
> > > >
> > > >       (c) The contribution was provided directly to me by
> > > > some other
> > > >          person who certified (a), (b) or (c) and I have
> > > > not modified
> > > >          it.
> > > >
> > > >       (d) I understand and agree that this project and
> > > > the contribution
> > > >          are public and that a record of the contribution
> > > > (including all
> > > >          personal information I submit with it, including
> > > > my sign-off) is
> > > >          maintained indefinitely and may be redistributed
> > > > consistent with
> > > >          this project or the open source license(s)
> > > > involved.
> > > >      ```
> > > >
> > > > 5) Remove the Contributions.txt file from the root of the
> > > > edk2 repository
> > > >    that contains the TianoCore Contribution Agreement.
> > > >
> > > > 6) Update all documentation to state that content
> > > > submitted under the
> > > >    BSD+Patent License no longer requires the Tianocore
> > > > Contribution
> > > >    Agreement which means the following line is not
> > > > required in commit
> > > >    messages for changes to files that are covered by a
> > > > BSD+Patent License.
> > > >
> > > >        Contributed-under: TianoCore Contribution
> > > > Agreement 1.1
> > > >
> > > > 7) Create Wiki page(s) that provide the details of the
> > > > BSD+Patent License
> > > >    change and provides the status of the license change
> > > > for each TianoCore
> > > >    repository and package.
> > > >
> > > > Once the conversion of the edk2 repository is complete,
> > > > work will begin
> > > > on the other repositories in the TianoCore project.
> > > >
> > > > Best regards,
> > > >
> > > > Mike
> > >
> > > _______________________________________________
> > > edk2-devel mailing list
> > > edk2-devel@lists.01.org
> > > https://lists.01.org/mailman/listinfo/edk2-devel

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2019-04-04  2:00 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-03-23  2:25 [PATCH V2] Change EDK II to BSD+Patent License Kinney, Michael D
2019-03-25 12:11 ` Laszlo Ersek
2019-03-25 17:24   ` Kinney, Michael D
2019-03-26 18:08 ` Leif Lindholm
2019-03-26 18:21   ` Kinney, Michael D
2019-03-26 18:54     ` Leif Lindholm
2019-04-02 20:42       ` Kinney, Michael D
2019-04-03 11:43 ` Laszlo Ersek
     [not found] ` <74D8A39837DF1E4DA445A8C0B3885C503F5C2D92@shsmsx102.ccr.corp.intel.com>
     [not found]   ` <74D8A39837DF1E4DA445A8C0B3885C503F5C2DFA@shsmsx102.ccr.corp.intel.com>
2019-04-04  2:00     ` [edk2] " Yao, Jiewen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox