Author: olamy
Date: Sat Mar 21 10:18:45 2009
New Revision: 756909

URL: http://svn.apache.org/viewvc?rev=756909&view=rev
Log:
[SCM-443] perforce provider : login failure must be displayed

Modified:
    
maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/login/PerforceLoginCommand.java

Modified: 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/login/PerforceLoginCommand.java
URL: 
http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/login/PerforceLoginCommand.java?rev=756909&r1=756908&r2=756909&view=diff
==============================================================================
--- 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/login/PerforceLoginCommand.java
 (original)
+++ 
maven/scm/trunk/maven-scm-providers/maven-scm-provider-perforce/src/main/java/org/apache/maven/scm/provider/perforce/command/login/PerforceLoginCommand.java
 Sat Mar 21 10:18:45 2009
@@ -70,7 +70,7 @@
                                                                     consumer, 
err );
                 isSuccess = consumer.isSuccess();
 
-                if ( isSuccess )
+                if ( !isSuccess )
                 {
                     String cmdLine = CommandLineUtils.toString( 
cl.getCommandline() );
 
@@ -84,7 +84,7 @@
         }
         catch ( CommandLineException e )
         {
-            throw new ScmException( "", e );
+            throw new ScmException( e.getMessage(), e );
         }
 
         return new LoginScmResult( cl.toString(), isSuccess ? "Login 
successful" : "Login failed",


Reply via email to