Re: [PATCH v2 09/10] udf: Replace license notice with SPDX identifier - Greg Kroah-Hartman

From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: "Pali Rohár" <pali@kernel.org>, "Bagas Sanjaya" <bagasdotme@gmail.com>
Cc: "Linux SPDX Licenses" <linux-spdx@vger.kernel.org>,
	"Linux DRI Development" <dri-devel@lists.freedesktop.org>,
	"Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>,
	"Linux Networking" <netdev@vger.kernel.org>,
	"Linux ARM" <linux-arm-kernel@lists.infradead.org>,
	"Linux Staging Drivers" <linux-staging@lists.linux.dev>,
	"Linux Watchdog Devices" <linux-watchdog@vger.kernel.org>,
	"Linux Kernel Actions" <linux-actions@lists.infradead.org>,
	"Diederik de Haas" <didi.debian@cknow.org>,
	"Kate Stewart" <kstewart@linuxfoundation.org>,
	"Philippe Ombredanne" <pombredanne@nexb.com>,
	"Thomas Gleixner" <tglx@linutronix.de>,
	"David Airlie" <airlied@redhat.com>,
	"Karsten Keil" <isdn@linux-pingi.de>,
	"Jay Vosburgh" <j.vosburgh@gmail.com>,
	"Andy Gospodarek" <andy@greyhouse.net>,
	"David S. Miller" <davem@davemloft.net>,
	"Eric Dumazet" <edumazet@google.com>,
	"Jakub Kicinski" <kuba@kernel.org>,
	"Paolo Abeni" <pabeni@redhat.com>,
	"Sam Creasey" <sammy@sammy.net>,
	"Dominik Brodowski" <linux@dominikbrodowski.net>,
	"Daniel Mack" <daniel@zonque.org>,
	"Haojian Zhuang" <haojian.zhuang@gmail.com>,
	"Robert Jarzmik" <robert.jarzmik@free.fr>,
	"Wim Van Sebroeck" <wim@linux-watchdog.org>,
	"Guenter Roeck" <linux@roeck-us.net>, "Jan Kara" <jack@suse.com>,
	"Andreas Färber" <afaerber@suse.de>,
	"Manivannan Sadhasivam" <mani@kernel.org>,
	"Simon Horman" <simon.horman@corigine.com>
Subject: Re: [PATCH v2 09/10] udf: Replace license notice with SPDX identifier
Date: Sat, 13 May 2023 18:59:30 +0900	[thread overview]
Message-ID: <2023051328-ambiance-outlook-cc30@gregkh> (raw)
In-Reply-To: <20230513094851.d3qdlfbvfc67vpdl@pali>

On Sat, May 13, 2023 at 11:48:51AM +0200, Pali Rohár wrote:
> On Friday 12 May 2023 17:06:20 Bagas Sanjaya wrote:
> > diff --git a/fs/udf/udftime.c b/fs/udf/udftime.c
> > index fce4ad976c8c29..d0fce5348fd3f3 100644
> > --- a/fs/udf/udftime.c
> > +++ b/fs/udf/udftime.c
> > @@ -1,21 +1,4 @@
> > -/* Copyright (C) 1993, 1994, 1995, 1996, 1997 Free Software Foundation, Inc.
> > -   This file is part of the GNU C Library.
> > -   Contributed by Paul Eggert (eggert@twinsun.com).
> > -
> > -   The GNU C Library is free software; you can redistribute it and/or
> > -   modify it under the terms of the GNU Library General Public License as
> > -   published by the Free Software Foundation; either version 2 of the
> > -   License, or (at your option) any later version.
> > -
> > -   The GNU C Library 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
> > -   Library General Public License for more details.
> > -
> > -   You should have received a copy of the GNU Library General Public
> > -   License along with the GNU C Library; see the file COPYING.LIB.  If not,
> > -   write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
> > -   Boston, MA 02111-1307, USA.  */
> > +// SPDX-License-Identifier: GPL-2.0-only
> >  
> >  /*
> >   * dgb 10/02/98: ripped this from glibc source to help convert timestamps
> 
> Please, dot not do this. It is really rude to people who worked on it in
> past (even if they do not care about this particular file anymore) as
> technically they still have ownership of this code / file. And such
> change never remove their ownership or copyright in most countries.
> 

Yes, copyright lines should never be removed.

Bagas, I recommend taking the Linux Foundation "copyright and licensing"
online course before you respin this series.  It's free online and might
help you in understanding some of the issues involved here and prevent
you from making these types of mistakes in the future.

thanks,

greg k-h