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

Do not use ~/.m2/settings.xml provided by Travis #742

Merged
merged 1 commit into from
Aug 18, 2018
Merged

Do not use ~/.m2/settings.xml provided by Travis #742

merged 1 commit into from
Aug 18, 2018

Conversation

Godin
Copy link
Member

@Godin Godin commented Aug 18, 2018

In Travis maven-javadoc-plugin:2.10.4 tries to download some artifacts and causes long build times:

...
[INFO] --- maven-javadoc-plugin:2.10.4:jar (attach-javadocs) @ org.jacoco.doc ---
...
[INFO] Downloading from sonatype: https://oss.sonatype.org/content/repositories/releases/org/ow2/asm/asm-commons/6.2.1/asm-commons-6.2.1-javadoc-resources.jar
[INFO] Downloading from sonatype-apache: https://repository.apache.org/releases/org/ow2/asm/asm-commons/6.2.1/asm-commons-6.2.1-javadoc-resources.jar
...
[INFO] JaCoCo :: Documentation ............................ SUCCESS [21:35 min]
...

Repositories sonatype and sonatype-apache are defined by Travis in its ~/.m2/settings.xml and seems that there is an issue with access to these repositoeis from Travis (can't reproduce locally).


Attempts to download disappear after upgrade of maven-javadoc-plugin to latest released version 3.0.1.
But I didn't managed to find ticket or corresponding change to understand why.

Also due to changes for MJAVADOC-498 this upgrade causes non fatal error messages

...
[INFO] --- maven-javadoc-plugin:3.0.1:jar (attach-javadocs) @ org.jacoco.doc ---
[ERROR] no module descriptor for org.jacoco:org.jacoco.doc
[ERROR] no module descriptor for org.jacoco:org.jacoco.core
[ERROR] no module descriptor for org.jacoco:org.jacoco.report
[ERROR] no module descriptor for org.jacoco:org.jacoco.agent
[ERROR] no module descriptor for org.jacoco:org.jacoco.agent.rt
[ERROR] no module descriptor for org.jacoco:org.jacoco.ant
[ERROR] no module descriptor for org.jacoco:org.jacoco.cli
[ERROR] no module descriptor for org.jacoco:org.jacoco.examples
[ERROR] no module descriptor for org.jacoco:jacoco-maven-plugin
...
[INFO] JaCoCo :: Documentation ............................ SUCCESS [  4.643 s]
[INFO] JaCoCo :: Distribution ............................. SUCCESS [  1.540 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
...

even when using JDK 8.


Since in absence of custom repositories attempts to download also disappear, I propose to simply stop using ~/.m2/settings.xml that is provided by Travis.

@Godin Godin self-assigned this Aug 18, 2018
@Godin Godin added this to Review in Current work items via automation Aug 18, 2018
@Godin Godin added this to the 0.8.2 milestone Aug 18, 2018
Copy link
Member

@marchof marchof left a comment

Choose a reason for hiding this comment

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

Nice that you identified this!

@Godin Godin merged commit 3288f34 into master Aug 18, 2018
Current work items automation moved this from Review to Done Aug 18, 2018
@Godin Godin deleted the travis branch August 18, 2018 21:15
@jacoco jacoco locked as resolved and limited conversation to collaborators Oct 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants