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

The Subversion parser of the maven-changelog-plugin chokes on spaces in usernames.

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.2
    • Fix Version/s: 1.9
    • Labels:
      None
    • Number of attachments :
      2

      Description

      We use client certificates for authentication to the web server which holds our SVN repository. This results in the changelog messages containing the CN field of the certificate; in my case e.g. it is "Henning Schmiedehausen".

      Subversion now produces log files like this:

      ------------------------------------------------------------------------
      r237 | Henning Schmiedehausen | 2005-08-10 13:02:39 +0200 (Wed, 10 Aug 2005) | 3 lines

      and the changelog report is empty. The reason for this is a bad regular expression in the SvnLogParser which does not expect user names to contain
      blanks.

      The attached patch fixes this by using a greedy RE. Changelog now works for me again. Please consider this patch to be part of the (not yet released?!?) 1.8.2 or 1.8.3 release.

      1. author-whitespace.patch
        4 kB
        Arnaud Bailly
      2. patch
        0.8 kB
        Henning Schmiedehausen

        Activity

        Hide
        Henning Schmiedehausen added a comment -

        patch vs. changelog-plugin 1.8.2 to fix the "space in usernames" issue.

        Show
        Henning Schmiedehausen added a comment - patch vs. changelog-plugin 1.8.2 to fix the "space in usernames" issue.
        Hide
        Arnaud Bailly added a comment -

        Another patch with different (more general) RE and fixes to test cases

        Show
        Arnaud Bailly added a comment - Another patch with different (more general) RE and fixes to test cases
        Hide
        Lukas Theussl added a comment -

        Patch applied. Thanks!

        Show
        Lukas Theussl added a comment - Patch applied. Thanks!

          People

          • Assignee:
            Unassigned
            Reporter:
            Henning Schmiedehausen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: