[jira] Commented: (LUCENE-721) Code coverage reports

2006-12-09 Thread Grant Ingersoll (JIRA)
[ http://issues.apache.org/jira/browse/LUCENE-721?page=comments#action_12457080 ] Grant Ingersoll commented on LUCENE-721: I'm not familiar w/ clover, but here's what I did, when trying out your patch (which works): 1. applied the

[jira] Commented: (LUCENE-721) Code coverage reports

2006-12-09 Thread Grant Ingersoll (JIRA)
[ http://issues.apache.org/jira/browse/LUCENE-721?page=comments#action_12457084 ] Grant Ingersoll commented on LUCENE-721: OK, I committed this, with a minor change so that it is off by default (thanks Erik, for the ANT info). To run,

[jira] Commented: (LUCENE-721) Code coverage reports

2006-11-24 Thread Grant Ingersoll (JIRA)
[ http://issues.apache.org/jira/browse/LUCENE-721?page=comments#action_12452544 ] Grant Ingersoll commented on LUCENE-721: Care to update this for Clover, Michael? Then I can hook it into the site and the nightly builds. Thanks, Grant

[jira] Commented: (LUCENE-721) Code coverage reports

2006-11-24 Thread Michael Busch (JIRA)
[ http://issues.apache.org/jira/browse/LUCENE-721?page=comments#action_12452553 ] Michael Busch commented on LUCENE-721: -- Sure Grant, I will update the patch soon to use clover instead of EMMA. Great that you will add it to the website!

Re: [jira] Commented: (LUCENE-721) Code coverage reports

2006-11-22 Thread Michael Busch
Chris Hostetter wrote: To throw another twist onto things, it would appear that the ASF has a License for Clover 1.3.2 donated by Cenqua that Committers have access to (see committers/donated-licenses/clover in SVN) ... it's not clear to me if that License would allow for auto generated reports

Re: [jira] Commented: (LUCENE-721) Code coverage reports

2006-11-22 Thread Simon Willnauer
I agree with Michael, if there is a clover licence we should use it. One major advantage of clover is the ide support for eclipse and netbeans. Emma has no Eclipse support yet and a old not maintained netbeans plugin. There is a ASF donated clover license under:

Re: [jira] Commented: (LUCENE-721) Code coverage reports

2006-11-21 Thread Chris Hostetter
: How about the nightly build, if we include this on the site? I'm not : familiar with it enough to comment. Yonik and Doug will know if/how : things can be set up there with EMMA jars, should we include this on the : site. To throw another twist onto things, it would appear that the ASF has a

[jira] Commented: (LUCENE-721) Code coverage reports

2006-11-21 Thread Otis Gospodnetic (JIRA)
[ http://issues.apache.org/jira/browse/LUCENE-721?page=comments#action_12451721 ] Otis Gospodnetic commented on LUCENE-721: - Nice. I think we can't include EMMA jars int he repository, though, so you'll want to add the ability to

[jira] Commented: (LUCENE-721) Code coverage reports

2006-11-21 Thread Otis Gospodnetic (JIRA)
[ http://issues.apache.org/jira/browse/LUCENE-721?page=comments#action_12451806 ] Otis Gospodnetic commented on LUCENE-721: - Sounds like my previous comment was off - if we don't distribute EMMA jars and just require the person to have

[jira] Commented: (LUCENE-721) Code coverage reports

2006-11-21 Thread Michael Busch (JIRA)
[ http://issues.apache.org/jira/browse/LUCENE-721?page=comments#action_12451727 ] Michael Busch commented on LUCENE-721: -- Good idea, I could do that. But currently we don't fetch the JUnit jars either, just give a hint where to download

Re: [jira] Commented: (LUCENE-721) Code coverage reports

2006-11-21 Thread Chris Hostetter
: Nice. I think we can't include EMMA jars int he repository, though, so : you'll want to add the ability to download the Jar on the fly, just like : Grant did it for the benchmark stuff. that's not strictly neccessary is it? ... coverage reports could just be an optional part of the build that

Re: [jira] Commented: (LUCENE-721) Code coverage reports

2006-11-21 Thread Michael Busch
Chris Hostetter wrote: : Nice. I think we can't include EMMA jars int he repository, though, so : you'll want to add the ability to download the Jar on the fly, just like : Grant did it for the benchmark stuff. that's not strictly neccessary is it? ... coverage reports could just be an