Skip to content

[MJAVADOC-538] Filter out 'Picked up'#11

Merged
rfscholte merged 1 commit intoapache:masterfrom
risdenk:MJAVADOC-538
Nov 19, 2018
Merged

[MJAVADOC-538] Filter out 'Picked up'#11
rfscholte merged 1 commit intoapache:masterfrom
risdenk:MJAVADOC-538

Conversation

@risdenk
Copy link
Copy Markdown
Contributor

@risdenk risdenk commented Nov 13, 2018

Tested with:

  • mvn -Prun-its verify

Additional testing on project I was having original issues

  • mvn install
  • export JAVA_TOOL_OPTIONS="-Dapple.awt.UIElement=true"
  • Set pom for project to use 3.0.2-SNAPSHOT
<pluginManagement>
  <plugins>
    <plugin>
      <groupId>org.apache.maven.plugins</groupId>
      <artifactId>maven-javadoc-plugin</artifactId>
      <version>3.0.2-SNAPSHOT</version>
    </plugin>
  </plugins>
</pluginManagement>
  • Ran the following on my project - mvn -Dmaven.javadoc.failOnError=true -Dmaven.javadoc.failOnWarnings=true javadoc:javadoc javadoc:test-javadoc

Prior to this change was getting failure from warning "Picked up ...". With this change, Javadoc no longer fails due to warning.

@yrodiere
Copy link
Copy Markdown

FWIW, I can confirm this solves the problem in my project. Thanks!

@risdenk
Copy link
Copy Markdown
Contributor Author

risdenk commented Nov 18, 2018

Added integration test.

Copy link
Copy Markdown
Contributor

@rfscholte rfscholte left a comment

Choose a reason for hiding this comment

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

Just a couple of small things to improve, almost there.

Comment thread pom.xml
Comment thread src/it/projects/MJAVADOC-538/pom.xml
Comment thread src/it/projects/MJAVADOC-538/invoker.properties Outdated
Comment thread src/it/projects/MJAVADOC-538/pom.xml
@risdenk
Copy link
Copy Markdown
Contributor Author

risdenk commented Nov 19, 2018

@rfscholte Addressed your review comments. Thanks!

@rfscholte
Copy link
Copy Markdown
Contributor

One final request: add yourself as contributor to the root pom.xml if you want.

Signed-off-by: Kevin Risden <krisden@apache.org>
@risdenk
Copy link
Copy Markdown
Contributor Author

risdenk commented Nov 19, 2018

@rfscholte added myself as contributor. Let me know if there is anything else :)

@rfscholte
Copy link
Copy Markdown
Contributor

https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven-javadoc-plugin/job/MJAVADOC-538/
I need to figure out what's wrong with Windows + Java 7 before merging (looks unrelated to this commit)

@rfscholte rfscholte merged commit 631a84e into apache:master Nov 19, 2018
@risdenk risdenk deleted the MJAVADOC-538 branch December 6, 2018 01:30
@jira-importer
Copy link
Copy Markdown

Resolve #807

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants