Re: [Batch 5 - patch 08/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 108

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, May 23, 2019 at 5:29 AM Thomas Gleixner <tglx@xxxxxxxxxxxxx> 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 <tglx@xxxxxxxxxxxxx>
> ---
> 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 <linux/bitops.h>
> --- 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.



[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux