Uploaded image for project: 'Subversion'
  1. Subversion
  2. SVN-1687

cvs2svn should warn, not exit, on invalid RCS files

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • all
    • cvs2svn-1.0
    • tools
    • None

    Description

      Jack Repenning points out:
      
         There are cases where cvs2svn aborts due to surprises in the ,v files.
         In some cases, I don't think this is the right behavior; it would be
         better to ignore the anomaly and press on, with a warning at the end.
         Here are the two such cases I've spotted:
      
         L106, "branch XXX already has name YYY, cannot also have name ZZZ"
         L157, "XXX is not a valid tag or branch name"
      
      The second problem may be partially addressed by r8168's resolution of issue
      #1524.  But there are probably other instances where cvs2svn exits when it could
      just warn, ignore, and move on instead.  Since conversions take so long, this
      will often be a preferable behavior.
      
      Regarding the first problem, Jack threw out this idea:
      
         The mult-named branch thing is similar, though I haven't (yet)
         encountered it in [my repository]: cvs allows it, it's clear
         enough that it's a problem for SVN, but a notice at the end seems
         sufficient.  In this case, I imagine I'd simply copy the branch
         to the additional name (as a manual finishing-up process).  Maybe
         I'd rename the one you chose with the other name, 'cause that's
         the one I really want. ...  But anyway, the conversion would be
         over, most people could get back to work, and there would be no
         data lost.
      

      Attachments

        1. 2_1687-patch.txt
          5 kB
          Karl Fogel
        2. 1_cvstagsize.tgz
          11 kB
          Subversion Importer

        Issue Links

          Activity

            People

              Unassigned Unassigned
              kfogel Karl Fogel
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: