You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ol...@apache.org on 2012/03/28 23:40:46 UTC

svn commit: r1306607 - /maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/checkout/SvnCheckOutConsumer.java

Author: olamy
Date: Wed Mar 28 21:40:46 2012
New Revision: 1306607

URL: http://svn.apache.org/viewvc?rev=1306607&view=rev
Log:
adapt SvnCheckOutConsumer to be svn 1.7 friendly as the ouput is now relative path not anymore full path as for  1.7-

Modified:
    maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/checkout/SvnCheckOutConsumer.java

Modified: maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/checkout/SvnCheckOutConsumer.java
URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/checkout/SvnCheckOutConsumer.java?rev=1306607&r1=1306606&r2=1306607&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/checkout/SvnCheckOutConsumer.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/checkout/SvnCheckOutConsumer.java Wed Mar 28 21:40:46 2012
@@ -19,16 +19,16 @@ package org.apache.maven.scm.provider.sv
  * under the License.
  */
 
-import java.io.File;
-import java.util.ArrayList;
-import java.util.List;
-
 import org.apache.commons.lang.StringUtils;
 import org.apache.maven.scm.ScmFile;
 import org.apache.maven.scm.ScmFileStatus;
 import org.apache.maven.scm.log.ScmLogger;
 import org.apache.maven.scm.provider.svn.svnexe.command.AbstractFileCheckingConsumer;
 
+import java.io.File;
+import java.util.ArrayList;
+import java.util.List;
+
 /**
  * @author <a href="mailto:trygvis@inamo.no">Trygve Laugst&oslash;l</a>
  * @author Olivier Lamy
@@ -40,13 +40,15 @@ public class SvnCheckOutConsumer
     private static final String CHECKED_OUT_REVISION_TOKEN = "Checked out revision";
 
     private List<ScmFile> files = new ArrayList<ScmFile>();
-    
+
     public SvnCheckOutConsumer( ScmLogger logger, File workingDirectory )
     {
         super( logger, workingDirectory );
     }
-    
-    /** {@inheritDoc} */
+
+    /**
+     * {@inheritDoc}
+     */
     protected void parseLine( String line )
     {
         String statusString = line.substring( 0, 1 );
@@ -94,7 +96,7 @@ public class SvnCheckOutConsumer
     {
         return getFiles();
     }
-    
+
     protected void addFile( ScmFile file )
     {
         files.add( file );
@@ -105,14 +107,18 @@ public class SvnCheckOutConsumer
         List<ScmFile> onlyFiles = new ArrayList<ScmFile>();
         for ( ScmFile file : files )
         {
-            if (!( !file.getStatus().equals( ScmFileStatus.DELETED )
-                && !new File(getWorkingDirectory(), file.getPath() ).isFile() ))
+            System.out.println( "workingDir:" + getWorkingDirectory().getPath() + ",path:" + file.getPath() );
+            // second part is for svn 1.7 as the co output is now relative not a full path as for svn 1.7-
+            if ( !( !file.getStatus().equals( ScmFileStatus.DELETED ) && !new File( getWorkingDirectory(),
+                                                                                    file.getPath() ).isFile() ) || !(
+                !file.getStatus().equals( ScmFileStatus.DELETED ) && !new File( getWorkingDirectory().getParent(),
+                                                                                file.getPath() ).isFile() ) )
             {
                 onlyFiles.add( file );
             }
         }
 
         return onlyFiles;
-    }        
-        
+    }
+
 }