SPDX tags in file?

Mark Millard markmi at dsl-only.net
Thu Dec 7 04:17:43 UTC 2017


On 2017-Dec-6, at 7:57 PM, Steve Kargl <sgk at troutmask.apl.washington.edu> wrote:

> On Wed, Dec 06, 2017 at 09:17:06PM -0500, Pedro Giffuni wrote:
>> 
>>> It seems that the application of SPDX license tags
>>> has been automated and done without reviewing whether
>>> the tag is correct.  For example, the BSD-4-Clause
>>> tag has been placed in the files in lib/msun/bsdsrc.
>>> Given the UCB letter concerning removal of clauses
>>> 3 and 4, these files should probably have had the
>>> Copyright updated and a different SPDX clause applied.
>>> 
>>> -- 
>>> Steve
>> 
>> The initial sweep was done manually, but as you might have noticed, it 
>> covered a lot of files and mistakes are certainly possible.
>> 
>> The idea at this time is/was not to replace licenses: I am not a lawyer 
>> but I think we may have to look at who has touched a file before doing 
>> any license change. That may be a complex process.
>> 
>> This said. checking for bsd-4-clause is a pretty good opportunity to 
>> review and modernize code. If the code comes from another BSD (and 
>> particularly NetBSD as I noticed during the sweep), it is likely 
>> upstream has updated the license as well and there may be interesting 
>> changes involved.
>> 
> 
> Not all revisions apply to all four files
> 
> r1573   rgrimes  BSD 4.4 Lite
> r8870   rgrimes  Trailing whitespace
> r84210  dillon   Add __FBSDID
> r92887  obrien   Fix SCM ID's
> r92917  obrien   Remove __P() usage.
> r93211  bde      Resurrect Lite1
> r97407  keramida Assume __STDC__
> r108533 schweikh Typos and whitespace
> r129312 stefanf  Remove some kludges (use C99 hexadecimal constant)
> r138924 das      Cosmetic changes only
> r138925 das      GC unused declaration
> r150318 bde      Fixed aliasing bugs in TRUNC()
> r152566 bde      Removed an unused declaration and style bugs
> r169209 bde      Document current (slightly broken) handling of special values
> r169212 bde      Fix tgamma() on some special args
> r176449 das      Eliminate some warnings
> r226414 das      Fix some non-standard variable declarations.
> r325966 pfg      spdx
> 
> If you don't count UCB as upstream (aka r1573), then FreeBSD is
> upstream.  Looking at NetBSD the commit message for b_tgamma.c
> is "Add tgamma{,f} from FreeBSD via  rudolf, netbsd at eq dot cz".
> OpenBSD is a little more complicated, but its initial version 
> appeared in 2008 while FreeBSD's appeard in 1994.
> 
> IMHO (non-lawyer) opinion, the only thing that might rise to the
> level of Copyright-able material would be r169212.  Bruce did not
> add his name as he has done elsewhere.
> 
> BTW, OpenBSD uses a 3-clause BSD license.

That prompted an old memory about newer code
for OpenBSD so I took a look. . .

https://www.openbsd.org/policy.html says:

ISC
The ISC copyright is functionally equivalent to a two-term BSD
copyright with language removed that is made unnecessary by the
Berne convention. This is the preferred license for new code
incorporated into OpenBSD. A sample license is available in the
file /usr/share/misc/license.template.


===
Mark Millard
markmi at dsl-only.net



More information about the freebsd-hackers mailing list