-
|
Can the release notes specify if the release candidates are supported by Maven PMD Plugin? |
Beta Was this translation helpful? Give feedback.
Replies: 11 comments 28 replies
-
|
maven-pmd-plugin 3.20.0/3.21.0 does indeed not support PMD 7. As this is a major version change, there are a couple of incompatibilities. See https://issues.apache.org/jira/browse/MPMD-379. There is a snapshot version available, that supports this. We use it in our CI build.
If you want to use it, you need to add apache's snapshot repo: <pluginRepository>
<id>apache.snapshots</id>
<name>Apache Snapshot Repository</name>
<url>https://repository.apache.org/snapshots</url>
<releases>
<enabled>false</enabled>
</releases>
<snapshots>
<enabled>true</enabled>
</snapshots>
</pluginRepository>This SNAPSHOT is built from that branch: https://github.com/apache/maven-pmd-plugin/tree/pmd7 I didn't have time yet to make it production ready... |
Beta Was this translation helpful? Give feedback.
-
|
Thanks, @adangel . |
Beta Was this translation helpful? Give feedback.
-
|
Heyo @adangel - so I'm working in a space where we have adopted Java 21 and use PMD, through Since the current released version of We don't have the option to revert support for Java 21, so the only options left are:
As a fix, we're okay to use release candidate (we usually don't), but since the only Would it be possible to push a release candidate for the |
Beta Was this translation helpful? Give feedback.
-
|
I'm encountering errors below while executing the Maven PMD Plugin Snapshot: I don't see these while executing the Ant task against 7.0.0-rc1. Am I to assume that's because the development pmd jars are also bundled with the Maven plugin? |
Beta Was this translation helpful? Give feedback.
-
|
It seems these rules have been updated since for PMD 7. No probs. I will have to check why Ant didn't flag an error. |
Beta Was this translation helpful? Give feedback.
-
|
With Ant and Ivy, I'm getting the following error for the pmd-cli compile dependencies: The 7.0.0-SNAPSHOT for pmd-ui is not available on Maven repo. Is there some configuration for ivy that I need to set up? |
Beta Was this translation helpful? Give feedback.
-
|
That is a known problem with 7.0.0-rc1 (see #4446). It will be fixed with rc2: PR #4475 will use a non-snapshot version for the pmd-ui/designer dependency. You can remove or exclude the dependency to pmd-cli to avoid this problem. Or better - replace the dependency to pmd-cli to pmd-ant. You don't need to pmd-cli to run the ant task, you only need pmd-ant. |
Beta Was this translation helpful? Give feedback.
-
|
Can Ivy be configured to download it from https://oss.sonatype.org/content/repositories/snapshots/? |
Beta Was this translation helpful? Give feedback.
-
|
Looking at https://github.com/Fernal73/PMDSample/blob/main/ivysettings.xml you just need to add this repo... |
Beta Was this translation helpful? Give feedback.
-
|
Is the configuration similar to sevntu? The above is borrowed from one of their sample configurations. |
Beta Was this translation helpful? Give feedback.
-
|
I was able to use maven-pmd-plugin 3.21.0 after applying the following changes to PMD 7.0.0-SNAPSHOT: #4609 |
Beta Was this translation helpful? Give feedback.
-
|
Here is now the new solution for PMD 7.0.0-rc4 and later: In order to use PMD 7 with maven-pmd-plugin a new
<dependency>
<groupId>net.sourceforge.pmd</groupId>
<artifactId>pmd-compat6</artifactId>
<!--<version>${pmdVersion}</version>-->
<version>7.0.0-rc4+with-pr4749</version> <!-- this contains the fix from #4749 -->
</dependency>It is important to add this dependency as the first in the list, so that maven-pmd-plugin sees the (old) This module is available beginning with version 7.0.0-rc4 and will be there at least for the first Note: This compatibility module only works for the built-in rules, that are still available in PMD 7. E.g. you need As PMD 7 revamped the Java module, if you have custom rules, you need to migrate these rules. |
Beta Was this translation helpful? Give feedback.
-
|
@adangel Thanks for working on this! Unfortunately, I'm still getting an error when trying to build junixsocket with the updated configuration: |
Beta Was this translation helpful? Give feedback.
-
|
Sorry I was unclear
|
Beta Was this translation helpful? Give feedback.
-
|
To further clarify #3. This affects any java version not just 21, and hence means no java folks who run into this can use PMD 7 currently with maven. |
Beta Was this translation helpful? Give feedback.
-
|
Here you are: <dependency>
<groupId>net.sourceforge.pmd</groupId>
<artifactId>pmd-compat6</artifactId>
<!--<version>${pmd.version}</version>-->
<version>7.0.0-rc4+with-pr4749</version>
</dependency> |
Beta Was this translation helpful? Give feedback.
-
|
Thank you! Still waiting for maven central to percolate it's indexing (but I can see the file has arrived). |
Beta Was this translation helpful? Give feedback.
-
|
I can confirm it works perfectly, as expected. Thanks for your extra above-and-beyond efforts, it is very much appreciated! |
Beta Was this translation helpful? Give feedback.
-
|
I really dont understand what I'm doing wrong, but I can't get past this error ... could anyone help? this is my setup |
Beta Was this translation helpful? Give feedback.
-
|
Please specify - in addition to pmd-java - also the dependencies to pmd-javascript and pmd-jsp. Otherwise the maven plugin tries to initialize the old versions of jsp/javascript languages, which won't work.
|
Beta Was this translation helpful? Give feedback.
-
|
thank you so much, that was indeed the issue :) |
Beta Was this translation helpful? Give feedback.
-
|
I'm seeing this error [ERROR] Failed to execute goal org.apache.maven.plugins:maven-pmd-plugin:3.21.2:pmd (default-cli) on project data: Execution default-cli of goal org.apache.maven.plugins:maven-pmd-plugin:3.21.2:pmd failed: An API incompatibility was encountered while executing org.apache.maven.plugins:maven-pmd-plugin:3.21.2:pmd: java.lang.NoSuchMethodError: 'java.lang.String net.sourceforge.pmd.Report$ProcessingError.getFile()' |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for the very quick response, I'm up and running now thanks |
Beta Was this translation helpful? Give feedback.
-
|
I'm still seeing intermittent errors about reporting of errors, despite using renderProcessingErrors false, running again normally works. Has anyone else seen this? |
Beta Was this translation helpful? Give feedback.
-
|
@netmindz Which kind of intermittent errors? Do the stacktraces look similar to the ones in #4757? |
Beta Was this translation helpful? Give feedback.
-
|
I would need to double-check, but I think it was java.lang.String net.sourceforge.pmd.Report$ProcessingError.getFile() |
Beta Was this translation helpful? Give feedback.
-
Yeah, I was wrong about that option. The missing method |
Beta Was this translation helpful? Give feedback.
Here is now the new solution for PMD 7.0.0-rc4 and later:
In order to use PMD 7 with maven-pmd-plugin a new
compatibility module has been created. This allows to use PMD 7 by simply adding one additional dependency:
It is important to add this dependency as the first in the list, so that maven-pmd-plugin sees the (old)
compatible versions of some classes.
This module is ava…