From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Mon, 08 Apr 2019 05:12:47 -0700 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id B3091307E040; Mon, 8 Apr 2019 12:12:46 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-121-29.rdu2.redhat.com [10.10.121.29]) by smtp.corp.redhat.com (Postfix) with ESMTP id 05E7F5B681; Mon, 8 Apr 2019 12:12:45 +0000 (UTC) Subject: Re: [edk2-devel] [edk2] [PATCH V3] Change EDK II to BSD+Patent License To: devel@edk2.groups.io, michael.d.kinney@intel.com References: From: "Laszlo Ersek" Message-ID: Date: Mon, 8 Apr 2019 14:12:44 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.42]); Mon, 08 Apr 2019 12:12:46 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 04/04/19 01:42, Michael D Kinney wrote: > 17a33094a9 edk2: Add License-History.txt I have a minuscule comment for this patch. In the "Key Dates" section, you list two dates: - July 18, 2017 - April 9, 2019 The 2nd date refers to a commit (that is, not patch authorship) date. I think that's fine. The first date however refers to a patch authorship date, not a commit date. I don't feel too strongly about this, but I think the consistency could be improved. In particular, from the commits listed in , the most relevant seems to be b6538c118ae8. The authorship date on that patch is indeed 2017-07-18; however its commit date is 2017-08-03. If, for one of the "key dates" above, we use a "commit date" (and not an "authorship date"), then we might want to do the same for the other "key date" (i.e., we might want to replace "July 18, 2017" with "August 3, 2017"). Anyway, I'll leave it up to you; I'm fine with the patch as-is. Reviewed-by: Laszlo Ersek > 8ee83c5dcd edk2: Change License.txt from 2-Clause BSD to BSD+Patent Reviewed-by: Laszlo Ersek > bc61733285 edk2: Replace BSD License with BSD+Patent License Reviewed-by: Laszlo Ersek > eece5f8a6e edk2: Remove Contributions.txt and update Readme.md Reviewed-by: Laszlo Ersek Thanks Laszlo