You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ad...@apache.org on 2019/08/01 19:30:30 UTC

[maven-pmd-plugin] branch it-mpmd-187 updated: do not use invoker.mavenOpts anymore - it collides with MAVEN_OPTS

This is an automated email from the ASF dual-hosted git repository.

adangel pushed a commit to branch it-mpmd-187
in repository https://gitbox.apache.org/repos/asf/maven-pmd-plugin.git


The following commit(s) were added to refs/heads/it-mpmd-187 by this push:
     new 7944a2f  do not use invoker.mavenOpts anymore - it collides with MAVEN_OPTS
7944a2f is described below

commit 7944a2f77a8e509ab8ba8e382c3c88b43ad61381
Author: Andreas Dangel <ad...@apache.org>
AuthorDate: Thu Aug 1 21:30:12 2019 +0200

    do not use invoker.mavenOpts anymore - it collides with MAVEN_OPTS
---
 src/it/MPMD-187/invoker.properties | 3 +--
 src/it/MPMD-187/verify.groovy      | 5 -----
 2 files changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/it/MPMD-187/invoker.properties b/src/it/MPMD-187/invoker.properties
index ac42a36..cdef1c3 100644
--- a/src/it/MPMD-187/invoker.properties
+++ b/src/it/MPMD-187/invoker.properties
@@ -15,5 +15,4 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.goals = clean verify
-invoker.mavenOpts = -Dpmd.skip=true -Dcpd.skip=true
+invoker.goals = clean verify -Dpmd.skip=true -Dcpd.skip=true
diff --git a/src/it/MPMD-187/verify.groovy b/src/it/MPMD-187/verify.groovy
index 996b217..4af16b7 100644
--- a/src/it/MPMD-187/verify.groovy
+++ b/src/it/MPMD-187/verify.groovy
@@ -21,11 +21,6 @@ File buildLog = new File( basedir, 'build.log' )
 assert buildLog.exists()
 
 File pmdXml = new File( basedir, "target/pmd.xml" )
-
-if ( pmdXml.exists() )
-{
-    throw new AssertionError( buildLog.getText() )
-}
 assert !pmdXml.exists()
 
 File cpdXml = new File( basedir, "target/cpd.xml" )