This is an automated email from the ASF dual-hosted git repository.
elharo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-scm.git
The following commit(s) were added to refs/heads/master by this push:
new b008e4c22 Reduce amount of log junk emitted (#1286)
b008e4c22 is described below
commit b008e4c2209cd8f14506780cabdaebd7a9702d8b
Author: Elliotte Rusty Harold <elh...@users.noreply.github.com>
AuthorDate: Fri Sep 12 13:08:50 2025 -0400
Reduce amount of log junk emitted (#1286)
* Reduce amount of log junk we emit
---
.../java/org/apache/maven/scm/plugin/AbstractScmMojo.java | 2 +-
.../main/java/org/apache/maven/scm/plugin/BranchMojo.java | 2 +-
.../main/java/org/apache/maven/scm/plugin/ChangeLogMojo.java | 2 +-
.../main/java/org/apache/maven/scm/plugin/CheckoutMojo.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 | 2 +-
.../scm/provider/hg/command/checkout/HgCheckOutCommand.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/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 cf84e1cce..0d84d4971 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/CheckoutMojo.java
b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckoutMojo.java
index e5382a149..ef279080f 100644
---
a/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckoutMojo.java
+++
b/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/CheckoutMojo.java
@@ -146,7 +146,7 @@ protected ScmResult checkout() throws
MojoExecutionException {
private void prepareOutputDirectory(File ouputDirectory) throws
MojoExecutionException {
try {
- this.getLog().info("Removing " + ouputDirectory);
+ this.getLog().debug("Removing " + ouputDirectory);
FileUtils.deleteDirectory(getCheckoutDirectory());
} catch (IOException e) {
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 eaeb41ea9..b05bac4c8 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
@@ -134,10 +134,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.";
throw new MojoExecutionException(msg, e);
@@ -154,7 +154,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..723d86cd3 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,7 +90,7 @@ 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
{
diff --git
a/maven-scm-providers/maven-scm-provider-hg/src/main/java/org/apache/maven/scm/provider/hg/command/checkout/HgCheckOutCommand.java
b/maven-scm-providers/maven-scm-provider-hg/src/main/java/org/apache/maven/scm/provider/hg/command/checkout/HgCheckOutCommand.java
index fc0b737dd..ae237748b 100644
---
a/maven-scm-providers/maven-scm-provider-hg/src/main/java/org/apache/maven/scm/provider/hg/command/checkout/HgCheckOutCommand.java
+++
b/maven-scm-providers/maven-scm-provider-hg/src/main/java/org/apache/maven/scm/provider/hg/command/checkout/HgCheckOutCommand.java
@@ -55,8 +55,8 @@ protected CheckOutScmResult executeCheckOutCommand(
File checkoutDir = fileSet.getBasedir();
try {
- if (logger.isInfoEnabled()) {
- logger.info("Removing " + checkoutDir);
+ if (logger.isDebugEnabled()) {
+ logger.debug("Removing " + checkoutDir);
}
FileUtils.deleteDirectory(checkoutDir);
} catch (IOException 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 d42ad6d74..53a8191fa 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
@@ -159,7 +159,7 @@ public CheckInScmResult
executeCommand(ScmProviderRepository repo, ScmFileSet fi
}
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());
}
}
