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

SvnChangeLogParser doesn't parse my log files correctly.

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.2
    • Fix Version/s: 1.8.1
    • Labels:
      None
    • Environment:
      WindowsXP, Maven 1.0.2, Subversion 1.1.3, Subclipse 0.9.25, Eclipse 3.0
    • Number of attachments :
      0

      Description

      I have parsing-log problems.

      The header pattern in SvnChangeLogParser.java doesn't match headers in Subversion log files in my environment.
      A header line is like this.

      r6 | (no author) | 2005-02-02 15:30:55 +0900 ($

      {day in Japanese}

      , 02 2 2005) | 1 line

      1. Author fields could be "(no author)".
      2. Gmt offset could be "+".

      Status remains GET_HEADER and entries have never been added .

        Activity

        Hide
        Brett Porter added a comment -

        (Sebastian Scholze)

        I Had the same problem with "(no author)".
        I changed the variable pattern to fix this issue (at least for me)

        Here is what I did:

        private static final String pattern =
        "^r(\\d+)\\s+\\|
        s+" + // revision number
        "(\\(\\S+\\s+\\S+\\)|\\S+)\\s+\\|
        s+" + // author username
        "(\\d+\\d+
        d+ " + // date 2002-08-24
        "\\d+:\\d+:
        d+) " + // time 16:01:00
        "(\\-+)(\\d\\d)(\\d
        d)"; // gmt offset -0400

        The RE is now checking for a username or for a string like "(no author)"

        Show
        Brett Porter added a comment - (Sebastian Scholze) I Had the same problem with "(no author)". I changed the variable pattern to fix this issue (at least for me) Here is what I did: private static final String pattern = "^r(\\d+)\\s+\\| s+" + // revision number "(\\(\\S+\\s+\\S+\\)|\\S+)\\s+\\| s+" + // author username "(\\d+ \\d+ d+ " + // date 2002-08-24 "\\d+:\\d+: d+) " + // time 16:01:00 "( \\-+ )(\\d\\d)(\\d d)"; // gmt offset -0400 The RE is now checking for a username or for a string like "(no author)"

          People

          • Assignee:
            Unassigned
            Reporter:
            ai abe
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: