[
https://issues.apache.org/jira/browse/ZOOKEEPER-2955?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16338255#comment-16338255
]
ASF GitHub Bot commented on ZOOKEEPER-2955:
-------------------------------------------
Github user mfenes commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/443#discussion_r163684223
--- Diff: build.xml ---
@@ -23,6 +23,48 @@ xmlns:artifact="antlib:org.apache.maven.artifact.ant"
xmlns:maven="antlib:org.apache.maven.artifact.ant"
xmlns:cs="antlib:com.puppycrawl.tools.checkstyle.ant">
+ <!-- ====================================================== -->
--- End diff --
@afine
Because I need
`<property name="clover.version" value="4.2.1" />`
in
`<property name="clover.jar"
location="${clover.home}/lib/clover-${clover.version}.jar"/>`
and the dependency versions at their original location were declared later
than clover.jar, so clover.version was undefined at this point.
I had 3 choices:
1. move the clover property set behind the dependency version declarations
just before the macro definitions, separating it from the other property set
declarations,
1. move all the dependency version declarations before the clover property
set,
1. or just declare the clover.version separately from the other dependency
versions in the clover property set block.
I've chosen option 2 as I thought that was the best out of the three.
Please let me know if there is a better way to solve this.
> Enable Clover code coverage report
> ----------------------------------
>
> Key: ZOOKEEPER-2955
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2955
> Project: ZooKeeper
> Issue Type: Test
> Components: tests
> Affects Versions: 3.5.3, 3.4.11
> Reporter: Mark Fenes
> Assignee: Mark Fenes
> Priority: Major
>
> We have limited code coverage support in ZK. Clover for Java was running in
> the past but was turned off.
> Enable Clover code coverage report to make us more confident on the quality,
> stability and compatibility of future ZK releases.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
