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:47:41 UTC
[maven-pmd-plugin] branch MPMD-291 updated (7b8005f -> 327d7d8)
This is an automated email from the ASF dual-hosted git repository.
adangel pushed a change to branch MPMD-291
in repository https://gitbox.apache.org/repos/asf/maven-pmd-plugin.git.
from 7b8005f [MPMD-291] Upgrade to PMD 6.17.0
new bb785ed [MPMD-292] - PMD Log is not always integrated into maven log
new 327d7d8 [MPMD-292] - Fix integration test builds on jenkins
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
src/it/MPMD-187/invoker.properties | 3 +--
.../java/org/apache/maven/plugins/pmd/AbstractPmdReport.java | 10 +++++++++-
2 files changed, 10 insertions(+), 3 deletions(-)
[maven-pmd-plugin] 02/02: [MPMD-292] - Fix integration test builds
on jenkins
Posted by ad...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
adangel pushed a commit to branch MPMD-291
in repository https://gitbox.apache.org/repos/asf/maven-pmd-plugin.git
commit 327d7d8a22139836de3c62d7fb2ba2aaa06e676c
Author: Andreas Dangel <ad...@apache.org>
AuthorDate: Thu Aug 1 21:47:05 2019 +0200
[MPMD-292] - Fix integration test builds on jenkins
---
src/it/MPMD-187/invoker.properties | 3 +--
1 file changed, 1 insertion(+), 2 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
[maven-pmd-plugin] 01/02: [MPMD-292] - PMD Log is not always
integrated into maven log
Posted by ad...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
adangel pushed a commit to branch MPMD-291
in repository https://gitbox.apache.org/repos/asf/maven-pmd-plugin.git
commit bb785ed73f7cf01afcfef3e07cfdb02b06d6e536
Author: Andreas Dangel <ad...@apache.org>
AuthorDate: Thu Aug 1 20:31:00 2019 +0200
[MPMD-292] - PMD Log is not always integrated into maven log
---
.../java/org/apache/maven/plugins/pmd/AbstractPmdReport.java | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/src/main/java/org/apache/maven/plugins/pmd/AbstractPmdReport.java b/src/main/java/org/apache/maven/plugins/pmd/AbstractPmdReport.java
index 53a6da2..4cadbcd 100644
--- a/src/main/java/org/apache/maven/plugins/pmd/AbstractPmdReport.java
+++ b/src/main/java/org/apache/maven/plugins/pmd/AbstractPmdReport.java
@@ -232,6 +232,13 @@ public abstract class AbstractPmdReport
@Parameter( defaultValue = "true", property = "pmd.showPmdLog" )
protected boolean showPmdLog = true;
+ /**
+ * This holds a strong reference in case we configured the logger to
+ * redirect to slf4j. See {@link #showPmdLog}. Without a strong reference,
+ * the logger might be garbage collected and the redirect to slf4j is gone.
+ */
+ private Logger julLogger;
+
/** The files that are being analyzed. */
protected Map<File, PmdFileInfo> filesToProcess;
@@ -576,7 +583,8 @@ public abstract class AbstractPmdReport
logger.addHandler( handler );
handler.setLevel( Level.ALL );
logger.setLevel( Level.ALL );
- getLog().debug( "Configured jul-to-slf4j bridge for " + logger.getName() );
+ julLogger = logger;
+ julLogger.fine( "Configured jul-to-slf4j bridge for " + logger.getName() );
}
static String getPmdVersion()