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

Set CVSROOT consistently in all commands in cvs-provider

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          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:

              Slack

                Issue deployment