From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43651) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bnLdw-0005bF-4w for qemu-devel@nongnu.org; Fri, 23 Sep 2016 04:15:37 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bnLdr-00078Q-To for qemu-devel@nongnu.org; Fri, 23 Sep 2016 04:15:35 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57638) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bnLdr-00078F-Mq for qemu-devel@nongnu.org; Fri, 23 Sep 2016 04:15:31 -0400 From: Markus Armbruster References: <000301d213c7$7142f980$53c8ec80$@ru> <87y42l93dr.fsf@dusky.pond.sub.org> <20160921100718.GO15535@redhat.com> <87mvj11ljr.fsf@dusky.pond.sub.org> <7500849B-8BCC-4743-92A6-5DBD4D8575E8@nutanix.com> <20160921143546.GS15535@redhat.com> <87zin1w9v4.fsf@dusky.pond.sub.org> <20160922080004.GB352@redhat.com> <87r38cjgon.fsf@dusky.pond.sub.org> Date: Fri, 23 Sep 2016 10:15:28 +0200 In-Reply-To: (Eric Blake's message of "Thu, 22 Sep 2016 08:30:09 -0500") Message-ID: <87intnxc9r.fsf@dusky.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PATCH] replay: Fix build with -Werror=unused-result List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake Cc: "Daniel P. Berrange" , Luiz Capitulino , Pavel Dovgalyuk , qemu-devel , Pavel Dovgalyuk , Felipe Franciosi Eric Blake writes: > On 09/22/2016 06:51 AM, Markus Armbruster wrote: >>> >>> I think the file probably should not have been listed as public domain >>> in the first place, as its initial contents were copied from qemu-common.h >>> which is not public domain. >> >> Ewww! Needs fixing. > > Indeed. Commit 5c02632 shows the initial file creation, but Luiz used a > different email address at that time. What would be the best line to > list as a copyright holder on both qemu-common.h and on compiler.h? Since qemu-common.h carried no licence notice back then[*], adding the line "/* public domain */" was to compiler.h was wrong, and is probably legally void. The immediate fix is therefore dropping that line. Then the project-wide license applies: GPLv2+. We can always copy the standard GPLv2+ licence notice to files that don't have one, if we think that's useful. >> Since qemu-common.h carries no license, GPLv2+ applies. [*] It still doesn't, but that's immaterial.