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

Set CVSROOT consistently in all commands in cvs-provider

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.0-beta-3
    • 1.0-beta-3
    • maven-scm-provider-cvs
    • None

    Description

      Most of the commands in the cvs provider use the command line argument "-d CVSROOT", but not all of them.
      The attached patch adds "-d CVSROOT" to these commands:

      • CvsDiffCommand
      • CvsStatusCommand
      • CvsUpdateCommand

      This change also made it necessary to alter the CvsUpdateCommandTest so that it works on Windows.
      That test now uses CvsScmTestUtils.getScmUrl() to get an scmUrl.

      CvsScmTestUtils.getScmUrl() has been corrected to use the vertical bar "|" as the delimiter in all places of the scm url.

      Attachments

        1. SCM-134.patch
          8 kB
          dennislundberg

        Activity

          People

            evenisse Emmanuel Venisse
            dennisl@apache.org Dennis Lundberg
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: