[
https://issues.apache.org/jira/browse/MPMD-399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17854149#comment-17854149
]
ASF GitHub Bot commented on MPMD-399:
-------------------------------------
michael-o commented on code in PR #154:
URL: https://github.com/apache/maven-pmd-plugin/pull/154#discussion_r1635302700
##########
src/main/java/org/apache/maven/plugins/pmd/PmdReport.java:
##########
@@ -513,23 +513,11 @@ private String determineAuxClasspath() throws
MavenReportException {
resolvedArtifact.getArtifact().getFile().toString());
}
- List<String> projectClasspath = includeTests
- ? localProject.getTestClasspathElements()
- : localProject.getCompileClasspathElements();
-
- // Add the project's target folder first
- classpath.addAll(projectClasspath);
- if (!localProject.isExecutionRoot()) {
- for (String path : projectClasspath) {
- File pathFile = new File(path);
- String[] children = pathFile.list();
-
- if (!pathFile.exists() || (children != null &&
children.length == 0)) {
- getLog().warn("The project " +
localProject.getArtifactId()
- + " does not seem to be compiled. PMD
results might be inaccurate.");
Review Comment:
I see. Maybe this goal should fork `compile` or ` test-compile`?
> Incorrect warning: The project X does not seem to be compiled. PMD results
> might be inaccurate.
> -----------------------------------------------------------------------------------------------
>
> Key: MPMD-399
> URL: https://issues.apache.org/jira/browse/MPMD-399
> Project: Maven PMD Plugin
> Issue Type: Bug
> Components: PMD
> Affects Versions: 3.22.0
> Reporter: AB-xdev
> Priority: Major
>
> PMD plugin is reporting the following warnings on in a multi-maven project:
> "The project X does not seem to be compiled. PMD results might be inaccurate."
> This is caused by the fact that not all modules have a
> * {{src/main/java}} or
> * {{src/test/java}} folder
> and therefore no {{target/classes}} or {{target/test-classes}} folders are
> generated.
> The affected code that causes the problem can be found here:
> [https://github.com/apache/maven-pmd-plugin/blob/1961d771ea585e9344c25dfde194ee64a1b9040b/src/main/java/org/apache/maven/plugins/pmd/PmdReport.java#L516-L532]
> h3. Reproduction example
> * Create a maven multi-module project
> * Leave {{src/main/java}} of one module empty
> * Execute {{pmd:aggregate-pmd}} or {{test pmd:aggregate-pmd-no-fork
> -DskipTests}}
> → (x) Warning is reported
> h3. Possible fixes
> * Make it possible to ignore this warning
> * Do not warn when at least {{target/classes}} or {{target/test-classes}} is
> present
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
