[ 
https://issues.apache.org/jira/browse/GERONIMO-5624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12913732#action_12913732
 ] 

David Jencks commented on GERONIMO-5624:
----------------------------------------

I'll review this patch in more detail soon but using WebAppInfoBuilder in some 
security builder tests is IMO not a good enough reason to move it from deploy 
time to runtime code.  I think the tests can  either be moved to 
web-2.5-deployment or changed to construct an info tree directly.  Is there any 
other reason to move this code to geronimo-web?

> use info tree for tomcat web app deployment
> -------------------------------------------
>
>                 Key: GERONIMO-5624
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-5624
>             Project: Geronimo
>          Issue Type: Improvement
>      Security Level: public(Regular issues) 
>          Components: Tomcat
>    Affects Versions: 3.0
>            Reporter: David Jencks
>            Assignee: David Jencks
>             Fix For: 3.0
>
>         Attachments: GERONIMO-5624.patch
>
>
> We can avoid firing up a digester instance for tomcat by using the same info 
> tree as for jetty.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to