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:35 UTC
[maven-release] 02/02: Remove constructors with unused parameters
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 );