Re: remove via

2012-06-07 Thread David Smiley (@MITRE.org)
-server/book -- View this message in context: http://lucene.472066.n3.nabble.com/remove-via-tp3987871p3988237.html Sent from the Lucene - Java Developer mailing list archive at Nabble.com. - To unsubscribe, e-mail: dev-unsubscr

Re: remove via

2012-06-07 Thread Robert Muir
I dont think there is any consensus here. Lets just keep it as-is. On Tue, Jun 5, 2012 at 6:40 PM, Robert Muir rcm...@gmail.com wrote: Hello, Currently we do this in CHANGES.txt: LUCENE-:  Fixed a horrible nasty bug. (Joe Contributor via John Doe Committer) I propose we remove via

Re: remove via

2012-06-06 Thread Dawid Weiss
+1. On Wed, Jun 6, 2012 at 1:27 AM, Steven A Rowe sar...@syr.edu wrote: +1 Steve -Original Message- From: Robert Muir [mailto:rcm...@gmail.com] Sent: Tuesday, June 05, 2012 6:40 PM To: dev@lucene.apache.org Subject: remove via Hello, Currently we do this in CHANGES.txt

Re: remove via

2012-06-06 Thread Tommaso Teofili
2012/6/6 Robert Muir rcm...@gmail.com Hello, Currently we do this in CHANGES.txt: LUCENE-: Fixed a horrible nasty bug. (Joe Contributor via John Doe Committer) I propose we remove via committers name from CHANGES.txt. I don't think any users care about who committed changes and I

Re: remove via

2012-06-06 Thread Erick Erickson
: LUCENE-:  Fixed a horrible nasty bug. (Joe Contributor via John Doe Committer) I propose we remove via committers name from CHANGES.txt. I don't think any users care about who committed changes and I think it de-emphasizes the actual contributor. If we need to know who committed something

Re: remove via

2012-06-06 Thread Mark Miller
On Jun 5, 2012, at 6:40 PM, Robert Muir wrote: Opinions? I disagree - I think it makes it really easy to track who actually did the commit (the person *responsible* if it's a bad commit or a good commit) and I think there is some credit in a committer applying someones patch. They are doing

Re: remove via

2012-06-06 Thread Dawid Weiss
code change. I think *via* is pretty clear regarding credit, and I think it has value in it's information. Even if you simply commit someone else work, *you* are contributing to the issue. You better have reviewed it, you better be willing to take responsibility for it. I don't think these

Re: remove via

2012-06-06 Thread Eric Pugh
I've looked at the via in the changelog to figure out which committer works in which areas the most, and therefore who to ping about a patch. And I do think that shepherding a patch file through to commit is worthy of some credit. It's often a fair amount of work to evaluate a patch file,

Re: remove via

2012-06-06 Thread Mark Miller
On Jun 6, 2012, at 8:26 AM, Dawid Weiss wrote: If you have contributed to the issue and your gut feeling is you're part of the patch's effort, you should include yourself as the author. Personally, I prefer the current std - where a strong review and the taking of responsibility earns you

RE: remove via

2012-06-06 Thread Uwe Schindler
To: dev@lucene.apache.org Subject: Re: remove via I've looked at the via in the changelog to figure out which committer works in which areas the most, and therefore who to ping about a patch. And I do think that shepherding a patch file through to commit is worthy of some credit. It's often a fair

Re: remove via

2012-06-06 Thread Martijn v Groningen
@lucene.apache.org Subject: Re: remove via I've looked at the via in the changelog to figure out which committer works in which areas the most, and therefore who to ping about a patch.  And I do think that shepherding a patch file through to commit is worthy of some credit. It's often a fair amount

Re: remove via

2012-06-06 Thread Mark Miller
Yes... I think it should be: LUCENE-: Fixed a horrible nasty bug. (Joe Contributor) I think it's very clear who the author was with: LUCENE-: Fixed a horrible nasty bug. (Joe Contributor via Mr Committer) I agree that committers should be generous and low key about their own

Re: remove via

2012-06-06 Thread Andrzej Bialecki
On 06/06/2012 00:40, Robert Muir wrote: Hello, Currently we do this in CHANGES.txt: LUCENE-: Fixed a horrible nasty bug. (Joe Contributor via John Doe Committer) I propose we remove viacommitters name from CHANGES.txt. I don't think any users care about who committed changes and I

Re: remove via

2012-06-06 Thread Chris Hostetter
: I've looked at the via in the changelog to figure out which committer : works in which areas the most, and therefore who to ping about a patch. That's a user for the info that i hadn't really considered, and definitely gives me pause... I guess i'm changing my opinion: -0. -Hoss

Re: remove via

2012-06-06 Thread Erick Erickson
Following where Hoss bravely leads, I'll change too to -0 Erick On Wed, Jun 6, 2012 at 1:56 PM, Chris Hostetter hossman_luc...@fucit.org wrote: : I've looked at the via in the changelog to figure out which committer : works in which areas the most, and therefore who to ping about a patch.

Re: remove via

2012-06-06 Thread Dawid Weiss
My initial +1 was because I guess I don't care if it's either way. As far as important stuff is done I wouldn't worry about who gets the credit... Dawid On Wed, Jun 6, 2012 at 8:03 PM, Erick Erickson erickerick...@gmail.com wrote: Following where Hoss bravely leads, I'll change too to -0

Re: remove via

2012-06-06 Thread Mark Miller
On Jun 6, 2012, at 2:45 PM, Dawid Weiss wrote: As far as important stuff is done I wouldn't worry about who gets the credit... I think that's kind of short sighted. These credits matter to some, and many of these things, while they may not matter to us personally, that does not make them

Re: remove via

2012-06-06 Thread Dawid Weiss
I think I understand your viewpoint but I'll stick with mine for now (that is: I don't care for 'via' attribution in CHANGES.txt, but I also don't object it -- I'll stick with whatever consensus there will be). Dawid On Wed, Jun 6, 2012 at 9:54 PM, Mark Miller markrmil...@gmail.com wrote: On

remove via

2012-06-05 Thread Robert Muir
Hello, Currently we do this in CHANGES.txt: LUCENE-: Fixed a horrible nasty bug. (Joe Contributor via John Doe Committer) I propose we remove via committers name from CHANGES.txt. I don't think any users care about who committed changes and I think it de-emphasizes the actual contributor

Re: remove via

2012-06-05 Thread Michael McCandless
+1 Mike McCandless http://blog.mikemccandless.com On Tue, Jun 5, 2012 at 6:40 PM, Robert Muir rcm...@gmail.com wrote: Hello, Currently we do this in CHANGES.txt: LUCENE-:  Fixed a horrible nasty bug. (Joe Contributor via John Doe Committer) I propose we remove via committers name

Re: remove via

2012-06-05 Thread Chris Hostetter
: LUCENE-: : Fixed a horrible nasty bug. (Joe Contributor via John Doe Committer) : : I propose we remove via committers name from CHANGES.txt. I don't FWIW: as first glance i thought you were suggesting that we should only use (Joe Contributor, John Doe Committer) ... which i would

Re: remove via

2012-06-05 Thread Robert Muir
On Wed, Jun 6, 2012 at 12:57 AM, Chris Hostetter hossman_luc...@fucit.org wrote: It seems (on second glance) that you are actually arguing that that the person who runs svn commit should get zero credit in CHANGES.txt -- unless they actually contributed to the development of the patch.

RE: remove via

2012-06-05 Thread Steven A Rowe
+1 Steve -Original Message- From: Robert Muir [mailto:rcm...@gmail.com] Sent: Tuesday, June 05, 2012 6:40 PM To: dev@lucene.apache.org Subject: remove via Hello, Currently we do this in CHANGES.txt: LUCENE-: Fixed a horrible nasty bug. (Joe Contributor via John Doe Committer