 Relationships |
|
| Anonymous | Login | Signup for a new account | 2012-09-13 19:31 PDT |
| Main | Blog | My View | View Issues | Change Log | Roadmap | IRC Chat | Repositories | Scrum Board |
| View Issue Details [ Jump to Notes ] [ Related Changesets ] | [ Issue History ] [ Print ] | |||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | |||
| 027 | Source Integration | WebSVN | public | 2009-04-20 12:06 | 2009-04-22 10:50 | |||
| Reporter | Peter Snoblin | |||||||
| Assigned To | John Reese | |||||||
| Priority | normal | Severity | major | Reproducibility | always | |||
| Status | resolved | Resolution | fixed | |||||
| Product Version | 0.13 | |||||||
| Target Version | 0.14 | Fixed in Version | 0.14 | |||||
| Summary | 027: Revision history parsing for users with non "\w+" names | |||||||
| Description | When 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 Information | The 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. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
|
Relationships |
|
 Relationships |
|
Notes |
|
 (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... |
 (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... |
 (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. |
|
(017) John Reese (administrator) 2009-04-22 10:50 |
Fix committed to master branch at 2009-04-22 17:46:16. |
|
Notes |
| 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 ] | ||
| Related Changesets |
|
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 => |
|
Issue History |
| Copyright © 2000 - 2012 MantisBT Group
Time: 0.1114 seconds. memory usage: 8,358 KB |
 |
