You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by el...@apache.org on 2025/08/06 17:33:24 UTC
(maven-scm) 01/01: Reduce amount of log junk we emit
This is an automated email from the ASF dual-hosted git repository.
elharo pushed a commit to branch logger
in repository https://gitbox.apache.org/repos/asf/maven-scm.git
commit 07b3848d5ef721e79bf728cc1a1d9ba2911136e9
Author: Elliotte Rusty Harold <el...@ibiblio.org>
AuthorDate: Wed Aug 6 18:33:01 2025 +0100
Reduce amount of log junk we emit
---
.../java/org/apache/maven/scm/plugin/AbstractScmMojo.java | 2 +-
.../src/main/java/org/apache/maven/scm/plugin/AddMojo.java | 2 +-
.../main/java/org/apache/maven/scm/plugin/BranchMojo.java | 2 +-
.../main/java/org/apache/maven/scm/plugin/ChangeLogMojo.java | 2 +-
.../apache/maven/scm/plugin/CheckLocalModificationsMojo.java | 2 +-
.../src/main/java/org/apache/maven/scm/plugin/DiffMojo.java | 2 +-
.../main/java/org/apache/maven/scm/plugin/ExportMojo.java | 2 +-
.../src/main/java/org/apache/maven/scm/plugin/TagMojo.java | 6 +++---
.../main/java/org/apache/maven/scm/plugin/UpdateMojo.java | 4 ++--
.../scm/provider/hg/command/status/HgStatusConsumer.java | 12 ++++++------
.../git/jgit/command/checkin/JGitCheckInCommand.java | 2 +-
.../provider/svn/svnexe/command/untag/SvnUntagCommand.java | 6 +++---
12 files changed, 22 insertions(+), 22 deletions(-)
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java
index cc1a8fd5b..2c356990a 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java
@@ -227,7 +227,7 @@ public void execute() throws MojoExecutionException {
for (Entry<String, String> entry : providerImplementations.entrySet()) {
String providerType = entry.getKey();
String providerImplementation = entry.getValue();
- getLog().info("Change the default '" + providerType + "' provider implementation to '"
+ getLog().debug("Change the default '" + providerType + "' provider implementation to '"
+ providerImplementation + "'.");
getScmManager().setScmProviderImplementation(providerType, providerImplementation);
}
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AddMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AddMojo.java
index a4145e001..09ceb1257 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AddMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AddMojo.java
@@ -54,7 +54,7 @@ public void execute() throws MojoExecutionException {
checkResult(result);
- getLog().info("" + result.getAddedFiles().size() + " files successfully added.");
+ getLog().debug("" + result.getAddedFiles().size() + " files successfully added.");
} catch (IOException | ScmException e) {
throw new MojoExecutionException("Cannot run add command : ", e);
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/BranchMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/BranchMojo.java
index 3ac8e46b1..855df16f3 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/BranchMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/BranchMojo.java
@@ -86,7 +86,7 @@ public void execute() throws MojoExecutionException {
ScmProvider provider = getScmManager().getProviderByRepository(repository);
String finalBranch = provider.sanitizeTagName(branch);
- getLog().info("Final Branch Name: '" + finalBranch + "'");
+ getLog().debug("Final Branch Name: '" + finalBranch + "'");
ScmBranchParameters scmBranchParameters = new ScmBranchParameters(message);
scmBranchParameters.setRemoteBranching(remoteBranching);
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/ChangeLogMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/ChangeLogMojo.java
index a91158e20..f6fc6be8e 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/ChangeLogMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/ChangeLogMojo.java
@@ -201,7 +201,7 @@ public void execute() throws MojoExecutionException {
ChangeLogSet changeLogSet = result.getChangeLog();
for (ChangeSet changeSet : changeLogSet.getChangeSets()) {
- getLog().info(changeSet.toString());
+ getLog().debug(changeSet.toString());
}
} catch (IOException | ScmException e) {
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java
index 1e22fac0d..58d331766 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckLocalModificationsMojo.java
@@ -61,7 +61,7 @@ public CheckLocalModificationsMojo(ScmManager manager, SettingsDecrypter setting
public void execute() throws MojoExecutionException {
if (skip) {
- getLog().info("check-local-modification execution has been skipped");
+ getLog().debug("check-local-modification execution has been skipped");
return;
}
super.execute();
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/DiffMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/DiffMojo.java
index ce1de9b60..439a23a81 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/DiffMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/DiffMojo.java
@@ -91,7 +91,7 @@ public void execute() throws MojoExecutionException {
checkResult(result);
- getLog().info(result.getPatch());
+ getLog().debug(result.getPatch());
try {
if (outputFile != null) {
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/ExportMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/ExportMojo.java
index 703100178..ea8a1b5fb 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/ExportMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/ExportMojo.java
@@ -99,7 +99,7 @@ protected void export() throws MojoExecutionException {
try {
if (this.exportDirectory.exists()) {
- this.getLog().info("Removing " + this.exportDirectory);
+ this.getLog().debug("Removing " + this.exportDirectory);
FileUtils.deleteDirectory(this.exportDirectory);
}
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java
index 3e3c2bf6f..3c26995c2 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/TagMojo.java
@@ -131,10 +131,10 @@ public void execute() throws MojoExecutionException {
if (addTimestamp) {
try {
- getLog().info("Using timestamp pattern '" + timestampFormat + "'");
+ getLog().debug("Using timestamp pattern '" + timestampFormat + "'");
dateFormat = new SimpleDateFormat(timestampFormat);
tagTimestamp = dateFormat.format(new Date());
- getLog().info("Using timestamp '" + tagTimestamp + "'");
+ getLog().debug("Using timestamp '" + tagTimestamp + "'");
} catch (IllegalArgumentException e) {
String msg = "The timestamp format '" + timestampFormat + "' is invalid.";
getLog().error(msg, e);
@@ -152,7 +152,7 @@ public void execute() throws MojoExecutionException {
ScmProvider provider = getScmManager().getProviderByRepository(repository);
finalTag = provider.sanitizeTagName(finalTag);
- getLog().info("Final Tag Name: '" + finalTag + "'");
+ getLog().debug("Final Tag Name: '" + finalTag + "'");
ScmTagParameters scmTagParameters = new ScmTagParameters(message);
scmTagParameters.setRemoteTagging(remoteTagging);
diff --git a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/UpdateMojo.java b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/UpdateMojo.java
index 351bcd05f..b0002e7db 100644
--- a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/UpdateMojo.java
+++ b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/UpdateMojo.java
@@ -90,14 +90,14 @@ public void execute() throws MojoExecutionException {
if (result instanceof UpdateScmResultWithRevision) {
String revision = ((UpdateScmResultWithRevision) result).getRevision();
- getLog().info("Storing revision in '" + revisionKey + "' project property.");
+ getLog().debug("Storing revision in '" + revisionKey + "' project property.");
if (project.getProperties() != null) // Remove the test when we'll use plugin-test-harness 1.0-alpha-2
{
project.getProperties().put(revisionKey, revision);
}
- getLog().info("Project at revision " + revision);
+ getLog().debug("Project at revision " + revision);
}
} catch (IOException | ScmException e) {
throw new MojoExecutionException("Cannot run update command : ", e);
diff --git a/maven-scm-providers/maven-scm-provider-hg/src/main/java/org/apache/maven/scm/provider/hg/command/status/HgStatusConsumer.java b/maven-scm-providers/maven-scm-provider-hg/src/main/java/org/apache/maven/scm/provider/hg/command/status/HgStatusConsumer.java
index f4c0eccae..c7f1f030b 100644
--- a/maven-scm-providers/maven-scm-provider-hg/src/main/java/org/apache/maven/scm/provider/hg/command/status/HgStatusConsumer.java
+++ b/maven-scm-providers/maven-scm-provider-hg/src/main/java/org/apache/maven/scm/provider/hg/command/status/HgStatusConsumer.java
@@ -44,17 +44,17 @@ public void doConsume(ScmFileStatus status, String trimmedLine) {
// Only include real files (not directories)
File tmpFile = new File(workingDir, trimmedLine);
if (!tmpFile.exists()) {
- if (logger.isInfoEnabled()) {
- logger.info("Not a file: " + tmpFile + ". Ignoring");
+ if (logger.isDebugEnabled()) {
+ logger.debug("Not a file: " + tmpFile + ". Ignoring");
}
} else if (tmpFile.isDirectory()) {
- if (logger.isInfoEnabled()) {
- logger.info("New directory added: " + tmpFile);
+ if (logger.isDebugEnabled()) {
+ logger.debug("New directory added: " + tmpFile);
}
} else {
ScmFile scmFile = new ScmFile(trimmedLine, status);
- if (logger.isInfoEnabled()) {
- logger.info(scmFile.toString());
+ if (logger.isDebugEnabled()) {
+ logger.debug(scmFile.toString());
}
repositoryStatus.add(scmFile);
}
diff --git a/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-jgit/src/main/java/org/apache/maven/scm/provider/git/jgit/command/checkin/JGitCheckInCommand.java b/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-jgit/src/main/java/org/apache/maven/scm/provider/git/jgit/command/checkin/JGitCheckInCommand.java
index 34b4053b6..51a175061 100644
--- a/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-jgit/src/main/java/org/apache/maven/scm/provider/git/jgit/command/checkin/JGitCheckInCommand.java
+++ b/maven-scm-providers/maven-scm-providers-git/maven-scm-provider-jgit/src/main/java/org/apache/maven/scm/provider/git/jgit/command/checkin/JGitCheckInCommand.java
@@ -156,7 +156,7 @@ public CheckInScmResult executeCommand(ScmProviderRepository repo, ScmFileSet fi
.setSign(parameters.getBoolean(CommandParameter.SCM_COMMIT_SIGN, true));
RevCommit commitRev = command.call();
- logger.info("commit done: " + commitRev.getShortMessage());
+ logger.debug("commit done: " + commitRev.getShortMessage());
checkedInFiles = JGitUtils.getFilesInCommit(git.getRepository(), commitRev, fileSet.getBasedir());
if (logger.isDebugEnabled()) {
for (ScmFile scmFile : checkedInFiles) {
diff --git a/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/untag/SvnUntagCommand.java b/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/untag/SvnUntagCommand.java
index e0e298af8..568345bc8 100644
--- a/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/untag/SvnUntagCommand.java
+++ b/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/untag/SvnUntagCommand.java
@@ -83,11 +83,11 @@ public ScmResult executeUntagCommand(
CommandLineUtils.StringStreamConsumer stderr = new CommandLineUtils.StringStreamConsumer();
- if (logger.isInfoEnabled()) {
- logger.info("Executing: " + SvnCommandLineUtils.cryptPassword(cl));
+ if (logger.isDebugEnabled()) {
+ logger.debug("Executing: " + SvnCommandLineUtils.cryptPassword(cl));
if (Os.isFamily(Os.FAMILY_WINDOWS)) {
- logger.info("Working directory: " + cl.getWorkingDirectory().getAbsolutePath());
+ logger.debug("Working directory: " + cl.getWorkingDirectory().getAbsolutePath());
}
}