Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated README to note Cobertura and JDK11 restriction #11

Merged
merged 4 commits into from
Aug 7, 2020

Conversation

steveswinsburg
Copy link
Contributor

In order to use Cobertura with Travis, you must specify an earlier version of the JDK. Cobertura will not work with travis out of the box as Travis uses JDK 11 (currently). The build will fail with:

[ERROR] Failed to execute goal org.codehaus.mojo:cobertura-maven-plugin:2.7:instrument (default-cli) on project xbdd-maven-plugin: Execution default-cli of goal org.codehaus.mojo:cobertura-maven-plugin:2.7:instrument failed: Plugin org.codehaus.mojo:cobertura-maven-plugin:2.7 or one of its dependencies could not be resolved: Could not find artifact com.sun:tools:jar:0 at specified path /usr/local/lib/jvm/openjdk11/../lib/tools.jar -> [Help 1]

In order to use Cobertura with Travis, you must specify an earlier version of the JDK. Cobertura will not work with travis out of the box as Travis uses JDK 11 (currently). The build will fail with:
```
[ERROR] Failed to execute goal org.codehaus.mojo:cobertura-maven-plugin:2.7:instrument (default-cli) on project xbdd-maven-plugin: Execution default-cli of goal org.codehaus.mojo:cobertura-maven-plugin:2.7:instrument failed: Plugin org.codehaus.mojo:cobertura-maven-plugin:2.7 or one of its dependencies could not be resolved: Could not find artifact com.sun:tools:jar:0 at specified path /usr/local/lib/jvm/openjdk11/../lib/tools.jar -> [Help 1]
```
@me-heer
Copy link

me-heer commented Dec 10, 2019

48 hours spent on getting CodeCov to work with Cobertura. I wish I had seen your Pull Request earlier. I tried JaCoCo as well. It seems that even JaCoCo doesn't support the latest JDK. Thanks a lot! I can sleep now!

@marcauberer
Copy link

Should be merged definetly.

@codecov
Copy link

codecov bot commented Aug 7, 2020

Codecov Report

Merging #11 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #11   +/-   ##
=======================================
  Coverage   50.00%   50.00%           
=======================================
  Files           1        1           
  Lines           4        4           
  Branches        1        1           
=======================================
  Hits            2        2           
  Misses          1        1           
  Partials        1        1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a4d9cef...223250f. Read the comment docs.

Copy link
Contributor

@thomasrockhu thomasrockhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @steveswinsburg !

@thomasrockhu thomasrockhu merged commit a997fe8 into codecov:master Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants