On Tue, Nov 07, 2017 at 07:29:03PM +0100, Greg Kroah-Hartman wrote: > On Tue, Nov 07, 2017 at 09:20:42AM -0800, Darrick J. Wong wrote: > > On Tue, Nov 07, 2017 at 08:39:40AM +0100, Greg Kroah-Hartman wrote: > > > On Mon, Nov 06, 2017 at 11:20:40PM -0800, Christoph Hellwig wrote: > > > > NAK, for both the libxfs patch and the kernel one. > > > > > > What libxfs patch? And what "kernel one" are you referring to here? > > > > > > > I wrote the file and it has no copyright header because it conatians > > > > trivial, non-copyrightable code. > > > > > > What file exactly? > > > > > > And from what I know, there is nothing that is "non-copyrightable". > > > > > > And this isn't changing the copyright of _ANYTHING_ it is just putting > > > the explicit license of the file, on each file in the kernel, because it > > > needs to be tracked. > > > > > > > I don't know why people think they can touch license information on > > > > files I've written without even asking me. > > > > > > Nothing was changed, the license should be the exact same as it was > > > before. But as I don't know what file you are referring to here, it's a > > > bit hard to determine what you are talking about exactly :( > > > > fs/xfs/libxfs/xfs_cksum.h > > Given that it had no license text on it at all, it "defaults" to GPLv2, > so the GPLv2 SPDX identifier was added to it. I'll point out here that this file is shared with a userspace package that has a mixed LGPL/GPL code base, so even if we disregard what Christoph says, this file could actually be LGPL (like fs/xfs/libxfs/xfs_fs.h) and not GPL. So from that perspective alone, your process on deciding what license tag should be used is flawed and these changes needed, at minimum, maintainer review. IMO, unannounced, unreviewed tree wide change via a back-door commits sent straight to Linus reek of an attempt to avoid review and oversight. And that is *completely unacceptible* when making claims about important details like licenses for *code you do not know anything about*. We have a documented process for a reason: to stop shit like this from happening. > No copyright was changed, nothing at all happened except we explicitly > list the license of the file, instead of it being "implicit" before. You keep saying "no copyright has changed", despite being given an explicit statement by the code author that this is *exactly what you have done*. Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- 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
