From 76e4715d2825275675fb0e7595babe0c9e552ff5 Mon Sep 17 00:00:00 2001
From: brett <brett@13f79535-47bb-0310-9956-ffa450edef68>
Date: Sat, 4 Jun 2005 04:41:50 +0000
Subject: [PATCH] PR: MPCHANGELOG-57 parse (no author) correctly from SVN logs

git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@179944 13f79535-47bb-0310-9956-ffa450edef68
---
 .../org/apache/maven/svnlib/SvnChangeLogParser.java    | 10 +++++-----
 changelog/src/test-resources/svnlib/svnlog.txt         |  2 +-
 .../apache/maven/svnlib/SvnChangeLogParserTest.java    |  2 +-
 changelog/xdocs/changes.xml                            |  1 +
 4 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/changelog/src/main/org/apache/maven/svnlib/SvnChangeLogParser.java b/changelog/src/main/org/apache/maven/svnlib/SvnChangeLogParser.java
index 320c42d7..b0b812c7 100644
--- a/changelog/src/main/org/apache/maven/svnlib/SvnChangeLogParser.java
+++ b/changelog/src/main/org/apache/maven/svnlib/SvnChangeLogParser.java
@@ -44,7 +44,7 @@ import org.apache.regexp.RESyntaxException;
  * @author <a href="mailto:dion@multitask.com.au">dIon Gillard</a>
  * @author <a href="mailto:pete-apache-dev@kazmier.com">Pete Kazmier</a>
  * @author Daniel Rall
- * @version $Id: SvnChangeLogParser.java,v 1.6 2004/07/08 08:36:51 evenisse Exp $
+ * @version $Id$
  */
 class SvnChangeLogParser implements ChangeLogParser
 {
@@ -82,10 +82,10 @@ class SvnChangeLogParser implements ChangeLogParser
     /** The pattern used to match svn header lines */
     private static final String pattern =
         "^r(\\d+)\\s+\\|\\s+" +          // revision number
-        "(\\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
+        "(\\(\\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 
 
     /** Current status of the parser */
     private int status = GET_HEADER;
diff --git a/changelog/src/test-resources/svnlib/svnlog.txt b/changelog/src/test-resources/svnlib/svnlog.txt
index ff95eb8b..aa1af793 100644
--- a/changelog/src/test-resources/svnlib/svnlog.txt
+++ b/changelog/src/test-resources/svnlib/svnlog.txt
@@ -42,7 +42,7 @@ Changed paths:
 Added a ctags target in the event I have to use vim.
 
 ------------------------------------------------------------------------
-r9 | kaz | 2002-08-23 11:11:52 -0400 (Fri, 23 Aug 2002) | 3 lines
+r9 | (no author) | 2002-08-24 00:11:52 +0900 (Fri, 23 Aug 2002) | 3 lines
 Changed paths:
    M /poolserver/trunk/build.xml
 
diff --git a/changelog/src/test/org/apache/maven/svnlib/SvnChangeLogParserTest.java b/changelog/src/test/org/apache/maven/svnlib/SvnChangeLogParserTest.java
index 11539a57..2da70a9f 100644
--- a/changelog/src/test/org/apache/maven/svnlib/SvnChangeLogParserTest.java
+++ b/changelog/src/test/org/apache/maven/svnlib/SvnChangeLogParserTest.java
@@ -90,7 +90,7 @@ public class SvnChangeLogParserTest extends TestCase
 
         entry = (ChangeLogEntry) entries.get(6);
         assertEquals("Entry 6 was parsed incorrectly", 
-                "kaz\n" +
+                "(no author)\n" +
                 DATE.parse("Fri Aug 23 11:11:52 EDT 2002") + "\n" +
                 "[/poolserver/trunk/build.xml, 9]\n" +
                 "Testing script out again ...\n\n",
diff --git a/changelog/xdocs/changes.xml b/changelog/xdocs/changes.xml
index c7a3b5f7..c4e9a3b0 100644
--- a/changelog/xdocs/changes.xml
+++ b/changelog/xdocs/changes.xml
@@ -26,6 +26,7 @@
   </properties>
   <body>
     <release version="1.8-SNAPSHOT" date="in SVN">
+      <action dev="brett" type="fix" issue="MPCHANGELOG-57" due-to="Sebastian Scholze">Correctly parse (no author) from SVN logs</action>
       <action dev="brett" type="fix" issue="MPCHANGELOG-53" due-to="Kenneth Leider">Add CVS monitor support</a>
       <action dev="brett" type="fix" issue="MPCHANGELOG-47">Quote argument with &lt; in it on Windows for the CVS provider - required if using CVSNT</action>
       <action dev="brett" type="fix" issue="MPCHANGELOG-55">Fix problem parsing revisions on Linux introduced by MPCHANGELOG-29</action>