Maven 2 & 3

ExecutionEvent give on the exception encountered (when having mojoFailed)

Details

  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

It could be usefull for having the exception details in ExecutionListener impls to have something like (to not wait the end of the maven execution) :

ExecutionEvent.getException().

could be done with
MojoExecution.getException()
will need some stuff for projectFailed
but at least could be interesting for mojo executions.
WDYT ?

Issue Links

relates to

Improvement - An improvement or enhancement to an existing feature or task. MNG-4926 ExecutionEvent give on the exception encountered (when having projectFailed , forkedProjectFailed)

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Benjamin Bentmann made changes -
Field Original Value New Value
Fix Version/s Issues to be reviewed for 3.x [ 13145 ]
Affects Version/s 3.0.1 [ 16331 ]
Affects Version/s Issues to be reviewed for 3.x [ 13145 ]
Hide
Permalink
Benjamin Bentmann added a comment -

Good idea, go for it!

Show
Benjamin Bentmann added a comment - Good idea, go for it!
Olivier Lamy made changes -
Assignee Olivier Lamy [ olamy ]
Hide
Permalink
Olivier Lamy added a comment -

so reduce the scope as having the information for projectFailed will need some API change.
Changing ExecutionEvent interface by adding a new method getException.

Show
Olivier Lamy added a comment - so reduce the scope as having the information for projectFailed will need some API change. Changing ExecutionEvent interface by adding a new method getException.
Olivier Lamy made changes -
Summary ExecutionEvent give on the exception encountered (when having mojoFailed projectFailed etc ..) ExecutionEvent give on the exception encountered (when having mojoFailed)
Olivier Lamy made changes -
Fix Version/s 3.0.2 [ 16952 ]
Fix Version/s Issues to be reviewed for 3.x [ 13145 ]
Benjamin Bentmann made changes -
Link This issue relates to MNG-4926 [ MNG-4926 ]
Hide
Permalink
Olivier Lamy added a comment -

fixed rev 1042189

Show
Olivier Lamy added a comment - fixed rev 1042189
Olivier Lamy made changes -
Status Open [ 1 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
Hide
Permalink
Benjamin Bentmann added a comment -

Revised in r1042200 for consistency with MNG-4926.

Show
Benjamin Bentmann added a comment - Revised in r1042200 for consistency with MNG-4926 .

People

  • Assignee:
    Olivier Lamy
    Reporter:
    Olivier Lamy
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: