Issue Details (XML | Word | Printable)

Key: MPCHANGELOG-70
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Henning Schmiedehausen
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Maven 1.x Changelog Plugin

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

Created: 10/Aug/05 12:25 PM   Updated: 03/Oct/05 12:15 PM   Resolved: 03/Oct/05 12:15 PM
Return to search
Component/s: None
Affects Version/s: 1.8.2
Fix Version/s: 1.9

Time Tracking:
Not Specified

File Attachments: 1. Text File author-whitespace.patch (4 kB)
2. File patch (0.8 kB)



 Description  « Hide

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.



Henning Schmiedehausen added a comment - 10/Aug/05 12:26 PM

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


Henning Schmiedehausen made changes - 10/Aug/05 12:26 PM
Field Original Value New Value
Attachment patch [ 16183 ]
Arnaud Bailly added a comment - 03/Oct/05 01:11 AM

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


Arnaud Bailly made changes - 03/Oct/05 01:11 AM
Attachment author-whitespace.patch [ 16882 ]
Lukas Theussl made changes - 03/Oct/05 11:27 AM
Fix Version/s 1.9 [ 12040 ]
Lukas Theussl added a comment - 03/Oct/05 12:15 PM

Patch applied. Thanks!


Lukas Theussl made changes - 03/Oct/05 12:15 PM
Status Open [ 1 ] Closed [ 6 ]
Resolution Fixed [ 1 ]