You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm-commits@maven.apache.org by br...@apache.org on 2005/01/11 10:32:12 UTC

cvs commit: maven-scm/maven-scm-providers/maven-scm-provider-svn/src/main/java/org/apache/maven/scm/provider/svn/command/add SvnAddCommand.java

brett       2005/01/11 01:32:12

  Modified:    maven-scm-providers/maven-scm-provider-svn/src/main/java/org/apache/maven/scm/provider/svn/command/add
                        SvnAddCommand.java
  Log:
  fix subversion add: must not be recursive
  
  Revision  Changes    Path
  1.3       +3 -1      maven-scm/maven-scm-providers/maven-scm-provider-svn/src/main/java/org/apache/maven/scm/provider/svn/command/add/SvnAddCommand.java
  
  Index: SvnAddCommand.java
  ===================================================================
  RCS file: /home/cvs/maven-scm/maven-scm-providers/maven-scm-provider-svn/src/main/java/org/apache/maven/scm/provider/svn/command/add/SvnAddCommand.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- SvnAddCommand.java	3 Jan 2005 03:19:56 -0000	1.2
  +++ SvnAddCommand.java	11 Jan 2005 09:32:12 -0000	1.3
  @@ -90,6 +90,8 @@
   
           cl.createArgument().setValue( "add" );
   
  +        cl.createArgument().setValue( "-N" );
  +
           for ( int i = 0; i < files.length; i++ )
           {
               cl.createArgument().setValue( files[i].getPath().replace( '\\', '/' ) );
  
  
  

Re: cvs commit: maven-scm/maven-scm-providers/maven-scm-provider-svn/src/main/java/org/apache/maven/scm/provider/svn/command/add SvnAddCommand.java

Posted by Brett Porter <br...@apache.org>.
Emmanuel Venisse wrote:

>>  +        cl.createArgument().setValue( "-N" );
>>  +
>>    
>>
>
>I prefer  "--non-recursive" instead of "-N" parameter. The -N is great for
>use it directly, but in our case it's created by a program and we can show
>it in console or log. So for a user, I think that is more simple to read a
>non simplified parameter.
>
>Emmanuel
>
>
>  
>
good point. feel free to make the change.

Re: cvs commit: maven-scm/maven-scm-providers/maven-scm-provider-svn/src/main/java/org/apache/maven/scm/provider/svn/command/add SvnAddCommand.java

Posted by Emmanuel Venisse <em...@venisse.net>.
>
>   +        cl.createArgument().setValue( "-N" );
>   +

I prefer  "--non-recursive" instead of "-N" parameter. The -N is great for
use it directly, but in our case it's created by a program and we can show
it in console or log. So for a user, I think that is more simple to read a
non simplified parameter.

Emmanuel