From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AA887C282DD for ; Thu, 23 May 2019 11:54:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6633720881 for ; Thu, 23 May 2019 11:54:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729762AbfEWLyy (ORCPT ); Thu, 23 May 2019 07:54:54 -0400 Received: from mail-vk1-f195.google.com ([209.85.221.195]:38521 "EHLO mail-vk1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729361AbfEWLyx (ORCPT ); Thu, 23 May 2019 07:54:53 -0400 Received: by mail-vk1-f195.google.com with SMTP id p24so1282458vki.5 for ; Thu, 23 May 2019 04:54:53 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=xsyqsSG9avkwn5rP2gipJbDhkwFG0EZ1O01OMnu2zag=; b=iurJUSbMlbMyi1HwIbBPvLVvYiT2LHnIBiQG0Xl+XSjV+ORl8NhyyyhqWgxCr0b61K TdnWt+L1Bmeoy0AvVq5WJ+z2f3uD2MNRln6g/N8QJLpFJdhTvGsmDXD8q2s572Uaq4/e Azdj72OU8OAXVfraCkyVRmJBvkkFBeNE3Iz3+YwVDzHVyjOH+rp68EzoX/C0Hvci29FA +lEvCk1YIpEze5upy/EAKkkclqRekxbgVLRWAfQrwjG2BOz9vG4Ckchla69usPyVXzC+ b/hPFVt+xJPsflYQiNLPHBvujsPPoSBBZLlfYU/kDZqO2GCLQOOE0Q3ejJLbs4SHy/ib Onqw== X-Gm-Message-State: APjAAAUZv25CM8ae0Kswildb8ZXA4HDyimPPYcNctp39lym6gXz+DbuJ QP66GhLYAIBncp9AZMtjWbkidcj5Ndl+iTCWlFgoP96A X-Google-Smtp-Source: APXvYqyZEm3yc3oc7Kml5nAu51Zc8uMqC9w9BXzuUr+9Kp/i70W3EJQ2jPNHQfnLSKFijfveTOXUK3Zda5eEZxQLhrQ= X-Received: by 2002:a1f:4ec6:: with SMTP id c189mr1360410vkb.17.1558612492540; Thu, 23 May 2019 04:54:52 -0700 (PDT) MIME-Version: 1.0 References: <20190523091437.334232837@linutronix.de> <20190523091650.100885018@linutronix.de> In-Reply-To: <20190523091650.100885018@linutronix.de> From: Richard Fontana Date: Thu, 23 May 2019 07:54:41 -0400 Message-ID: Subject: Re: [Batch 5 - patch 08/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 108 To: Thomas Gleixner Cc: linux-spdx@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: linux-spdx-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-spdx@vger.kernel.org On Thu, May 23, 2019 at 5:29 AM Thomas Gleixner wrote: > > Based on 1 normalized pattern(s): > > gnupg is free software you can redistribute it and or modify it > under the terms of the gnu general public license as published by > the free software foundation either version 2 of the license or at > your option any later version gnupg is distributed in the hope that > it will be useful but without any warranty without even the implied > warranty of merchantability or fitness for a particular purpose see > the gnu general public license for more details you should have > received a copy of the gnu general public license along with this > program if not write to the free software foundation inc 59 temple > place suite 330 boston ma 02111 1307 usa At least some of these bear FSF copyrights. I noticed some other patches that made reference to "GNU CC" (suggesting the code may have been taken from old versions of GCC?) but I am not sure they had FSF copyrights. I feel it is especially awkward to remove the continuously-preferred legal notice of the copyright holder, particularly when that copyright holder is also the recognized license steward of the license in question. Since John Sullivan is on this list maybe he can comment. Obviously if the FSF is okay with it, it's fine, but I am not sure that is John's view given a comment he made the other day. As a generalization of the above concern, it seems relatively problematic to me to replace license notices that were attached to code taken from non-Linux projects, where that is evident, compared to code contributed to Linux in the first instance. (Similarly, I would normally not advise Red Hat employees to alter upstream third-party legal notices downstream, except in very unusual circumstances.) Richard > > extracted by the scancode license scanner the SPDX license identifier > > GPL-2.0-or-later > > has been chosen to replace the boilerplate/reference in 4 file(s). > > Signed-off-by: Thomas Gleixner > --- > https://git.kernel.org/pub/scm/linux/kernel/git/tglx/linux-spdx.git/commit/?h=batch5&id=cad8bc93c21d > --- > lib/mpi/mpi-bit.c | 15 +-------------- > lib/mpi/mpi-cmp.c | 15 +-------------- > lib/mpi/mpicoder.c | 15 +-------------- > lib/mpi/mpiutil.c | 15 +-------------- > 4 files changed, 4 insertions(+), 56 deletions(-) > > --- a/lib/mpi/mpi-bit.c > +++ b/lib/mpi/mpi-bit.c > @@ -1,21 +1,8 @@ > +// SPDX-License-Identifier: GPL-2.0-or-later > /* mpi-bit.c - MPI bit level fucntions > * Copyright (C) 1998, 1999 Free Software Foundation, Inc. > * > * This file is part of GnuPG. > - * > - * GnuPG is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License as published by > - * the Free Software Foundation; either version 2 of the License, or > - * (at your option) any later version. > - * > - * GnuPG is distributed in the hope that it will be useful, > - * but WITHOUT ANY WARRANTY; without even the implied warranty of > - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > - * GNU General Public License for more details. > - * > - * You should have received a copy of the GNU General Public License > - * along with this program; if not, write to the Free Software > - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA > */ > > #include "mpi-internal.h" > --- a/lib/mpi/mpi-cmp.c > +++ b/lib/mpi/mpi-cmp.c > @@ -1,21 +1,8 @@ > +// SPDX-License-Identifier: GPL-2.0-or-later > /* mpi-cmp.c - MPI functions > * Copyright (C) 1998, 1999 Free Software Foundation, Inc. > * > * This file is part of GnuPG. > - * > - * GnuPG is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License as published by > - * the Free Software Foundation; either version 2 of the License, or > - * (at your option) any later version. > - * > - * GnuPG is distributed in the hope that it will be useful, > - * but WITHOUT ANY WARRANTY; without even the implied warranty of > - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > - * GNU General Public License for more details. > - * > - * You should have received a copy of the GNU General Public License > - * along with this program; if not, write to the Free Software > - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA > */ > > #include "mpi-internal.h" > --- a/lib/mpi/mpicoder.c > +++ b/lib/mpi/mpicoder.c > @@ -1,21 +1,8 @@ > +// SPDX-License-Identifier: GPL-2.0-or-later > /* mpicoder.c - Coder for the external representation of MPIs > * Copyright (C) 1998, 1999 Free Software Foundation, Inc. > * > * This file is part of GnuPG. > - * > - * GnuPG is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License as published by > - * the Free Software Foundation; either version 2 of the License, or > - * (at your option) any later version. > - * > - * GnuPG is distributed in the hope that it will be useful, > - * but WITHOUT ANY WARRANTY; without even the implied warranty of > - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > - * GNU General Public License for more details. > - * > - * You should have received a copy of the GNU General Public License > - * along with this program; if not, write to the Free Software > - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA > */ > > #include > --- a/lib/mpi/mpiutil.c > +++ b/lib/mpi/mpiutil.c > @@ -1,21 +1,8 @@ > +// SPDX-License-Identifier: GPL-2.0-or-later > /* mpiutil.ac - Utility functions for MPI > * Copyright (C) 1998, 1999 Free Software Foundation, Inc. > * > * This file is part of GnuPG. > - * > - * GnuPG is free software; you can redistribute it and/or modify > - * it under the terms of the GNU General Public License as published by > - * the Free Software Foundation; either version 2 of the License, or > - * (at your option) any later version. > - * > - * GnuPG is distributed in the hope that it will be useful, > - * but WITHOUT ANY WARRANTY; without even the implied warranty of > - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > - * GNU General Public License for more details. > - * > - * You should have received a copy of the GNU General Public License > - * along with this program; if not, write to the Free Software > - * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA > */ > > #include "mpi-internal.h" > > -- Richard Fontana Senior Commercial Counsel Red Hat, Inc.