From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f49.google.com (mail-wm1-f49.google.com [209.85.128.49]) by mx.groups.io with SMTP id smtpd.web11.7633.1593772681427797907 for ; Fri, 03 Jul 2020 03:38:01 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@nuviainc-com.20150623.gappssmtp.com header.s=20150623 header.b=XoqmwOS0; spf=pass (domain: nuviainc.com, ip: 209.85.128.49, mailfrom: leif@nuviainc.com) Received: by mail-wm1-f49.google.com with SMTP id j18so31569039wmi.3 for ; Fri, 03 Jul 2020 03:38:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nuviainc-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=QhaQBvyIWbCVZYM+V/fi1nndMyDdGGQVHYptHBBGLDk=; b=XoqmwOS0B4GKybVb2dPlWVA895/RWYmOecMCDLbhpUKqDkCje/XJq/xOETBr3WcL4W m43yCGlrxYaDq889rcWiffxmc65NioJzY+gh9xWQRSJEpVHcEDbP9lQoOJNCWgf9suGE L5Hej+CvAGviYX26bCK8k3PTMaIex+iQOMNPy0VMFuqs9XRIxFJbEXONvrXGdGU5+aXH 9/B/Gf3cRmdNrJQKQ1vSr3w5dwa6LUJPyCwjVG7fv4F0H47BJko3NIbL4/dOjOm94HDX osNEzjNEDksmR+Gh/lRljZYtyPNK6fRXM0k82LKsHIcPB3oKvDQAfPrntdKFdZemQ7i8 Qawg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=QhaQBvyIWbCVZYM+V/fi1nndMyDdGGQVHYptHBBGLDk=; b=NzU78Y9IyciUXBSbzSGwV9l12bHXB2f9MxDrof7XR/qD/Hr3/Fl8qUwuORolFQDfQr L6031PhcNHZE0YCpTZuBtygaLz+TRDs/KkysX407E7rSHToZXsZatzhaSC1rdaY7OzhH MRV0dyYu0p7ULRhbb4tIUL2ZZ4dznKrIyGy71FOER6vgoAtia5MFhbcLkLhVTeYzxLGU GW1f4utTpKazusf29egcktqn7H5ccnCMAAB40kXYAGk01nLNF1f4Fz7CXXGwqfGA+Ex2 rOdqqYnRP7xCA0EMKCzUJ/QzON7tq43lGG94RW112Jch2XqImjcORG2xGE2D78BZ0XPc 0bFw== X-Gm-Message-State: AOAM530RHhFcNZ7y7l/2ii17zXK0GnCVWUqa9I4lUSyv5w0pUKMUIn/H QKcScwAJXHW5OnYt/ClkegPDEA== X-Google-Smtp-Source: ABdhPJyWXsRHSkrcILRtIQZPCEvt0ZEarIZ94GcIJLJYN51i6WAvCdX1b5QWRIi1FMq6MbMtOJZyEg== X-Received: by 2002:a1c:6384:: with SMTP id x126mr35600252wmb.144.1593772679969; Fri, 03 Jul 2020 03:37:59 -0700 (PDT) Return-Path: Received: from vanye ([2001:470:1f09:12f0:b26e:bfff:fea9:f1b8]) by smtp.gmail.com with ESMTPSA id c11sm12834278wmb.45.2020.07.03.03.37.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Jul 2020 03:37:59 -0700 (PDT) Date: Fri, 3 Jul 2020 11:37:57 +0100 From: "Leif Lindholm" To: "Gao, Liming" Cc: "devel@edk2.groups.io" , "ard.biesheuvel@arm.com" , Laszlo Ersek , Rebecca Cran , Andrew Fish , "Justen, Jordan L" , "Kinney, Michael D" Subject: Re: [edk2-devel] License Check - was OvmfPkg: Adding support for bhyve as OvmfPkg/Bhyve Message-ID: <20200703103757.GU6739@vanye> References: <92612908-44f4-30d4-ae3a-3ad595e2141f@bsdio.com> <68b55c49-1e49-e082-ca38-68c3d321611b@redhat.com> <20200702105419.GN6739@vanye> <20200702141323.GQ6739@vanye> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Jul 03, 2020 at 01:40:26 +0000, Gao, Liming wrote: > > >>> *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 :) > > People may miss it. So, the checker is helpful to detect the issue. The feature is useful, but enabling it by default is not the correct decision for all TianoCore repos, and the situation for non-bsd+patent contributions is less than ideal. > > > > > > But also because: > > > - The thread that spawned this also raised the problem of > > > machine-generated files. > > This is a gap. We have no rule for the generated file. > > > > - 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. > > > > The patch to add the license checker is reviewed in edk2 mail list > for several weeks. > I don't get other comments. Can you give the suggestion on how to > improve the communication in edk2 community? I think that for something as fundamental as this, we need to actively chase feedback. I know that I will never manage to always read all emails to the lists, so there is always a risk I will miss something I'm not cc:d on. For something with as big an impact as a tightening of requirements in PatchCheck.py, if sufficient feedback (like at least 2-3 maintainers outside of BaseTools) has not been received, then it would make sense to ping *all* maintainers, alternatively ping the stewards and ask us to go gather feedback. > Besides, there is another new checker of ECC to check coding style > for each patch. Can you give your comment? > https://edk2.groups.io/g/devel/message/61966 I have never managed to get ECC running in any of my setups. Perhaps I should start trying to track down why, or at least raise a bugzilla for someone else to investigate. Regards, Leif > > 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 :-( > > Yes. This checker is for current allowed license. It doesn't resolve this issue. > > Thanks > Liming > > > > >