Re: [patch V2 02/11] LICENSES: Add the GPL 2.0 license

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

 



On Sat, 18 Nov 2017, Charlemagne Lasse wrote:
> 2017-11-16 19:33 GMT+01:00 Thomas Gleixner <tglx@xxxxxxxxxxxxx>:
> > Add the full text of the GPL 2.0 license to the kernel tree.  It was
> > copied directly from:
> >
> >    https://spdx.org/licenses/GPL-2.0.html#licenseText
> >
> > Add the required tags for reference and tooling.
> >
> > Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> 
> NACKed-by: Charlemagne Lasse <charlemagnelasse@xxxxxxxxx>
> 
> This is neither the GPL-2.0 from https://www.gnu.org/licenses/gpl-2.0.txt
> (which you should have used) or
> https://spdx.org/licenses/GPL-2.0.html#licenseText
> 
> Please download the correct ASCII version from gnu.org and add your SPDX
> info in front of it. But I have also added detailed info about
> non-whitespace changes in your patch.

Indeed. I messed that up when I noticed that the license from the SPDX site
is indeed different from the COPYING file in a few minor aspects.

I started to rumage around to find out why and found the matching one on

  https://opensource.org/licenses/GPL-2.0

which is linked to from the SPDX page. I downloaded that version to check
the differences. The opensource page has the original version of the GPLv2
and not the one which got modified by FSF later on without changing the
version number.

I then decided to copy the text from COPYING, which is the right thing to
do as Jonas pointed, got dragged into dealing with a regression and forgot
about it. Neither did I update the changelog.

Thanks for spotting it!

I'll send out a revised version later.

Thanks,

	tglx


--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux