[ 
https://jira.codehaus.org/browse/MNG-5479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=339905#comment-339905
 ] 

Jason van Zyl commented on MNG-5479:
------------------------------------

This is easy to fix, and can be added to the code for the 
AbstractMavenLifecycleParticipant#afterSessionEnd
                
> ExecutionEvent.Type.SessionEnded omited when runtime exception thrown
> ---------------------------------------------------------------------
>
>                 Key: MNG-5479
>                 URL: https://jira.codehaus.org/browse/MNG-5479
>             Project: Maven 2 & 3
>          Issue Type: Bug
>          Components: Embedding, IDEs
>    Affects Versions: 3.0.4, 3.0.5
>            Reporter: Milos Kleint
>             Fix For: 3.2
>
>         Attachments: maven.patch
>
>
> in LifecycleStarter.java the ExecutionEvent.Type.SessionStarted is always 
> fired, but ExecutionEvent.Type.SessionEnded appears only be fired when in 
> some cases, omitted in case of RuntimeException thrown. IMHO the event should 
> be fired in finally {} block. See attached patch.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to