<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>You have a good point that automation would be improved. &nbsp;I agree that automation is always better than manual work. &nbsp;And this is one reason why I think the @authors tags are not good, since they often do NOT reflect who worked on the file. &nbsp;SVN has the true record.</div><div><br></div><div>In the case of the AUTHORS file, I''m not sure automation is really worth the effort. &nbsp;As we all know, becoming a committer is not an automatic process, so this is one more thing that the project lead has to do. &nbsp;(There are just a few anyway... I'm usually not the bottleneck. :-)</div><div><br></div><div>So future distributions will have the AUTHORS list, and of course SVN will have it. &nbsp;But there's also Ohloh, which automatically produces the list of committers as well as a timeline of each person's commits. &nbsp;See&nbsp;<a href="https://www.ohloh.net/p/dna/contributors">https://www.ohloh.net/p/dna/contributors</a>.</div><div><br></div><div>Best regards,</div><div><br></div><div>Randall</div><br><div><div>On Jan 15, 2009, at 3:45 PM, Sergey Litsenko wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; "><div><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">Let me clarify - it's not precludes efforts. I'm just saying that it is optional - if author tag exists - fine, if not - fine too.<span class="Apple-converted-space">&nbsp;</span><br>I'm not completely against removal of author tags - I'm just seeing it from diferent prospective - in case it's needed to get full list of contributors - we may automate that process of getting list of contributors - without restricting these tags - thats all.<br></div><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">Sergiy<br><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font size="2" face="Tahoma"><hr size="1"><b><span style="font-weight: bold; ">From:</span></b><span class="Apple-converted-space">&nbsp;</span>John Verhaeg &lt;<a href="mailto:jverhaeg@redhat.com">jverhaeg@redhat.com</a>><br><b><span style="font-weight: bold; ">To:</span></b><span class="Apple-converted-space">&nbsp;</span>Sergey Litsenko &lt;<a href="mailto:litsenko_sergey@yahoo.com">litsenko_sergey@yahoo.com</a>><br><b><span style="font-weight: bold; ">Cc:</span></b><span class="Apple-converted-space">&nbsp;</span><a href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a><br><b><span style="font-weight: bold; ">Sent:</span></b><span class="Apple-converted-space">&nbsp;</span>Friday, 16 January, 2009 4:29:27 AM<br><b><span style="font-weight: bold; ">Subject:</span></b><span class="Apple-converted-space">&nbsp;</span>Re: [dna-dev] Re: @author tags in our codebase<br></font><br><div style="font-family: 'Times New Roman'; font-size: 12pt; color: rgb(0, 0, 0); margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">I'm not completely sold on this, but I don't really have much of an argument against it either, beyond worrying that potential contributors might not feel they're getting "enough" attribution without the direct tie-in between the source and their names.&nbsp; Not sure if I understand Sergey's comments about accepting contributor efforts - how does this preclude those efforts?&nbsp; Other than that, I'd give a +1.<br><br>John Verhaeg<br>Red Hat, Inc.<br>(314) 336-2950<br><br>----- "Sergey Litsenko" &lt;<a href="mailto:litsenko_sergey@yahoo.com">litsenko_sergey@yahoo.com</a>> wrote:<span class="Apple-converted-space">&nbsp;</span><br>|<div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">|<div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">|<span class="Apple-converted-space">&nbsp;</span><span style="font-weight: bold; ">-1<br>|<span class="Apple-converted-space">&nbsp;</span></span>It is optional - would be better to allow keeping authors tags while automating process of getting full list.<br>|<span class="Apple-converted-space">&nbsp;</span><br>| On one hand, I'm not sold on the idea that if the author tag will be removed it might help to brought more commiters. If&nbsp; I see some issue in the source code and I know how to fix it - I'll provide a patch/fix whetever I'm in the "magic list" or not.<span class="Apple-converted-space">&nbsp;</span><br>|<span class="Apple-converted-space">&nbsp;</span><br>| The only question that might be asked:&nbsp; are OSS team members interested in accepting such contributor's efforts or not?<span class="Apple-converted-space">&nbsp;</span><br>| It's about mentality and maturity of committers as well as part of OSS project's culture established by team members&nbsp; - e.g. responding to user questions, issues, etc - in e-mails, users forums, etc and welcoming user's opinions on how things should work (functional requirements). Also you can use something like following: "@author DNA Expert Group" for classes/packages designed / developed by group of authors.<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Generally, I believe that everything that is part of Java Language specification (e.g.) and/or part of general practices is valid and justified to be part of any source code.<br>|<span class="Apple-converted-space">&nbsp;</span><br>| On the other hand, since<span class="Apple-converted-space">&nbsp;</span><span style="font-weight: bold; ">it's possible to automate process of getting list of autors from both - SCM repository and source code/POM - maintenance would not be that hard</span>, and IMO efforts should go that way rather than removing tags. I bet that I can find some maven plugin or develop my own to automate that process. For example,<span class="Apple-converted-space">&nbsp;</span><a href="http://www.statsvn.org">http://www.statsvn.org</a><span class="Apple-converted-space">&nbsp;</span>and<span class="Apple-converted-space">&nbsp;</span><a href="http://stat-scm.sourceforge.net">http://stat-scm.sourceforge.net</a><span class="Apple-converted-space">&nbsp;</span>are good starting points.&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>|<span class="Apple-converted-space">&nbsp;</span><br>|<span class="Apple-converted-space">&nbsp;</span><span style="font-weight: bold; ">I will vote "+2" on the ability to automate process of getting full list of contributors (maven report plugin) and makit it part of distribution and Maven project site</span>.<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Sergiy<br>| &nbsp;<br>|<div style="font-family: arial, helvetica, sans-serif; font-size: 13px; margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">|<font size="2" face="Tahoma"><hr size="1"><b><span style="font-weight: bold; ">From:</span></b><span class="Apple-converted-space">&nbsp;</span>"<a href="mailto:dna-dev-request@lists.jboss.org">dna-dev-request@lists.jboss.org</a>" &lt;<a href="mailto:dna-dev-request@lists.jboss.org">dna-dev-request@lists.jboss.org</a>><br>|<span class="Apple-converted-space">&nbsp;</span><b><span style="font-weight: bold; ">To:</span></b><span class="Apple-converted-space">&nbsp;</span><a href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a><br>|<span class="Apple-converted-space">&nbsp;</span><b><span style="font-weight: bold; ">Sent:</span></b><span class="Apple-converted-space">&nbsp;</span>Wednesday, 14 January, 2009 6:43:27 PM<br>|<span class="Apple-converted-space">&nbsp;</span><b><span style="font-weight: bold; ">Subject:</span></b><span class="Apple-converted-space">&nbsp;</span>dna-dev Digest, Vol 10, Issue 5<br>|<span class="Apple-converted-space">&nbsp;</span></font><br>| Send dna-dev mailing list submissions to<br>| &nbsp;&nbsp;&nbsp;<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" ymailto="mailto:dna-dev@lists.jboss.org" target="_blank" href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a><br>|<span class="Apple-converted-space">&nbsp;</span><br>| To subscribe or unsubscribe via the World Wide Web, visit<br>| &nbsp;&nbsp;&nbsp;<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="https://lists.jboss.org/mailman/listinfo/dna-dev">https://lists.jboss.org/mailman/listinfo/dna-dev</a><br>| or, via email, send a message with subject or body 'help' to<br>| &nbsp;&nbsp;&nbsp;<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" ymailto="mailto:dna-dev-request@lists.jboss.org" target="_blank" href="mailto:dna-dev-request@lists.jboss.org">dna-dev-request@lists.jboss.org</a><br>|<span class="Apple-converted-space">&nbsp;</span><br>| You can reach the person managing the list at<br>| &nbsp;&nbsp;&nbsp;<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" ymailto="mailto:dna-dev-owner@lists.jboss.org" target="_blank" href="mailto:dna-dev-owner@lists.jboss.org">dna-dev-owner@lists.jboss.org</a><br>|<span class="Apple-converted-space">&nbsp;</span><br>| When replying, please edit your Subject line so it is more specific<br>| than "Re: Contents of dna-dev digest..."<br>|<span class="Apple-converted-space">&nbsp;</span><br>|<span class="Apple-converted-space">&nbsp;</span><br>| Today's Topics:<br>|<span class="Apple-converted-space">&nbsp;</span><br>| &nbsp; 1. Re: @author tags in our codebase (Randall Hauch)<br>| &nbsp; 2. Re: @author tags in our codebase (Vatsal)<br>|<span class="Apple-converted-space">&nbsp;</span><br>|<span class="Apple-converted-space">&nbsp;</span><br>| ----------------------------------------------------------------------<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Message: 1<br>| Date: Tue, 13 Jan 2009 16:03:23 -0600<br>| From: Randall Hauch &lt;<a rel="nofollow" ymailto="mailto:rhauch@redhat.com" target="_blank" href="mailto:rhauch@redhat.com">rhauch@redhat.com</a>><br>| Subject: Re: [dna-dev] @author tags in our codebase<br>| To: JBoss DNA &lt;<a rel="nofollow" ymailto="mailto:dna-dev@lists.jboss.org" target="_blank" href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a>><br>| Message-ID: &lt;<a rel="nofollow" ymailto="mailto:160467FE-4D64-4943-BBAC-3D9535FD670C@redhat.com" target="_blank" href="mailto:160467FE-4D64-4943-BBAC-3D9535FD670C@redhat.com">160467FE-4D64-4943-BBAC-3D9535FD670C@redhat.com</a>><br>| Content-Type: text/plain; charset="us-ascii"<br>|<span class="Apple-converted-space">&nbsp;</span><br>| We never really came to a consensus on this question, and I'd like to&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| try to do that.&nbsp; To be clear, here is the proposal:<br>|<span class="Apple-converted-space">&nbsp;</span><br>| 1) Remove the @author lines from the code, and instead rely upon SVN&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| as the official master record of individual contributions<br>| 2) Change the Eclipse preference files to remove the @author lines&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| from the code templates<br>| 3) Add a AUTHORS file to the distribution(s); this file will contain&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| the names and email addresses for all contributors, and can even allow&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| a contributor to describe their contribution if they so desire.<br>| 4) Change the headers to remove the "@author" wording and to replace&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| it with "See the AUTHORS file in the<br>| distribution for a full listing of individual contributors."<br>| 5) Change the POM files to include the AUTHORS file in each&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| distribution.<br>|<span class="Apple-converted-space">&nbsp;</span><br>| The AUTHORS file would look like this:<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Randall Hauch (<a rel="nofollow" ymailto="mailto:rhauch@redhat.com" target="_blank" href="mailto:rhauch@redhat.com">rhauch@redhat.com</a>)<br>| John Verhaeg (<a rel="nofollow" ymailto="mailto:jverhaeg@redhat.com" target="_blank" href="mailto:jverhaeg@redhat.com">jverhaeg@redhat.com</a>)<br>| Dan Florian (<a rel="nofollow" ymailto="mailto:dflorian@redhat.com" target="_blank" href="mailto:dflorian@redhat.com">dflorian@redhat.com</a>)<br>| Stefano Maestri (<a rel="nofollow" ymailto="mailto:stefano.maestri@javalinux.it" target="_blank" href="mailto:stefano.maestri@javalinux.it">stefano.maestri@javalinux.it</a>)<br>| Serge Pagop (<a rel="nofollow" ymailto="mailto:Serge.Pagop@innoq.com" target="_blank" href="mailto:Serge.Pagop@innoq.com">Serge.Pagop@innoq.com</a>)<br>| Michael Trezzi (<a rel="nofollow" ymailto="mailto:michael@mathwizard.org" target="_blank" href="mailto:michael@mathwizard.org">michael@mathwizard.org</a>)<br>| Alexandre Porcelli (<a rel="nofollow" ymailto="mailto:porcelli@devexp.com.br" target="_blank" href="mailto:porcelli@devexp.com.br">porcelli@devexp.com.br</a>)<br>| Sergiy Litsenko (<a rel="nofollow" ymailto="mailto:litsenko_sergey@yahoo.com" target="_blank" href="mailto:litsenko_sergey@yahoo.com">litsenko_sergey@yahoo.com</a>)<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Note that unlike the @author tags, this file will list all&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| contributors, and the names of new contributors will be appended to&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| the list by the project lead.&nbsp; (No names will be removed from this&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| file.)<br>|<span class="Apple-converted-space">&nbsp;</span><br>| I would prefer to hear from every contributor, so please respond with&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| +1 if you agree with this proposal, 0 if you don't care, or -1 if you&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| want to keep the @author tags.&nbsp; If you vehemently want to keep the&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| @author tags and names in the source file, please say so.<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Best regards,<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Randall<br>|<span class="Apple-converted-space">&nbsp;</span><br>| On Nov 18, 2008, at 3:33 PM, Randall Hauch wrote:<br>|<span class="Apple-converted-space">&nbsp;</span><br>| ><br>| > On Nov 18, 2008, at 2:52 PM, Stefano Maestri wrote:<br>| ><br>| >><br>| >> Randall Hauch wrote on 17/11/08 22:17:<br>| >>> I've recently read a suggestions for open source communities that&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| >>> the<br>| >>> author names are removed from the content.&nbsp; In the case of DNA's<br>| >>> codebase, that would mean removing the @author tags.<br>| >> May I ask where?<br>| ><br>| > I knew someone was going to ask. :-)&nbsp; I had to go back and look, but&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| > here are a few:<br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://video.google.com/videoplay?docid=-4216011961522818645&amp;ei=8o0YSbiFOY6qrgLC2PnLDQ&amp;q=poisonous+people">http://video.google.com/videoplay?docid=-4216011961522818645&amp;ei=8o0YSbiFOY6qrgLC2PnLDQ&amp;q=poisonous+people</a><br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://docs.ofbiz.org/display/OFBADMIN/Coding+Conventions">http://docs.ofbiz.org/display/OFBADMIN/Coding+Conventions</a><br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://subversion.tigris.org/hacking.html#other-conventions">http://subversion.tigris.org/hacking.html#other-conventions</a><br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://blogs.sun.com/ahe/entry/author_tags">http://blogs.sun.com/ahe/entry/author_tags</a><br>| ><br>| >><br>| >>><br>| >>> tags:<br>| >>><br>| >>>&nbsp; 1. When there are no @author tags, then there is a far smaller<br>| >>>&nbsp; &nbsp; notion of ownership by the author(s).&nbsp; On one side of this, the<br>| >>>&nbsp; &nbsp; author(s) may not appreciate changes to "their" code, and on the<br>| >>>&nbsp; &nbsp; other side, non-authors may feel intimidated about working on<br>| >>>&nbsp; &nbsp; code for which they are not an author.&nbsp; IMO, we want to<br>| >>>&nbsp; &nbsp; _discourage_ ownership and _encourage_ everyone to work in any<br>| >>>&nbsp; &nbsp; area of the code they want.<br>| >>><br>| >> +1...but is really @author tag intimating someone, or giving&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| >> ownership<br>| >> to some other? Quiet frankly not for me.<br>| ><br>| > I hope it doesn't discourage people from contributing and diving in&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| > wherever they want.&nbsp; BTW, it's quite possible that no matter what&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| > our policy, some people may not like it.&nbsp; For example, if we were to&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| > adopt a policy of NOT including @author tags, some people may refuse&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| > to join the community because they see the @author tag as proof they&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| > worked on it.&nbsp; It takes all kinds of people. :-)<br>| ><br>| >><br>| >> Anyway I agree on the _discurage_ownership and _encourage_everyone to<br>| >> work in any area, so if it can help, remove @author tag.<br>| >><br>| >>>&nbsp; 1. @author tags can be inaccurate.&nbsp; SVN has the true history of who<br>| >>>&nbsp; &nbsp; contributed exactly what code.<br>| >>><br>| >> +1<br>| ><br>| > IMO, this is perhaps the biggest justifiable reason.&nbsp; Its rubbish if&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| > its not up-to-date, so it seems far better to not have @author tags.<br>| ><br>| >><br>| >>><br>| >>> The only benefit I can think of is that the @author tag does help to<br>| >>> give some notion of who is the "expert" of the class, in case they<br>| >>> need to be consulted.&nbsp; However, I don't believe this is really&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| >>> much of<br>| >>> a reason, since it's far better to consult the SVN history and see&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| >>> who<br>| >>> actually modified the different parts of the code.&nbsp; In fact, the<br>| >>> annotated views in Fisheye even show on many of the lines the name&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| >>> of<br>| >>> the last person to change it.&nbsp; For example,<br>| >>> see<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-common/src/main/java/org/jboss/dna/common/i18n/I18n.java?r=120">http://fisheye.jboss.org/browse/DNA/trunk/dna-common/src/main/java/org/jboss/dna/common/i18n/I18n.java?r=120</a><br>| >>><br>| >> abosolutely better to use fisheye...if fine people of JBoss.org would<br>| >> also mind to upgrade it to a more recent version it would be even<br>| >> better. Also Jira integration may help a lot.<br>| >><br>| >> I would just add that if we decide to remove the tag we have to&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| >> change<br>| >> also the license information at the beginnig of any file which say:<br>| >> /* 2<br>| >> &lt;<a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l2">http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l2</a><span class="Apple-converted-space">&nbsp;</span><br>| >> ><br>| >> * JBoss, Home of Professional Open Source. 3<br>| >> &lt;<a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l3">http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l3</a><span class="Apple-converted-space">&nbsp;</span><br>| >> ><br>| >> * Copyright 2008, Red Hat Middleware LLC, and individual&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| >> contributors 4<br>| >> &lt;<a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l4">http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l4</a><span class="Apple-converted-space">&nbsp;</span><br>| >> ><br>| >> * as indicated by the @author tags. See the copyright.txt file in&nbsp;<span class="Apple-converted-space">&nbsp;</span><br>| >> the 5<br>| >> &lt;<a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l5">http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l5</a><span class="Apple-converted-space">&nbsp;</span><br>| >> ><br>| >> * distribution for a full listing of individual contributors.<br>| >><br>| ><br>| > Yes, we'd have to update the headers.<br>| ><br>| > Best regards,<br>| ><br>| > Randall<br>| ><br>|<span class="Apple-converted-space">&nbsp;</span><br>| -------------- next part --------------<br>| An HTML attachment was scrubbed...<br>| URL:<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://lists.jboss.org/pipermail/dna-dev/attachments/20090113/d2486bd9/attachment-0001.html">http://lists.jboss.org/pipermail/dna-dev/attachments/20090113/d2486bd9/attachment-0001.html</a><br>|<span class="Apple-converted-space">&nbsp;</span><br>| ------------------------------<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Message: 2<br>| Date: Wed, 14 Jan 2009 13:13:23 +0530<br>| From: Vatsal &lt;<a rel="nofollow" ymailto="mailto:vatsal.avasthi@gmail.com" target="_blank" href="mailto:vatsal.avasthi@gmail.com">vatsal.avasthi@gmail.com</a>><br>| Subject: Re: [dna-dev] @author tags in our codebase<br>| To:<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" ymailto="mailto:dna-dev@lists.jboss.org" target="_blank" href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a><br>| Message-ID:<br>| &nbsp;&nbsp;&nbsp; &lt;<a rel="nofollow" ymailto="mailto:c82836c60901132343w54b690a9ha9c183e5ee61d019@mail.gmail.com" target="_blank" href="mailto:c82836c60901132343w54b690a9ha9c183e5ee61d019@mail.gmail.com">c82836c60901132343w54b690a9ha9c183e5ee61d019@mail.gmail.com</a>><br>| Content-Type: text/plain; charset="iso-8859-1"<br>|<span class="Apple-converted-space">&nbsp;</span><br>| Sometimes it is fun to see your name in author tag of a project at a later<br>| date when the project has matured but my vote would be for Randall's<br>| suggestions due to practical &amp; maintenance reasons discussed earlier, so a<br>| +1 from me for this proposal(though I am not a contributor yet :) )...<br>| - Vatsal<br>|<span class="Apple-converted-space">&nbsp;</span><br>| On Wed, Jan 14, 2009 at 3:33 AM, Randall Hauch &lt;<a rel="nofollow" ymailto="mailto:rhauch@redhat.com" target="_blank" href="mailto:rhauch@redhat.com">rhauch@redhat.com</a>> wrote:<br>|<span class="Apple-converted-space">&nbsp;</span><br>| > We never really came to a consensus on this question, and I'd like to try<br>| > to do that.&nbsp; To be clear, here is the proposal:<br>| ><br>| > 1) Remove the @author lines from the code, and instead rely upon SVN as the<br>| > official master record of individual contributions<br>| > 2) Change the Eclipse preference files to remove the @author lines from the<br>| > code templates<br>| > 3) Add a AUTHORS file to the distribution(s); this file will contain the<br>| > names and email addresses for all contributors, and can even allow a<br>| > contributor to describe their contribution if they so desire.<br>| > 4) Change the headers to remove the "@author" wording and to replace it<br>| > with "See the AUTHORS file in the<br>| > distribution for a full listing of individual contributors."<br>| > 5) Change the POM files to include the AUTHORS file in each distribution.<br>| ><br>| > The AUTHORS file would look like this:<br>| ><br>| > Randall Hauch (<a rel="nofollow" ymailto="mailto:rhauch@redhat.com" target="_blank" href="mailto:rhauch@redhat.com">rhauch@redhat.com</a>)<br>| ><br>| > John Verhaeg (<a rel="nofollow" ymailto="mailto:jverhaeg@redhat.com" target="_blank" href="mailto:jverhaeg@redhat.com">jverhaeg@redhat.com</a>)<br>| > Dan Florian (<a rel="nofollow" ymailto="mailto:dflorian@redhat.com" target="_blank" href="mailto:dflorian@redhat.com">dflorian@redhat.com</a>)<br>| > Stefano Maestri (<a rel="nofollow" ymailto="mailto:stefano.maestri@javalinux.it" target="_blank" href="mailto:stefano.maestri@javalinux.it">stefano.maestri@javalinux.it</a>)<br>| > Serge Pagop (<a rel="nofollow" ymailto="mailto:Serge.Pagop@innoq.com" target="_blank" href="mailto:Serge.Pagop@innoq.com">Serge.Pagop@innoq.com</a>)<br>| > Michael Trezzi (<a rel="nofollow" ymailto="mailto:michael@mathwizard.org" target="_blank" href="mailto:michael@mathwizard.org">michael@mathwizard.org</a>)<br>| > Alexandre Porcelli (<a rel="nofollow" ymailto="mailto:porcelli@devexp.com.br" target="_blank" href="mailto:porcelli@devexp.com.br">porcelli@devexp.com.br</a>)<br>| > Sergiy Litsenko (<a rel="nofollow" ymailto="mailto:litsenko_sergey@yahoo.com" target="_blank" href="mailto:litsenko_sergey@yahoo.com">litsenko_sergey@yahoo.com</a>)<br>| ><br>| ><br>| > Note that unlike the @author tags, this file will list all contributors,<br>| > and the names of new contributors will be appended to the list by the<br>| > project lead.&nbsp; (No names will be removed from this file.)<br>| ><br>| > I would prefer to hear from every contributor, so please respond with +1 if<br>| > you agree with this proposal, 0 if you don't care, or -1 if you want to keep<br>| > the @author tags.&nbsp; If you vehemently want to keep the @author tags and names<br>| > in the source file, please say so.<br>| ><br>| > Best regards,<br>| ><br>| > Randall<br>| ><br>| > On Nov 18, 2008, at 3:33 PM, Randall Hauch wrote:<br>| ><br>| ><br>| > On Nov 18, 2008, at 2:52 PM, Stefano Maestri wrote:<br>| ><br>| ><br>| > Randall Hauch wrote on 17/11/08 22:17:<br>| ><br>| > I've recently read a suggestions for open source communities that the<br>| ><br>| > author names are removed from the content.&nbsp; In the case of DNA's<br>| ><br>| > codebase, that would mean removing the @author tags.<br>| ><br>| > May I ask where?<br>| ><br>| ><br>| > I knew someone was going to ask. :-)&nbsp; I had to go back and look, but here<br>| > are a few:<br>| ><br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://video.google.com/videoplay?docid=-4216011961522818645&amp;ei=8o0YSbiFOY6qrgLC2PnLDQ&amp;q=poisonous+people">http://video.google.com/videoplay?docid=-4216011961522818645&amp;ei=8o0YSbiFOY6qrgLC2PnLDQ&amp;q=poisonous+people</a><br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://docs.ofbiz.org/display/OFBADMIN/Coding+Conventions">http://docs.ofbiz.org/display/OFBADMIN/Coding+Conventions</a><br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://subversion.tigris.org/hacking.html#other-conventions">http://subversion.tigris.org/hacking.html#other-conventions</a><br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://blogs.sun.com/ahe/entry/author_tags">http://blogs.sun.com/ahe/entry/author_tags</a><br>| ><br>| ><br>| ><br>| > tags:<br>| ><br>| ><br>| >&nbsp; 1. When there are no @author tags, then there is a far smaller<br>| ><br>| >&nbsp; &nbsp; notion of ownership by the author(s).&nbsp; On one side of this, the<br>| ><br>| >&nbsp; &nbsp; author(s) may not appreciate changes to "their" code, and on the<br>| ><br>| >&nbsp; &nbsp; other side, non-authors may feel intimidated about working on<br>| ><br>| >&nbsp; &nbsp; code for which they are not an author.&nbsp; IMO, we want to<br>| ><br>| >&nbsp; &nbsp; _discourage_ ownership and _encourage_ everyone to work in any<br>| ><br>| >&nbsp; &nbsp; area of the code they want.<br>| ><br>| ><br>| > +1...but is really @author tag intimating someone, or giving ownership<br>| ><br>| > to some other? Quiet frankly not for me.<br>| ><br>| ><br>| > I hope it doesn't discourage people from contributing and diving in<br>| > wherever they want.&nbsp; BTW, it's quite possible that no matter what our<br>| > policy, some people may not like it.&nbsp; For example, if we were to adopt a<br>| > policy of NOT including @author tags, some people may refuse to join the<br>| > community because they see the @author tag as proof they worked on it.&nbsp; It<br>| > takes all kinds of people. :-)<br>| ><br>| ><br>| > Anyway I agree on the _discurage_ownership and _encourage_everyone to<br>| ><br>| > work in any area, so if it can help, remove @author tag.<br>| ><br>| ><br>| >&nbsp; 1. @author tags can be inaccurate.&nbsp; SVN has the true history of who<br>| ><br>| >&nbsp; &nbsp; contributed exactly what code.<br>| ><br>| ><br>| > +1<br>| ><br>| ><br>| > IMO, this is perhaps the biggest justifiable reason.&nbsp; Its rubbish if its<br>| > not up-to-date, so it seems far better to not have @author tags.<br>| ><br>| ><br>| ><br>| > The only benefit I can think of is that the @author tag does help to<br>| ><br>| > give some notion of who is the "expert" of the class, in case they<br>| ><br>| > need to be consulted.&nbsp; However, I don't believe this is really much of<br>| ><br>| > a reason, since it's far better to consult the SVN history and see who<br>| ><br>| > actually modified the different parts of the code.&nbsp; In fact, the<br>| ><br>| > annotated views in Fisheye even show on many of the lines the name of<br>| ><br>| > the last person to change it.&nbsp; For example,<br>| ><br>| > see<br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-common/src/main/java/org/jboss/dna/common/i18n/I18n.java?r=120">http://fisheye.jboss.org/browse/DNA/trunk/dna-common/src/main/java/org/jboss/dna/common/i18n/I18n.java?r=120</a><br>| ><br>| ><br>| > abosolutely better to use fisheye...if fine people of JBoss.org would<br>| ><br>| > also mind to upgrade it to a more recent version it would be even<br>| ><br>| > better. Also Jira integration may help a lot.<br>| ><br>| ><br>| > I would just add that if we decide to remove the tag we have to change<br>| ><br>| > also the license information at the beginnig of any file which say:<br>| ><br>| > /* 2<br>| ><br>| > &lt;<br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l2">http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l2</a><br>| > ><br>| ><br>| > * JBoss, Home of Professional Open Source. 3<br>| ><br>| > &lt;<br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l3">http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l3</a><br>| > ><br>| ><br>| > * Copyright 2008, Red Hat Middleware LLC, and individual contributors 4<br>| ><br>| > &lt;<br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l4">http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l4</a><br>| > ><br>| ><br>| > * as indicated by the @author tags. See the copyright.txt file in the 5<br>| ><br>| > &lt;<br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l5">http://fisheye.jboss.org/browse/DNA/trunk/dna-graph/src/main/java/org/jboss/dna/graph/GraphI18n.java?r=598#l5</a><br>| > ><br>| ><br>| > * distribution for a full listing of individual contributors.<br>| ><br>| ><br>| ><br>| > Yes, we'd have to update the headers.<br>| ><br>| > Best regards,<br>| ><br>| > Randall<br>| ><br>| ><br>| ><br>| > _______________________________________________<br>| > dna-dev mailing list<br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" ymailto="mailto:dna-dev@lists.jboss.org" target="_blank" href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a><br>| ><span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="https://lists.jboss.org/mailman/listinfo/dna-dev">https://lists.jboss.org/mailman/listinfo/dna-dev</a><br>| ><br>| ><br>| -------------- next part --------------<br>| An HTML attachment was scrubbed...<br>| URL:<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://lists.jboss.org/pipermail/dna-dev/attachments/20090114/4b648c37/attachment.html">http://lists.jboss.org/pipermail/dna-dev/attachments/20090114/4b648c37/attachment.html</a><br>|<span class="Apple-converted-space">&nbsp;</span><br>| ------------------------------<br>|<span class="Apple-converted-space">&nbsp;</span><br>| _______________________________________________<br>| dna-dev mailing list<br>|<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" ymailto="mailto:dna-dev@lists.jboss.org" target="_blank" href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a><br>|<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="https://lists.jboss.org/mailman/listinfo/dna-dev">https://lists.jboss.org/mailman/listinfo/dna-dev</a><br>|<span class="Apple-converted-space">&nbsp;</span><br>|<span class="Apple-converted-space">&nbsp;</span><br>| End of dna-dev Digest, Vol 10, Issue 5<br>| **************************************<br>|</div></div></div><br>|<hr size="1">Stay connected to the people that matter most with a smarter inbox.<span class="Apple-converted-space">&nbsp;</span><a rel="nofollow" target="_blank" href="http://au.rd.yahoo.com/galaxy/mail/tagline2/*http://au.docs.yahoo.com/mail/smarterinbox">Take a look</a>.<br>| _______________________________________________ dna-dev mailing list<span class="Apple-converted-space">&nbsp;</span><a href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a><span class="Apple-converted-space">&nbsp;</span><a href="https://lists.jboss.org/mailman/listinfo/dna-dev">https://lists.jboss.org/mailman/listinfo/dna-dev</a></div></div></div></div><br><hr size="1">Stay connected to the people that matter most with a smarter inbox.<span class="Apple-converted-space">&nbsp;</span><a href="http://au.rd.yahoo.com/galaxy/mail/tagline2/*http://au.docs.yahoo.com/mail/smarterinbox" target="_blank">Take a look</a>._______________________________________________<br>dna-dev mailing list<br><a href="mailto:dna-dev@lists.jboss.org">dna-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/dna-dev">https://lists.jboss.org/mailman/listinfo/dna-dev</a><br></div></span></blockquote></div><br></body></html>