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

Eliminate Javadoc errors #546

Merged
merged 1 commit into from
Sep 18, 2024
Merged

Conversation

merks
Copy link
Contributor

@merks merks commented Sep 18, 2024

No description provided.

Copy link

github-actions bot commented Sep 18, 2024

Test Results

  375 files  ±0    375 suites  ±0   41m 47s ⏱️ - 3m 32s
1 895 tests ±0  1 892 ✅ ±0  3 💤 ±0  0 ❌ ±0 
6 685 runs  ±0  6 676 ✅ ±0  9 💤 ±0  0 ❌ ±0 

Results for commit 31eaa60. ± Comparison against base commit 4a873fb.

♻️ This comment has been updated with latest results.

@merks merks force-pushed the pr-javadoc-errors branch 10 times, most recently from 5258bca to 0c5a170 Compare September 18, 2024 14:24
@@ -25,4 +25,5 @@ source.lib/repository-tools-ant.jar = src_ant/
output.lib/repository-tools-ant.jar = bin_ant/
jars.extra.classpath=platform:/plugin/org.apache.ant/lib/ant.jar,\
platform:/plugin/org.eclipse.equinox.p2.jarprocessor/lib/jarprocessor-ant.jar
pom.model.property.javadoc.excludePackageNames = org.eclipse.equinox.p2.internal.repository.tools.tasks
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@laeubi @HannesWell

I tried to use this to eliminate the last Javadoc errors.

But still I get this:

16:32:38  [ERROR] MavenReportException: Error while generating Javadoc: 
16:32:38  Exit code: 1
16:32:38  /home/jenkins/agent/workspace/p2_PR-546/bundles/org.eclipse.equinox.p2.repository.tools/src_ant/org/eclipse/equinox/p2/internal/repository/tools/tasks/ProcessRepoTask.java:25: error: package org.eclipse.equinox.internal.p2.jarprocessor.ant does not exist
16:32:38  import org.eclipse.equinox.internal.p2.jarprocessor.ant.JarProcessorTask;
16:32:38                                                         ^
16:32:38  /home/jenkins/agent/workspace/p2_PR-546/bundles/org.eclipse.equinox.p2.repository.tools/src_ant/org/eclipse/equinox/p2/internal/repository/tools/tasks/ProcessRepoTask.java:64: error: cannot find symbol
16:32:38  	private JarProcessorTask jarProcessor = null;
16:32:38  	        ^
16:32:38    symbol:   class JarProcessorTask
16:32:38    location: class ProcessRepoTask
16:32:38  2 errors
16:32:38  Command line was: /opt/tools/java/temurin/jdk-17/jdk-17.0.10+7/bin/javadoc -J-Duser.language= -J-Duser.country= @options @packages

Any idea?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

We should consider to add this to the eclipse-parent so that not each project that needs it has to duplicate it.

@merks merks merged commit ad97cb5 into eclipse-equinox:master Sep 18, 2024
11 checks passed
@merks merks deleted the pr-javadoc-errors branch September 18, 2024 19:02
@merks
Copy link
Contributor Author

merks commented Sep 18, 2024

@laeubi @HannesWell

I think it would be good if the p2 verification build would fail from now on when there are Javadoc errors.

@laeubi
Copy link
Member

laeubi commented Sep 19, 2024

You just need to define this property in the main pom like PDE does:

https://github.com/eclipse-pde/eclipse.pde/blob/b00c27d7366cd34c764d3046f32ed808d301bbfd/pom.xml#L30

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.

3 participants