From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id E259C21CE749E for ; Thu, 20 Jul 2017 15:51:35 -0700 (PDT) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 20 Jul 2017 15:53:32 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.40,386,1496127600"; d="scan'208";a="113711764" Received: from jljusten-skl.jf.intel.com (HELO localhost) ([10.54.75.36]) by orsmga002.jf.intel.com with ESMTP; 20 Jul 2017 15:53:32 -0700 MIME-Version: 1.0 To: "Kinney, Michael D" , "edk2-devel@lists.01.org" , Laszlo Ersek Message-ID: <150059121174.4016.1590734145039602530@jljusten-skl.jf.intel.com> From: Jordan Justen In-Reply-To: Cc: Leif Lindholm , Andrew Fish References: <20170720054411.18928-1-michael.d.kinney@intel.com> <20170720054411.18928-8-michael.d.kinney@intel.com> <150053255036.1046.7072266056287610184@jljusten-skl> User-Agent: alot/0.5.1 Date: Thu, 20 Jul 2017 15:53:31 -0700 Subject: Re: [Patch V3 7/8] edk2: Move License.txt file to root X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Jul 2017 22:51:36 -0000 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On 2017-07-20 08:09:52, Kinney, Michael D wrote: > Hi Jordan, > = > Yes. The ECP change is on purpose to make ECP license > consistent with the majority of the edk2 content. Nice. For some reason I thought we couldn't make this change previously. You'll update the commit message to note the change? > The summary of all licenses used in all components has > been added to Readme.md in the root of the edk2 repo, > so any developer that looks at this repo know all the = > licenses in use without having to search through all > the directories in the repository. = I disagree. I'd like to keep the OvmfPkg/License.txt file. I also think that any other package that uses multiple licenses should do the same. If you drop "[Patch V3 6/8] edk2/OvmfPkg: Add MIT license files" and the OvmfPkg change in this patch, then: Reviewed-by: Jordan Justen > = > > -----Original Message----- > > From: Justen, Jordan L > > Sent: Wednesday, July 19, 2017 11:36 PM > > To: Kinney, Michael D ; edk2- > > devel@lists.01.org; Laszlo Ersek > > Cc: Leif Lindholm ; Andrew Fish > > > > Subject: Re: [Patch V3 7/8] edk2: Move License.txt file to root > > = > > On 2017-07-19 22:44:10, Michael D Kinney wrote: > > > https://bugzilla.tianocore.org/show_bug.cgi?id=3D642 > > > > > > Add top level License.txt file with the BSD 2-Clause > > > License that is used by the majority of the EKD II open > > > source project content. Merge copyright statements > > > from the BSD 2-Clause License files in each package > > > directory and remove the duplication License.txt > > > file from package directories. > > > > > > Cc: Leif Lindholm > > > Cc: Andrew Fish > > > Cc: Jordan Justen > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > > Signed-off-by: Michael D Kinney > > > --- > > > AppPkg/License.txt | 25 ----------------- > > > ArmPkg/License.txt | 26 ------------------ > > > ArmPlatformPkg/License.txt | 25 ----------------- > > > BaseTools/License.txt | 25 ----------------- > > > BeagleBoardPkg/License.txt | 26 ------------------ > > > CorebootModulePkg/License.txt | 25 ----------------- > > > CorebootPayloadPkg/License.txt | 25 ----------------- > > > CryptoPkg/License.txt | 25 ----------------- > > > DuetPkg/License.txt | 25 ----------------- > > > EdkCompatibilityPkg/License.txt | 29 ------------------ > > -- > > = > > ECP currently has a 3-clause BSD license in License.txt. Is this > > change intended? I think that the ECP source file headers may > > point to > > the 2-clause BSD license though. > > = > > Maybe worth noting in the commit message. > > = > > > EdkShellBinPkg/License.txt | 25 ----------------- > > > EdkShellPkg/License.txt | 25 ----------------- > > > EmbeddedPkg/License.txt | 26 ------------------ > > > EmulatorPkg/License.txt | 25 ----------------- > > > FatBinPkg/License.txt | 25 ----------------- > > > FatPkg/License.txt | 25 ----------------- > > > IntelFrameworkModulePkg/License.txt | 25 ----------------- > > > IntelFrameworkPkg/License.txt | 25 ----------------- > > > IntelFsp2Pkg/License.txt | 25 ----------------- > > > IntelFsp2WrapperPkg/License.txt | 25 ----------------- > > > IntelFspPkg/License.txt | 25 ----------------- > > > IntelFspWrapperPkg/License.txt | 25 ----------------- > > > IntelSiliconPkg/License.txt | 25 ----------------- > > > ArmVirtPkg/License.txt =3D> License.txt | 6 ++-- > > > MdeModulePkg/License.txt | 25 ----------------- > > > MdePkg/License.txt | 25 ----------------- > > > NetworkPkg/License.txt | 25 ----------------- > > > Nt32Pkg/License.txt | 25 ----------------- > > > Omap35xxPkg/License.txt | 26 ------------------ > > > OptionRomPkg/License.txt | 25 ----------------- > > > OvmfPkg/License.txt | 50 ------------------ > > ---------------- > > = > > I'm not sure about Laszlo, but I prefer to keep the current > > License.txt at the OvmfPkg root. It gives an 'overview' of the > > licenses used in the package without having to search the > > sub-directories. > > = > > -Jordan > > = > > > PcAtChipsetPkg/License.txt | 25 ----------------- > > > PerformancePkg/License.txt | 25 ----------------- > > > QuarkPlatformPkg/License.txt | 25 ----------------- > > > QuarkSocPkg/License.txt | 25 ----------------- > > > SecurityPkg/License.txt | 25 ----------------- > > > ShellBinPkg/License.txt | 25 ----------------- > > > ShellPkg/License.txt | 25 ----------------- > > > SignedCapsulePkg/License.txt | 25 ----------------- > > > SourceLevelDebugPkg/License.txt | 25 ----------------- > > > StdLib/License.txt | 25 ----------------- > > > StdLibPrivateInternalFiles/License.txt | 25 ----------------- > > > UefiCpuPkg/License.txt | 25 ----------------- > > > Vlv2DeviceRefCodePkg/License.txt | 25 ----------------- > > > Vlv2TbltDevicePkg/License.txt | 25 -----------------