Uploaded image for project: 'Maven SCM'
  1. Maven SCM
  2. SCM-330

Bug when trying to commit a large number of source files using the SVN command line tool as a part of the Maven2 Release plugin

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Won't Fix
    • 1.0-beta-3
    • None
    • maven-scm-provider-svn
    • None
    • Windows XP

    Description

      The SVN command generated by the "createCommandLine" method in the SvnCheckInCommand class does not consider the maximum length of the generated "svn commit" command line imposed by Windows XP, see http://support.microsoft.com/kb/830473

      If using the SCM-SVN provider implementation as a part of the Maven2 Release plugin on a windows XP platform and you are trying to commit a lot of modified pom.xml's the SCM-SVN provider implementation will fail since it does not consider the maximum length of the generated "svn commit" command line imposed by the underlying operating system.

      We have implemented a temporary fix for this problem in the "createCommandLine" method of the SvnCheckInCommand class by simply removing the code line:
      SvnCommandLineUtils.addFiles( cl, fileSet.getFiles() );
      from the "createCommandLine" method in release 1.0-beta-3.

      When studying the implementation of the "createCommandLine" method in release 1.0 it seems though this new release also has the same problem with handling "svn commit" command lines of great lengh.

      Would it not be possible to have "createCommandLine" method simply generate an implicit "svn commit" command line and not generate an explicit "svn commit" as currently implemented ... in this way the length of the commandl ine will never exceed the limits imposed by say Windows XP ?

      Regards
      Jorgen Fastrup

      Attachments

        Activity

          People

            Unassigned Unassigned
            agentjfp Jorgen Fastrup
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: