Date: Wed, 1 Apr 2015 14:29:16 +0200 From: Mateusz Guzik <mjguzik@gmail.com> To: Gleb Smirnoff <glebius@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r280955 - in head/sys: modules/notrandom dev/notrandom Message-ID: <20150401122916.GA10926@dft-labs.eu> In-Reply-To: <20150401121344.GC64665@FreeBSD.org> References: <20150401113628.GA16649@dft-labs.eu> <20150401114313.GZ64665@FreeBSD.org> <20150401115204.GB16649@dft-labs.eu> <20150401115729.GA64665@FreeBSD.org> <20150401120229.GA2201@dft-labs.eu> <20150401121344.GC64665@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Apr 01, 2015 at 03:13:44PM +0300, Gleb Smirnoff wrote: > Mateusz, > > On Wed, Apr 01, 2015 at 02:02:29PM +0200, Mateusz Guzik wrote: > M> > You can add the seven_region variable to dev/null/null.c and implement > M> > the new device here. Now notrandom.c shares > 50% of code with null.c. > M> > M> Yeah I know, in fact I got most of the code from that driver. > M> > M> I just wanted to have a file with my own copyright in the tree, what's > M> so bad about it. > M> > M> If you feel strongly about it feel free to move it, but I would prefer > M> if you did not. > > You are increasing size of kernel object file merely for a selfish reason. > Sure, but who does not? > While some systems do not care about kernel object size, some embedded do. > Size of source code also matters. You added new code in FreeBSD, that > is likely to be supported for a long time. Of course maintaing two files, > where one is paste of other, requires more efforts than maintaining > single file. > Fair point, but cmon. We can cut IPv6 instead. Net win. > Mateusz, if you keep the momentum, in the next years we will see lots > of files with your copyright. :) There is no reason to create one > just for this. Also, you can legitimately add your name into header of > null.c if you move code of notrandom there. > Yeah, but that's not the same as having my own file, even if I copy-pasted most of it. :( I understand you have concerns, but can we please leave this over the weekend? I'll be happy and maybe it will grow on you in the meantime. I must say this funcionality is so awesome it deserves a separate file regardless of who gets the copyright. -- Mateusz Guzik <mjguzik gmail.com>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150401122916.GA10926>