You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2020/05/15 06:43:33 UTC
[maven-release] branch MRELEASE-1023 created (now 9bca21f)
This is an automated email from the ASF dual-hosted git repository.
rfscholte pushed a change to branch MRELEASE-1023
in repository https://gitbox.apache.org/repos/asf/maven-release.git.
at 9bca21f Remove constructors with unused parameters
This branch includes the following new commits:
new 77e0931 Remove unused private fields
new 9bca21f Remove constructors with unused parameters
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.
[maven-release] 01/02: Remove unused private fields
Posted by rf...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
rfscholte pushed a commit to branch MRELEASE-1023
in repository https://gitbox.apache.org/repos/asf/maven-release.git
commit 77e09313c4e24258cdf6378957c584907612407b
Author: Piotrek Żygieło <pz...@users.noreply.github.com>
AuthorDate: Tue Apr 14 14:15:11 2020 +0200
Remove unused private fields
---
.../apache/maven/shared/release/exec/MavenExecutorException.java | 8 --------
1 file changed, 8 deletions(-)
diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java
index 3bd996c..5bc6a17 100644
--- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java
+++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java
@@ -27,10 +27,6 @@ package org.apache.maven.shared.release.exec;
public class MavenExecutorException
extends Exception
{
- private String stdErr;
-
- private String stdOut;
-
private int exitCode;
public MavenExecutorException( String message, int exitCode, String stdOut, String stdErr )
@@ -38,16 +34,12 @@ public class MavenExecutorException
super( message );
this.exitCode = exitCode;
- this.stdOut = stdOut;
- this.stdErr = stdErr;
}
public MavenExecutorException( String message, String stdOut, String stdErr, Exception e )
{
super( message, e );
- this.stdOut = stdOut;
- this.stdErr = stdErr;
}
public MavenExecutorException( String message, Throwable cause )
[maven-release] 02/02: Remove constructors with unused parameters
Posted by rf...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
rfscholte pushed a commit to branch MRELEASE-1023
in repository https://gitbox.apache.org/repos/asf/maven-release.git
commit 9bca21f27675b15782294318a309cac02200350b
Author: Piotrek Żygieło <pz...@users.noreply.github.com>
AuthorDate: Fri May 8 12:00:06 2020 +0200
Remove constructors with unused parameters
---
.../org/apache/maven/shared/release/exec/ForkedMavenExecutor.java | 5 ++---
.../apache/maven/shared/release/exec/InvokerMavenExecutor.java | 2 +-
.../apache/maven/shared/release/exec/MavenExecutorException.java | 8 +-------
3 files changed, 4 insertions(+), 11 deletions(-)
diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java
index e4a5539..af0051f 100644
--- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java
+++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/ForkedMavenExecutor.java
@@ -141,13 +141,12 @@ public class ForkedMavenExecutor
if ( result != 0 )
{
- throw new MavenExecutorException( "Maven execution failed, exit code: \'" + result + "\'", result,
- stdOut.toString(), stdErr.toString() );
+ throw new MavenExecutorException( "Maven execution failed, exit code: \'" + result + "\'", result );
}
}
catch ( CommandLineException e )
{
- throw new MavenExecutorException( "Can't run goal " + goals, stdOut.toString(), stdErr.toString(), e );
+ throw new MavenExecutorException( "Can't run goal " + goals, e );
}
finally
{
diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/InvokerMavenExecutor.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/InvokerMavenExecutor.java
index df61c39..eb9e044 100644
--- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/InvokerMavenExecutor.java
+++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/InvokerMavenExecutor.java
@@ -392,7 +392,7 @@ public class InvokerMavenExecutor
{
throw new MavenExecutorException(
"Maven execution failed, exit code: \'" + invocationResult.getExitCode() + "\'",
- invocationResult.getExitCode(), "", "" );
+ invocationResult.getExitCode() );
}
}
catch ( MavenInvocationException e )
diff --git a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java
index 5bc6a17..74300c7 100644
--- a/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java
+++ b/maven-release-manager/src/main/java/org/apache/maven/shared/release/exec/MavenExecutorException.java
@@ -29,19 +29,13 @@ public class MavenExecutorException
{
private int exitCode;
- public MavenExecutorException( String message, int exitCode, String stdOut, String stdErr )
+ public MavenExecutorException( String message, int exitCode )
{
super( message );
this.exitCode = exitCode;
}
- public MavenExecutorException( String message, String stdOut, String stdErr, Exception e )
- {
- super( message, e );
-
- }
-
public MavenExecutorException( String message, Throwable cause )
{
super( message, cause );