Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-1676

CVS password causes CVSSCM.isUpdatableModule() to yield incorrect result

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • cvs-plugin
    • None
    • Platform: All, OS: All

      If a hudson project includes a username and password in the root (e.g.
      :pserver:USER:PWD@host:/cvsroot). However, CVS (or at least the version we use -
      CVSNT 2.5.03) does not store the password in the CVS/Root file. The comparison
      between the project defined root and the root in CVS/Root
      (:pserver:USER@host:/cvsroot) is therefore always false and causes project to be
      refreshed completely and built every night.

      Suggested solution: Only compare password if password is specified in the
      CVS/Root folder.

      Logging suggestion: Log the reason why isUpdatableModule() returns false. There
      is absolutely no indication of why the server builds the same project over and
      over again. The only way I could trace this was to actually find the message in
      source code and work my way back in the code.

          [JENKINS-1676] CVS password causes CVSSCM.isUpdatableModule() to yield incorrect result

          john_bester_attix5 created issue -
          Michael Clarke made changes -
          Assignee New: Michael Clarke [ mc1arke ]
          Michael Clarke made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Michael Clarke made changes -
          Fix Version/s New: current [ 10162 ]
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Closed [ 6 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 131749 ] New: JNJira + In-Review [ 200900 ]

            mc1arke Michael Clarke
            john_bester_attix5 john_bester_attix5
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: