[ 
https://issues.apache.org/jira/browse/SCM-745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17962860#comment-17962860
 ] 

ASF GitHub Bot commented on SCM-745:
------------------------------------

jira-importer commented on issue #956:
URL: https://github.com/apache/maven-scm/issues/956#issuecomment-2964635942

   **[Michael 
Osipov](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=michael-o)**
 commented
   
   Fixed with 
[fb4e948aff7f21e77fef669e0364ea4e9afb2f2d](https://gitbox.apache.org/repos/asf?p=maven-scm.git;a=commit;h=fb4e948aff7f21e77fef669e0364ea4e9afb2f2d).
   




> Perforce blame result parsing too greedy
> ----------------------------------------
>
>                 Key: SCM-745
>                 URL: https://issues.apache.org/jira/browse/SCM-745
>             Project: Maven SCM (Moved to GitHub Issues)
>          Issue Type: Bug
>          Components: maven-scm-provider-perforce
>    Affects Versions: 1.8.1, 1.9
>            Reporter: Todd Currie
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: 1.11.1
>
>         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
(v8.20.10#820010)

Reply via email to