Re: [tracecompass-dev] Removing contributor list from copyright header

[Matthew Khouzam <matthew.khouzam@xxxxxxxxxxxx>]

On 15-08-06 02:39 PM, Oberhuber, Martin wrote:
> 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.
I agree 100% with this, it adds to the search engine visibility of a
name. Which looks very good on a cv. This is actually a selling tool
used to recruit into open source too.

I can always say: your code is visible and stuff. But head hunters afaik
do not browse git trees as much as they should.
> 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