Maven 1.x Changelog Plugin
  1. Maven 1.x Changelog Plugin
  2. MPCHANGELOG-34

Plugin does not find the password for the CVSROOT

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4
    • Fix Version/s: None
    • Labels:
      None
    • Environment:
      Windows XP and 2000, cvs client version 1.11.2
    • Number of attachments :
      1

      Description

      Plugin does not find the password for the CVSROOT in .cvspass if the CVSROOT in .cvspass is preceded by a "/1". I do not know what the "/1" means, but is was placed their by CVS.

      ***

      • From Maven's DEBUG logging. Notice the "Didn't find password"
      • message
        ***
        maven-changelog-plugin:report:
        [echo] Generating the changelog report
        Didn't find password for CVSROOT ':pserver:anoncvs@castor.exolab.org:/cvs/castor'.
        org.netbeans.lib.cvsclient.connection.AuthenticationException: Authentication failed. Response from server was: "E PAM authenticate error: Authentication failure".
        ChangeLog found: 0 entries
        attaining goal build:end

      ***

      • From .cvspass
        ***
        /1 :pserver:anoncvs@castor.exolab.org:2401/cvs/castor Ay=0=h<Z

        Activity

        Hide
        Paul Spencer added a comment -

        This problem was introduced in rc2. I did not have this problem in rc1 or beta-10

        Show
        Paul Spencer added a comment - This problem was introduced in rc2. I did not have this problem in rc1 or beta-10
        Hide
        analogue added a comment -

        Patch while loop in CVSConnection.lookupPassword() as follows:

        while ((line = reader.readLine()) != null)
        {
        if (line.startsWith("/1 "))
        line = line.substring("/1 ".length());

        if (line.startsWith(cvsRoot))

        { password = line.substring(cvsRoot.length() + 1); break; }

        }

        Show
        analogue added a comment - Patch while loop in CVSConnection.lookupPassword() as follows: while ((line = reader.readLine()) != null) { if (line.startsWith("/1 ")) line = line.substring("/1 ".length()); if (line.startsWith(cvsRoot)) { password = line.substring(cvsRoot.length() + 1); break; } }
        Hide
        Gary Gregory added a comment -

        I am seing the same "Didn't find password for CVSROOT" in RC-3. Is the "/1" an index since .cvspass can have multiple entries?

        Show
        Gary Gregory added a comment - I am seing the same "Didn't find password for CVSROOT" in RC-3. Is the "/1" an index since .cvspass can have multiple entries?
        Hide
        Rafael Santiago, Jr. added a comment -

        Currently, it seems that the plugin only onderstands cvspass lines in the format created by running "maven -Dpassword=XXXXXX changelog:create-cvspass", namely:

        1. The line in CVS cannot start with /1 .
        2. The line has to match what you specify in project.xml. In other words, if you have a port number in your <repository><connection>, it HAS to be in the .cvspass. If you don't specify a port, it can't be in the .cvspass line either.

        Show
        Rafael Santiago, Jr. added a comment - Currently, it seems that the plugin only onderstands cvspass lines in the format created by running "maven -Dpassword=XXXXXX changelog:create-cvspass", namely: 1. The line in CVS cannot start with /1 . 2. The line has to match what you specify in project.xml. In other words, if you have a port number in your <repository><connection>, it HAS to be in the .cvspass. If you don't specify a port, it can't be in the .cvspass line either.
        Hide
        ksaito added a comment -

        I made the patch.

        Show
        ksaito added a comment - I made the patch.
        Hide
        dion gillard added a comment -

        This looks good.

        Show
        dion gillard added a comment - This looks good.

          People

          • Assignee:
            Unassigned
            Reporter:
            Paul Spencer
          • Votes:
            5 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: