Notice: this is a static mirror for historical purposes.

View Issue Details Jump to Notes ] Related Changesets ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
027Source IntegrationWebSVNpublic2009-04-20 12:062009-04-22 10:50
ReporterPeter Snoblin 
Assigned ToJohn Reese 
Product Version0.13 
Target Version0.14Fixed in Version0.14 
Summary027: Revision history parsing for users with non "\w+" names
DescriptionWhen importing the history of a repository where the username of the revision author does not match the "\w+" regexp fails to import any data. This is a particular problem when the repository was created via cvs2svn, as the initial revision of such a repository will have been authored by "(no author)" (and the '(' character doesn't match \w)
Additional InformationThe offending file is SourceWebSVN.php:303
To get it working, I change the '(\w+)' portion of the regexp to '([^|]+)'.
I haven't tested extensively, and I suspect that it may not perform as well (there will likely be at least some backtracking), but it worked for me.
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
User avatar (014)
Harald Hotz-Behofsits (reporter)
2009-04-20 23:52

I have/had a similar issue. ReImporting an exported subset of a subversion repository failed until I commented out in process_svn_log the break after # Should only happen at the end...
User avatar (015)
Peter Snoblin (reporter)
2009-04-21 06:28

As an aside, using the '--xml' option to 'svn log' along with e.g. PHP's SimpleXML would alleviate the entire need for parsing the revision log by hand. Of course, then you'd have to deal with XML...
User avatar (016)
John Reese (administrator)
2009-04-22 10:43

I think the regex solution you provided should solve the problem. I will get this fixed shortly.
User avatar (017)
John Reese (administrator)
2009-04-22 10:50

Fix committed to master branch at 2009-04-22 17:46:16.

- Related Changesets
Source Integration: master 90e7ace6
Timestamp: 2009-04-22 17:46:16
Author: John Reese
Committer: John Reese
Details ] Diff ]
Fix 027: Use a more permissive regex for extracting commit username.
mod - SourceWebSVN/SourceWebSVN.php Diff ] File ]

- Issue History
Date Modified Username Field Change
2009-04-20 12:06 Peter Snoblin New Issue
2009-04-20 23:52 Harald Hotz-Behofsits Note Added: 014
2009-04-21 06:28 Peter Snoblin Note Added: 015
2009-04-22 10:43 John Reese Note Added: 016
2009-04-22 10:43 John Reese Assigned To => John Reese
2009-04-22 10:43 John Reese Status new => assigned
2009-04-22 10:43 John Reese Target Version => 0.16
2009-04-22 10:50 John Reese Note Added: 017
2009-04-22 10:50 John Reese Status assigned => resolved
2009-04-22 10:50 John Reese Fixed in Version => 0.16
2009-04-22 10:50 John Reese Resolution open => fixed
2009-04-22 10:50 John Reese Changeset attached master 90e7ace6 =>

Copyright © 2000 - 2012 MantisBT Group
Time: 0.1114 seconds.
memory usage: 8,358 KB
Powered by Mantis Bugtracker

hosted with