2017-12-10 15:08:52

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH v2] iio: mma8452: replace license description with SPDX specifier

On Sat, 18 Nov 2017 23:23:57 -0500
harinath Nampally <[email protected]> wrote:

> > This replaces the custom license information text with the appropriate
> > SPDX identifier. While the information here stays the same, it is easier
> > to read.
> > Signed-off-by: Martin Kepplinger <[email protected]>
> > Acked-by: Peter Meerwald-Stadler <[email protected]>
> Acked-by: Harinath Nampally <[email protected]>

Applied. I still not totally sure that the intent of introducing SPDX
specifiers was to replace rather than supplement the license text.
It's been pointed out for instance that some of the BSD licenses have
explicit names in the text relevant to a particular file. For GPL v2
that isn't the case though so I suppose this one does no harm.

Jonathan
>
> On Sat, Nov 18, 2017 at 11:29 AM, Philippe Ombredanne
> <[email protected]> wrote:
> > On Sat, Nov 18, 2017 at 4:53 PM, Jonathan Cameron <[email protected]> wrote:
> >> On Sat, 18 Nov 2017 10:10:11 +0100
> >> Martin Kepplinger <[email protected]> wrote:
> >>
> >>> This replaces the custom license information text with the appropriate
> >>> SPDX identifier. While the information here stays the same, it is easier
> >>> to read.
> >>>
> >>> Signed-off-by: Martin Kepplinger <[email protected]>
> >>> Acked-by: Peter Meerwald-Stadler <[email protected]>
> >>
> >> I'm not 100% sure the intent of the SPDX work is to remove
> >> existing licence text. So far the big sets have only been
> >> adding tags to files missing their licenses entirely...
> >>
> >> Anyone found any specific guidance on this?
> >
> > Jonathan:
> > you might want to check the doc patches from tglx [1] as well as
> > several related patches from greg k-h such as these [2] and his
> > initial pull [3]
> >
> > To get a lot of details you can check all the recent SPDX-related posts too [4]
> >
> > [1] https://marc.info/?l=linux-kernel&m=151051532322831&w=2
> > [2] https://marc.info/?l=linux-kernel&m=151068111802610&w=2
> > [3] https://marc.info/?l=linux-kernel&m=150963579219623&w=2
> > [4] https://marc.info/?l=linux-kernel&w=2&r=1&s=spdx&q=b
> > --
> > Cordially
> > Philippe Ombredanne


2017-12-10 19:00:59

by Philippe Ombredanne

[permalink] [raw]
Subject: Re: [PATCH v2] iio: mma8452: replace license description with SPDX specifier

On Sun, Dec 10, 2017 at 4:08 PM, Jonathan Cameron <[email protected]> wrote:
> On Sat, 18 Nov 2017 23:23:57 -0500
> harinath Nampally <[email protected]> wrote:
>
>> > This replaces the custom license information text with the appropriate
>> > SPDX identifier. While the information here stays the same, it is easier
>> > to read.
>> > Signed-off-by: Martin Kepplinger <[email protected]>
>> > Acked-by: Peter Meerwald-Stadler <[email protected]>
>> Acked-by: Harinath Nampally <[email protected]>
>
> Applied. I still not totally sure that the intent of introducing SPDX
> specifiers was to replace rather than supplement the license text.
> It's been pointed out for instance that some of the BSD licenses have
> explicit names in the text relevant to a particular file. For GPL v2
> that isn't the case though so I suppose this one does no harm.

Thanks!
There might be indeed a few corner cases that would require some
discussions and the best way would be to have a proper sign off of the
authors when possible in these cases (or all cases)
But to me at least, reducing the boilerplate to the max (such as
replacing 50 lines of legalese with a single SPDX id line) is a big
boost of content/noise ratio that we should thrive for.
--
Cordially
Philippe Ombredanne