[ 
http://issues.apache.org/jira/browse/GERONIMO-1657?page=comments#action_12368870
 ] 

Jacek Laskowski commented on GERONIMO-1657:
-------------------------------------------

Upon applying the patch and doing 'maven -o clean default' I got:

java:compile:
<depend closure="false" srcdir="1.4" dump="false" 
destdir="c:\oss\geronimo\modules\deploy-jsr88/target/classes"></depend>    
[echo] Compiling to c:\oss\geronimo\modules\dep
loy-jsr88/target/classes
    [javac] Compiling 24 source files to 
C:\oss\geronimo\modules\deploy-jsr88\target\classes
    [javac] 
C:\oss\geronimo\modules\deploy-jsr88\src\java\org\apache\geronimo\deployment\plugin\jmx\JMXDeploymentManager.java:164:
 unreported exception java.lang.Exception;
 must be caught or declared to be thrown
    [javac]             CommandSupport.addWebURLs(kernel, result);
    [javac]                                      ^
    [javac] 1 error

Would you please get rid of that so I could apply the patch? Please create a 
patch in the top directory of Geronimo's source tree or in the module itself 
(modules/deploy-jsr88 in this case).

> CommandSupport doesn't bubble up the exception. Prints stacktrace.
> ------------------------------------------------------------------
>
>          Key: GERONIMO-1657
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1657
>      Project: Geronimo
>         Type: Bug
>   Components: deployment
>     Reporter: Prasad Kashyap
>  Attachments: Command.patch
>
> C:\Apache\geronimo\modules\deploy-jsr88\src\java\org\apache\geronimo\deployment\plugin\local\
> CommandSupport.addWebURLs() prints the stacktrace of an exception. It should 
> be bubbled up to determine the correct status of the deploy.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to