[
https://jira.codehaus.org/browse/SCM-745?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Todd Currie updated SCM-745:
----------------------------
Attachment: maven-scm-provider-perforce-blame-auther.patch
Required patch with test to fix invalid greedy match in author parsing.
> Perforce Blame Result Parsing Error
> -----------------------------------
>
> Key: SCM-745
> URL: https://jira.codehaus.org/browse/SCM-745
> Project: Maven SCM
> Issue Type: Bug
> Components: maven-scm-provider-perforce
> Affects Versions: 1.8.1, 1.9
> Reporter: Todd Currie
> Attachments: maven-scm-provider-perforce-blame-auther.patch
>
>
> The regex for the parsing of the author information from the command line
> results is incorrect. It is overly greedy and is consuming additional
> characters depending upon the change comments entered by the author.
> org.apache.maven.scm.provider.perforce.command.blame.PerforceFilelogConsumer
> 39c39
> < private static final String LINE_PATTERN = "#(\\d+).*on (.*) by (.*)@";
> ---
> > private static final String LINE_PATTERN = "#(\\d+).*?on (.*?) by
> > (.*?)@";
> Example of failure:
> Changelist: 624405
> Date: 2013/07/19 14:05
> Client: ws_advertising
> User: mccabej
> Description:
> Fix retry order so that the @Retryable proxy always executes first.
> Resulting Author is "mccabej@ws_advertising (ktext) 'Fix retry order so that
> the "
--
This message was sent by Atlassian JIRA
(v6.1.6#6162)
