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