Hi,
although I still like to enjoy my final week of the holidays, I had to
verify this release since I kind of blocked the previous attempt.
I can confirm that my mentioned issues have been fixed apart from the
performance, but good reports are more important to me.
But it looks like there's a small minor issue: it looks like Cobertura
warns that interfaces and classes without code coverage are not
instrumented, see:
[cobertura] INFO [main]
net.sourceforge.cobertura.reporting.html.HTMLReport - D
ata file does not contain instrumentation information for the file
com/thoughtwo
rks/qdox/tools/QDoxTester.java. Ensure this class was instrumented, and
this da
ta file contains the instrumentation information.
[cobertura] INFO [main]
net.sourceforge.cobertura.reporting.html.HTMLReport - D
ata file does not contain instrumentation information for the file
com/thoughtwo
rks/qdox/writer/ModelWriter.java. Ensure this class was instrumented, and
this
data file contains the instrumentation information.
QDoxTester is a class without codecoverage (it is a wrapper with a static
void main(String[]), so QDox can be tested from cmdline)
ModelWriter is an interface.
For both cases the info message is IMHO wrong, since:
- QDoxTester was instrumented, but there's just no data.
- How can you instrument interfaces?
So depending on the number of interfaces and uncovered classes it can
generate a lot of messages.
It would be nice if Cobertura could keep track of instrumented files, so
these messages can disappear.
I'll give a +0.5 because the reports are now correct, but I think users
will probably hit this issue too and might have more trouble with
analyzing the real cause.
Thanks,
Robert
ps. With my +0.5 you can still continue the release as a RM. It's up to
you to decide if it is more important to fix these first or to push this
new release to Maven Central.
pps. Next time add "(take 2)" to the subject so everybody knows it's a
fresh vote.
On Mon, 19 Aug 2013 03:59:19 +0200, Steven Christou
<schristo...@gmail.com> wrote:
Baptiste, I think that extending the vote for this would be best. Most
people will be back from vacation probably on Monday so hopefully
someone will take a look at it. I also made sure to compile it with jdk
5 and released it, so that is not an issue anymore.
The vote will go again for another 72 hours starting now.
Thanks,
Steve.
On 08/18/2013 02:47 AM, Baptiste Mathus wrote:
That's it.
See http://www.apache.org/foundation/voting.html#LazyConsensus for
precisions.
I guess the few responses are because of the summertime and holiday.
Basically, once the three days are off, the RM (release manager, you
in this case) can then proceed to what he announced. And btw the RM
can also decide to delay a few more the action if he wants to get more
feedback.
I'm currently not easily accessing a computer. Did you triple check
this time you didn't compiled for jdk7 bytecode? Which target version
did you settle on for the plugin? I think that'll be an important
information to remind in the announcement mail.
Cheers
Le 18 août 2013 09:51, "Lennart Jörelid" <lennart.jore...@gmail.com
<mailto:lennart.jore...@gmail.com>> a écrit :
I'd say yes - that would be the essence of "lazy" consensus.
While I haven't had a chance to test out the new release of the
Cobertura Maven Plugin yet, I'd simply add my
+1
for good measure here.
:)
2013/8/18 Steven Christou <schristo...@gmail.com
<mailto:schristo...@gmail.com>>
It has been over 72 hours and the cobertura maven plugin
didn't receive any vote. What would be the next step? Should I
perform a release with a lazy consensus?
Thanks,
Steve.
On 08/14/2013 09:03 PM, Steven Christou wrote:
Hi,
I'd like to release version 2.6 of the Cobertura-Maven-Plugin
This plugin provides the features of Cobertura within the
Maven 2 & 3 environment.
We solved 2 issues:
Release Notes - Maven 2.x Cobertura Plugin - Version 2.6
** Improvement
* [MCOBERTURA-175] - Upgrade to Cobertura 2.0
* [MCOBERTURA-176] - Add support for new feature
IgnoreTrivial and IgnoreMethodAnnotation in cobertura 2.0
(Thank you David Ignjic)
There were a number of issues that were in the previous
version of cobertura which have been fixed for this release.
To see a complete list, go to:
https://github.com/cobertura/cobertura/issues?milestone=1&state=closed
https://github.com/cobertura/cobertura/issues/18
https://github.com/cobertura/cobertura/issues/9
The plugin will be using version 2.0.3 of cobertura.
Robert had mentioned a performance issue that was in the
current version of cobertura. I am aware of the issue and on
the next release of cobertura I will try to address the issue.
Staging Repositories:
General:
<https://nexus.codehaus.org/content/groups/staging/>https://nexus.codehaus.org/content/groups/staging/org/codehaus/mojo/cobertura-maven-plugin/2.6/
Exclusive:
https://nexus.codehaus.org/content/repositories/orgcodehausmojo-106/org/codehaus/mojo/cobertura-maven-plugin/2.6/
(Staging) Site:
https://dav.codehaus.org/mojo/cobertura-maven-plugin/
(Current) Site:
http://mojo.codehaus.org/cobertura-maven-plugin/
SCM Tag:
http://svn.codehaus.org/mojo/tags/cobertura-maven-plugin-2.6/
ADDITIONAL_INFORMATION
[ ] +1
[ ] +0
[ ] -1
The vote is open for 72 hours and will succeed by lazy
consensus.
Regards Cheers
Steven Christou
--
--
+==============================+
| Bästa hälsningar,
| [sw. "Best regards"]
|
| Lennart Jörelid
| EAI Architect & Integrator
|
| jGuru Europe AB
| Mölnlycke - Kista
|
| Email:l...@jguru.se <mailto:l...@jguru.se>
| URL:www.jguru.se <http://www.jguru.se>
| Phone
| (skype): jgurueurope
| (intl):+46 708 507 603 <tel:%2B46%20708%20507%20603>
| (domestic): 0708 - 507 603
+==============================+
--
Using Opera's revolutionary email client: http://www.opera.com/mail/
---------------------------------------------------------------------
To unsubscribe from this list, please visit:
http://xircles.codehaus.org/manage_email
