Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [tracecompass-dev] Removing contributor list from copyright header

Fust FYI, we also discussed this on the Eclipse Platform PMC yesterday:
https://wiki.eclipse.org/Eclipse/PMC

John Arthorne brought up two good points:
1. for some contributors, having their name listed in the Copyright Header is a small signal of appreciation.
2. the copyright headers never claimed to be complete or accurate, since it always said "... and others" in the first line.
    Thus there is no urgency in pruning or cleaning them.

So what we decided for the Eclipse Platform was that
- We will not require updating copyright headers any more
- If a contributor's patch contains a copyright update adding his name, we will keep it in there (no forced pruning)
- We will not remove existing copyright headers from any files.

I felt that these were good arguments that you might want to consider ...

Thanks,
Martin
--
Martin Oberhuber, SMTS / Product Owner - Development Tools, Wind River
direct +43.662.457915.85  fax +43.662.457915.6


-----Original Message-----
From: tracecompass-dev-bounces@xxxxxxxxxxx [mailto:tracecompass-dev-bounces@xxxxxxxxxxx] On Behalf Of Alexandre Montplaisir
Sent: Thursday, August 06, 2015 8:15 PM
To: matthew.khouzam@xxxxxxxxxxxx
Cc: tracecompass developer discussions
Subject: Re: [tracecompass-dev] Removing contributor list from copyright header

No of course, no hurry to remove all existing contributor lists. All I'm suggesting is that we start removing them iteratively, as we update copyright headers for example.


On 2015-07-30 09:32 AM, Matthew Khouzam wrote:
> So to be clear, there will NOT be a patch removing everyone's name at 
> once (that would make a lot of noise).
>
> But we will stop writing our names. This sounds like less work, I'm 
> all for it!
>
> On 15-07-30 07:01 AM, Bernd Hufmann wrote:
>> Hi Alex
>>
>> thanks for letting us know about the discussions and decision. I'm ok 
>> to not update the contributor list anymore and, for new files, not 
>> include it. About removing the contributor list from existing files 
>> actively, I'm not sure. I'd like to hear the other Trace Compass 
>> committers' opinion about it before going ahead with it. Let's give 
>> them a chance to put their comment on this email thread.
>>
>> /Bernd
>>
>> On 07/28/2015 07:21 PM, Alexandre Montplaisir wrote:
>>> Hi all,
>>>
>>> As discussed in bug 435933 [1], it is no longer necessary to list 
>>> individual contributors in the source files's copyright headers.
>>> Eclipse IP checks now use the git log to track contributions, so 
>>> this list is now effectively useless. The same information is 
>>> available with "git shortlog" and similar commands. It also 
>>> increases patch noise and can cause unwanted merge conflicts.
>>>
>>> CDT now recommends to not update this list anymore [2].
>>>
>>> What do you say we drop the contributor lists altogether? Maybe not 
>>> actively remove it from everywhere right now, but as we modify files 
>>> we can take the opportunity to remove it, and new files could come 
>>> with no list at all?
>>>
>>> Cheers,
>>> Alex
>>>
>>>
>>> [1] https://bugs.eclipse.org/bugs/show_bug.cgi?id=435933
>>> [2] http://dev.eclipse.org/mhonarc/lists/cdt-dev/msg29374.html
>>> _______________________________________________
>>> tracecompass-dev mailing list
>>> tracecompass-dev@xxxxxxxxxxx
>>> To change your delivery options, retrieve your password, or 
>>> unsubscribe from this list, visit 
>>> https://dev.eclipse.org/mailman/listinfo/tracecompass-dev
>> _______________________________________________
>> tracecompass-dev mailing list
>> tracecompass-dev@xxxxxxxxxxx
>> To change your delivery options, retrieve your password, or 
>> unsubscribe from this list, visit 
>> https://dev.eclipse.org/mailman/listinfo/tracecompass-dev
> _______________________________________________
> tracecompass-dev mailing list
> tracecompass-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or 
> unsubscribe from this list, visit 
> https://dev.eclipse.org/mailman/listinfo/tracecompass-dev

_______________________________________________
tracecompass-dev mailing list
tracecompass-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://dev.eclipse.org/mailman/listinfo/tracecompass-dev


Back to the top