On 11/17/2012 11:28 PM, Robin H. Johnson wrote:
> On Sat, Nov 17, 2012 at 11:06:38PM -0500, Richard Yao wrote:
>> On 11/17/2012 10:39 PM, Greg KH wrote:
>>> Anyway, I now see a _very_ dangerous commit in the "Copyright" branch
>>> that better not get merged into the tree, as it's wrong, and illegal
>>> under all countries that follow the "normal" body of Copyright Law.  It
>>> should be removed right now before someone gets into trouble, not the
>>> least of which would be the orginization that the copyright is now being
>>> attributed to.
>>>
>>> Come on people, this is basic copyright law, it's not something
>>> radically new.  It's something that _all_ software developers should
>>> know, either from school, or any company they have ever worked at.
>>>
>>> Please fix this now.
>> klondike discussed the copyright branch changes with robbat2 before they
>> started and there was no problem at the time. We have retained all
>> copyright notices and looking at the branch, I find nothing objectionable.
> To note here, since I was CC'd directly:
> I said changed files should get the modified notice, as Gentoo should have
> copyright on the changes that are explicitly new by Gentoo. I didn't say to 
> add
> it to every file in the repo (but I will admit that I didn't tell him not to
> either).
> 
> I'll state it clearly what should be the case:
> - the s/systemd/eudev/ line, and insertion of "From prior code in systemd and
>   pre-systemd udev" being added now is fine.
> - WHEN substantial changes are made to an existing file, the copyright
>   attribution should be amended to include the Gentoo Foundation. The
>   attribution should NOT be changed before this. Better text given the 
> existing
>   wording would be:
>   Portions Copyright 2012 Gentoo Foundation.
> 
> - Files that have no copyright notice should NOT be touched until such time as
>   a major addition is added to them.
> 
> http://dpaste.com/832634/ is what I approved with klondike (his 2nd paste to 
> me
> in the discussion).
> 
> Copying out of the pastebin so we have a permanent record.
>> Original:
>> /***
>> This file is part of systemd.
>>
>> Copyright 2011 Lennart Poettering
>>
>> systemd is free software; you can redistribute it and/or modify it
>> under the terms of the GNU Lesser General Public License as published by
>> the Free Software Foundation; either version 2.1 of the License, or
>> (at your option) any later version.
>>
>> systemd is distributed in the hope that it will be useful, but
>> WITHOUT ANY WARRANTY; without even the implied warranty of
>> MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
>> Lesser General Public License for more details.
>>
>> You should have received a copy of the GNU Lesser General Public License
>> along with systemd; If not, see <http://www.gnu.org/licenses/>.
>> ***/
>>
>> Modification:
>>
>> /***
>> This file is part of eudev.
>> From prior code in systemd and pre-systemd udev.
>>
>> Copyright 2011 Lennart Poettering
>> Copyright 2012 Gentoo Foundation
>>
>> eudev is free software; you can redistribute it and/or modify it
>> under the terms of the GNU Lesser General Public License as published by
>> the Free Software Foundation; either version 2.1 of the License, or
>> (at your option) any later version.
>>
>> eudev is distributed in the hope that it will be useful, but
>> WITHOUT ANY WARRANTY; without even the implied warranty of
>> MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
>> Lesser General Public License for more details.
>>
>> You should have received a copy of the GNU Lesser General Public License
>> along with eudev; If not, see <http://www.gnu.org/licenses/>.
>> ***/

These changes were made in a branch so that they could be reviewed for
the consequences of such misunderstandings before going into HEAD. I
will let klondike fix the branch. We can review it again after he has
finished.

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to