From owner-dev-commits-src-main@freebsd.org Tue Apr 20 17:02:14 2021 Return-Path: Delivered-To: dev-commits-src-main@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 9509E5DFE5E; Tue, 20 Apr 2021 17:02:14 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4FPqjf1W70z3Fs9; Tue, 20 Apr 2021 17:02:13 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (localhost [127.0.0.1]) by gndrsh.dnsmgr.net (8.13.3/8.13.3) with ESMTP id 13KH2B25071848; Tue, 20 Apr 2021 10:02:11 -0700 (PDT) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: (from freebsd@localhost) by gndrsh.dnsmgr.net (8.13.3/8.13.3/Submit) id 13KH2BEF071847; Tue, 20 Apr 2021 10:02:11 -0700 (PDT) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <202104201702.13KH2BEF071847@gndrsh.dnsmgr.net> Subject: Re: git: 686cf5468c4a - main - flex: Regen bootstrap files In-Reply-To: To: Warner Losh Date: Tue, 20 Apr 2021 10:02:11 -0700 (PDT) CC: Jung-uk Kim , "Rodney W. Grimes" , src-committers , "" , dev-commits-src-main@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 4FPqjf1W70z3Fs9 X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-BeenThere: dev-commits-src-main@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for the main branch of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 20 Apr 2021 17:02:14 -0000 [ Charset UTF-8 unsupported, converting... ] > On Tue, Apr 20, 2021 at 10:24 AM Jung-uk Kim wrote: > > > On 21. 4. 20., Rodney W. Grimes wrote: > > >> On 21. 4. 20., Rodney W. Grimes wrote: > > >>>> The branch main has been updated by jkim: > > >>>> > > >>>> URL: > > https://cgit.FreeBSD.org/src/commit/?id=686cf5468c4a85c2a6385cdbab2383900876a41d > > >>>> > > >>>> commit 686cf5468c4a85c2a6385cdbab2383900876a41d > > >>>> Author: Jung-uk Kim > > >>>> AuthorDate: 2021-04-19 18:20:51 +0000 > > >>>> Commit: Jung-uk Kim > > >>>> CommitDate: 2021-04-19 18:20:51 +0000 > > >>>> > > >>>> flex: Regen bootstrap files > > >>>> > > >>>> This also partially reverts r326025 (8a16b7a18f5d). I do not see > > any > > >>>> point of adding SPDX tag in generated file. > > >>> > > >>> The SPDX identifies the copyright that follows it, so please put this > > back. > > >> > > >> It is a generated file. If you really think it is necessary, you need > > >> to patch its source, i.e., contrib/flex/src/parse.y, and regen the file. > > > > > > Just because it is a generated file does not change the fact that > > > its contents are copyrighted. Leaving the copyright out at best > > > defaults to Bern convention, so the fact of the mater is the contents > > > is copyrighted, and it is best to explicitly state that it is. > > > > Of course, it is copyrighted. My point was parse.y should have been > > patched, not the generated file, if necessary. > > > > Yes. It was a mistake to put it into the generated file. He removed the SPDX from parse.y, now says that is the right place to put the SPDX. You agree. So can we have this undone? > > >> However, I don't see much sense in automatically adding BSD license in > > >> every generated file in the first place. In fact, I think it should not > > >> appear on generated file at all. It may say something along the line of > > >> "generated by flex" instead but IANAL. > > > > > > Well, someone before you did see since in it, and hence put that SPDX in > > it. > > > > Let me put it in another way. Do we really want to tag contrib code? > > > > I tend to agree. > > I tend to agree because the SPDX tags are generally only looked at in the > source tree by existing tools. The generated code won't be looked at > typically. > We should work to get this upstream, imho. Is parse.y not a source file? What am I missing? > By itself, the SPDX tag is just information. There's no legal requirement in > copyright law to have it. Current project policy states they are just > informative, > though there's some desire to have a good framework to allow code with > just a copyright and a SDPX tag into the tree, so long as that framework > can be used to know what the license is for any file in a clear and > unambiguous > way. > > Warner -- Rod Grimes rgrimes@freebsd.org