Author: evenisse Date: Wed Sep 21 06:53:23 2005 New Revision: 290698 URL: http://svn.apache.org/viewcvs?rev=290698&... Log: Fix cvs authentication with pserver: o Fix cvsroot comparaison with entries in .cvspass o Add default cvs port in cvsroot for pserver Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/CvsScmProvider.java maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/command/login/CvsLoginCommand.java maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/command/login/CvsPass.java maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/repository/CvsScmProviderRepository.java Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/CvsScmProvider.java URL: http://svn.apache.org/viewcvs/maven/scm/trunk... ============================================================================== --- maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/CvsScmProvider.java (original) +++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/CvsScmProvider.java Wed Sep 21 06:53:23 2005 @@ -246,6 +246,9 @@ if ( transport.equalsIgnoreCase( TRANSPORT_PSERVER ) ) { + // set default port, it's necessary for checking entries in .cvspass + port = 2401; + if ( tokens.length == 4 ) { String userhost = tokens[1]; Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/command/login/CvsLoginCommand.java URL: http://svn.apache.org/viewcvs/maven/scm/trunk... ============================================================================== --- maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/command/login/CvsLoginCommand.java (original) +++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/command/login/CvsLoginCommand.java Wed Sep 21 06:53:23 2005 @@ -49,7 +49,7 @@ CvsPass passGenerator = new CvsPass( getLogger() );- passGenerator.setCvsroot( repo.getCvsRoot() ); + passGenerator.setCvsroot( repo.getCvsRootForCvsPass() ); passGenerator.setPassword( repo.getPassword() ); try Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/command/login/CvsPass.java URL: http://svn.apache.org/viewcvs/maven/scm/trunk... ============================================================================== --- maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/command/login/CvsPass.java (original) +++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/command/login/CvsPass.java Wed Sep 21 06:53:23 2005 @@ -109,14 +109,13 @@ while ( ( line = reader.readLine() ) != null ) { - if ( !line.startsWith( cvsRoot ) ) + if ( !line.startsWith( cvsRoot ) && !line.startsWith( "/1 " + cvsRoot ) ) { -// buf.append( line ).append( System.getProperty("line.separator") ); buf.append( line ).append( "\n" ); } else { - logger.debug( "cvsroot " + cvsRoot + " already exist in " + passFile.getAbsolutePath() ); + logger.debug( "cvsroot " + cvsRoot + " already exist in " + passFile.getAbsolutePath() + ". SKIPPED." ); return; } Modified: maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/repository/CvsScmProviderRepository.java URL: http://svn.apache.org/viewcvs/maven/scm/trunk... ============================================================================== --- maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/repository/CvsScmProviderRepository.java (original) +++ maven/scm/trunk/maven-scm-providers/maven-scm-provider-cvs/src/main/java/org/apache/maven/scm/provider/cvslib/repository/CvsScmProviderRepository.java Wed Sep 21 06:53:23 2005 @@ -73,6 +73,11 @@ return cvsroot; }+ public String getCvsRootForCvsPass() + { + return ":" + transport + ":" + getUser() + "@" + host + ":" + port + path; + } + /** * @return The subtype (like pserver). */