[ 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.