From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web11.7887.1593700323733559571 for ; Thu, 02 Jul 2020 07:32:04 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: ard.biesheuvel@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 04CA231B; Thu, 2 Jul 2020 07:32:01 -0700 (PDT) Received: from [192.168.1.81] (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 9D7DC3F68F; Thu, 2 Jul 2020 07:31:59 -0700 (PDT) Subject: Re: [edk2-devel] License Check - was OvmfPkg: Adding support for bhyve as OvmfPkg/Bhyve To: devel@edk2.groups.io, leif@nuviainc.com, "Gao, Liming" Cc: Laszlo Ersek , Rebecca Cran , Andrew Fish , "Justen, Jordan L" , "Kinney, Michael D" References: <92612908-44f4-30d4-ae3a-3ad595e2141f@bsdio.com> <68b55c49-1e49-e082-ca38-68c3d321611b@redhat.com> <20200702105419.GN6739@vanye> <20200702141323.GQ6739@vanye> From: "Ard Biesheuvel" Message-ID: Date: Thu, 2 Jul 2020 16:31:49 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: <20200702141323.GQ6739@vanye> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit On 7/2/20 4:13 PM, Leif Lindholm via groups.io wrote: > On Thu, Jul 02, 2020 at 13:49:45 +0000, Gao, Liming wrote: >> Leif: >> >>> -----Original Message----- >>> From: devel@edk2.groups.io On Behalf Of Leif Lindholm >>> Sent: Thursday, July 2, 2020 6:54 PM >>> To: Laszlo Ersek >>> Cc: Rebecca Cran ; edk2-devel-groups-io ; Ard Biesheuvel ; >>> Andrew Fish ; Justen, Jordan L ; Kinney, Michael D >>> Subject: Re: [edk2-devel] License Check - was OvmfPkg: Adding support for bhyve as OvmfPkg/Bhyve >>> >>> On Thu, Jul 02, 2020 at 11:27:25 +0200, Laszlo Ersek wrote: >>>> This likely comes from BaseTools commit a4cfb842fca9 >>>> ("BaseTools/PatchCheck.py: Add LicenseCheck", 2020-06-12). >>>> >>>> One approach would be to remove "VbeShim.h" from the tracked files under >>>> OvmfPkg, replacing it with a PREBUILD command in the OVMF DSC files. >>>> (Then Bhyve could do the same.) >>>> >>>> However, the generator, namely "VbeShim.sh", is not written in Python, >>>> but in (POSIX) shell, and so it can't be called from PREBUILD (I think >>>> it would break OVMF builds on Windows). >>>> >>>> I don't know what to tell you, other than the blanket license >>>> enforcement from commit a4cfb842fca9 is likely wrong. >>> >>> *Reads patch* >>> *Figuratively spits coffee all over keyboard* >>> >>> No, this is not OK. >>> >>> We *STILL* have no agreed process for accepting non bsd+patent content >>> since we dropped the contribution agreement. I have tried to raise >>> this issue several times in the past, and there has never been any >>> outcome from resulting discussions. >>> >>> So now I'm going to send out a two-patch set consisting of: >>> - Reverting a4cfb842fca9. (Doing nothing is better than implying that >>> anything !bsd+patent can currently be added to the tree.) >>> - Deleting the statement in ReadmMe.rst erroneously claiming that the >>> includion of these other licenses are acceptable until such a point >>> an active decision has been taken, approved by the community, that >>> this is permitted. >>> >> >> If only bsd+patent is allowed, the checker can be enhanced to check this license only. >> I don't understand why remove this checker. > > Mainly because that was the easiest thing to do :) > > But also because: > - The thread that spawned this also raised the problem of > machine-generated files. > - I am somewhat unhappy the checker got merged in the first place > without wider community feedback. BaseTools and its contents are > used for many repositories (even within TianoCore), and this added > unconditional check breaks the use for some of those. > I think the fundamental problem is that contributing code under a contribution agreement that includes a patent grant is not the same as contributing it under a patent grant license, given that the latter can only be done by the author of the code, while the former could be done by anyone. This means our current licensing policy is actually more restrictive that the old one, making it more difficult to incorporate 'second hand' code. I don't think we can fix this with a patch though :-(